Fix: Tests: unchecked `close()` return value
authorFrancis Deslauriers <francis.deslauriers@efficios.com>
Mon, 1 Nov 2021 19:31:26 +0000 (15:31 -0400)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Fri, 17 Dec 2021 04:45:28 +0000 (23:45 -0500)
CID 1465101 (#1 of 1): Unchecked return value (CHECKED_RETURN)
9. check_return: Calling close without checking return value (as is done
elsewhere 177 out of 185 times).

CID 1465100 (#1 of 1): Unchecked return value (CHECKED_RETURN)
4. check_return: Calling close without checking return value (as is done
elsewhere 177 out of 185 times)

CID 1465099 (#1 of 1): Unchecked return value (CHECKED_RETURN) 4.
check_return: Calling close without checking return value (as is done
elsewhere 177 out of 185 times).

CID 1465098 (#1 of 1): Unchecked return value (CHECKED_RETURN) 4.
check_return: Calling close without checking return value (as is done
elsewhere 177 out of 185 times).

CID 1465097 (#1 of 1): Unchecked return value (CHECKED_RETURN) 4.
check_return: Calling close without checking return value (as is done
elsewhere 177 out of 185 times).

Reported-by: Coverity Scan
Signed-off-by: Francis Deslauriers <francis.deslauriers@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Change-Id: I8e2552c75ab7cec5aa3707e2c1c4d9f2484b501a

tests/regression/kernel/select_poll_epoll.c

index b1eaf6045f9b3b16dcffc9bee88f61b0d358197b..d0d7e4aeafae124566b73157c2565a4aef4a1bcb 100644 (file)
@@ -302,7 +302,10 @@ void test_epoll(void)
        }
 
 error:
-       close(epollfd);
+       ret = close(epollfd);
+       if (ret) {
+               perror("close");
+       }
 end:
        return;
 }
@@ -347,7 +350,10 @@ void test_pepoll(void)
        }
 
 error:
-       close(epollfd);
+       ret = close(epollfd);
+       if (ret) {
+               perror("close");
+       }
 end:
        return;
 }
@@ -579,7 +585,10 @@ void epoll_pwait_invalid_pointer(void)
        }
 
 error:
-       close(epollfd);
+       ret = close(epollfd);
+       if (ret) {
+               perror("close");
+       }
 end:
        return;
 }
@@ -625,7 +634,10 @@ void epoll_pwait_int_max(void)
        }
 
 error:
-       close(epollfd);
+       ret = close(epollfd);
+       if (ret) {
+               perror("close");
+       }
 end:
        return;
 }
@@ -840,7 +852,10 @@ error_unmap:
        }
 
 error:
-       close(epollfd);
+       ret = close(epollfd);
+       if (ret) {
+               perror("close");
+       }
 end:
        return;
 }
This page took 0.027417 seconds and 5 git commands to generate.