Skip to content

Commit 2502dec

Browse files
Merge pull request #888 from staniewzki/fix-coverity-assert
Add explicit condition check for Coverity
2 parents ac823bb + 912be58 commit 2502dec

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

test/utils/utils_linux.cpp

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,9 +29,12 @@ TEST_F(test, utils_get_file_size_invalid_args) {
2929
int fd = utils_create_anonymous_fd();
3030
ASSERT_GE(fd, 0);
3131

32-
ret = utils_get_file_size(fd, &size);
33-
EXPECT_EQ(ret, 0);
34-
EXPECT_EQ(size, 0);
32+
// Explicit condition for coverity
33+
if (fd >= 0) {
34+
ret = utils_get_file_size(fd, &size);
35+
EXPECT_EQ(ret, 0);
36+
EXPECT_EQ(size, 0);
37+
}
3538
}
3639

3740
TEST_F(test, utils_set_file_size_invalid_args) {

0 commit comments

Comments
 (0)