From: Christopher Michael <cp.michael@samsung.com>
Date: Thu, 2 May 2019 11:47:22 +0000 (+0200)
Subject: evas_test_image: Fix resource leak
X-Git-Tag: accepted/tizen/unified/20190509.041000~58
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=552b4381afa04a4c8123cfc00452b6b539674005;p=platform%2Fupstream%2Fefl.git

evas_test_image: Fix resource leak

Summary:
Coverity reports that we leak variable 'f' going out of scope here, so
if we are going to exit the test, lets close the file before we do

Fixes CID1400945

@fix
Depends on D8770

Reviewers: raster, cedric, zmike, bu5hm4n, segfaultxavi

Reviewed By: segfaultxavi

Subscribers: segfaultxavi, #reviewers, #committers

Tags: #efl

Differential Revision: https://phab.enlightenment.org/D8771
---

diff --git a/src/tests/evas/evas_test_image.c b/src/tests/evas/evas_test_image.c
index 259cff7..19ae8ff 100644
--- a/src/tests/evas/evas_test_image.c
+++ b/src/tests/evas/evas_test_image.c
@@ -647,6 +647,7 @@ _file_to_memory(const char *filename, char **result)
    if ((size_t)size != fread(*result, sizeof(char), size, f))
      {
         free(*result);
+        fclose(f);
         return -1;
      }
    fclose(f);