Skip to content

Commit 5b7ca1c

Browse files
Merge pull request #870 from staniewzki/utils-coverity
Fix `utils_linux.cpp` coverity defects
2 parents aeab266 + b7cf6f0 commit 5b7ca1c

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

test/utils/utils_linux.cpp

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,17 +23,19 @@ TEST_F(test, utils_shm_open_invalid_args) {
2323

2424
TEST_F(test, utils_get_file_size_invalid_args) {
2525
size_t size;
26-
auto ret = utils_get_file_size(-1, &size);
26+
auto ret = utils_get_file_size(0xffffff, &size);
2727
EXPECT_EQ(ret, -1);
2828

2929
int fd = utils_create_anonymous_fd();
30+
ASSERT_GE(fd, 0);
31+
3032
ret = utils_get_file_size(fd, &size);
3133
EXPECT_EQ(ret, 0);
3234
EXPECT_EQ(size, 0);
3335
}
3436

3537
TEST_F(test, utils_set_file_size_invalid_args) {
36-
auto ret = utils_set_file_size(-1, 256);
38+
auto ret = utils_set_file_size(0xffffff, 256);
3739
EXPECT_EQ(ret, -1);
3840
}
3941

0 commit comments

Comments
 (0)