diff options
| author | Stefan Schmidt <s.schmidt@samsung.com> | 2020-06-09 14:29:35 +0200 |
|---|---|---|
| committer | Stefan Schmidt <s.schmidt@samsung.com> | 2020-06-10 10:07:59 +0200 |
| commit | b209bb051b16cfb5b20ac6070cd3400c46996388 (patch) | |
| tree | 0fd213720b5f7f32a45edbe648b65cd06ef8ce23 | |
| parent | cfbdcdc9c81c5cd14d8dc9f1db4d102d8052b473 (diff) | |
| download | efl-b209bb051b16cfb5b20ac6070cd3400c46996388.tar.gz | |
tests: evas: check fd being non -1 before using close()
eina_file_mkstemp would return -1 in an error case. Make sure we check
the return when clsoing here.
CID: 1400790
Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11957
| -rw-r--r-- | src/tests/evas/evas_test_image.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/tests/evas/evas_test_image.c b/src/tests/evas/evas_test_image.c index 7ee16048c8..8106a0869e 100644 --- a/src/tests/evas/evas_test_image.c +++ b/src/tests/evas/evas_test_image.c @@ -904,7 +904,8 @@ EFL_START_TEST(evas_object_image_map_unmap) // save file, verify its pixels fd = eina_file_mkstemp("/tmp/evas-test.XXXXXX.png", &tmp); - close(fd); + fail_if(fd <= 0); + fail_if(close(fd)); if (efl_file_save(o, tmp, NULL, NULL)) { Eina_Rw_Slice sorig, sdest; |