devilhorns pushed a commit to branch master.
http://git.enlightenment.org/core/efl.git/commit/?id=30363956f1d1cb8d0457ccf784400baa2ff8f5c8
commit 30363956f1d1cb8d0457ccf784400baa2ff8f5c8
Author: Chris Michael <cp.mich...@samsung.com>
Date: Thu Jun 25 14:59:28 2015 -0400
edje_cc: Fix dereference before null check
Summary: Fix Coverity CID1302700 which reports a dereference before
null check. Basically, it is safe to call free() with a possible NULL
value so we can remove the 'if' check here.
@fix
Signed-off-by: Chris Michael <cp.mich...@samsung.com>
---
src/bin/edje/edje_cc_out.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/bin/edje/edje_cc_out.c b/src/bin/edje/edje_cc_out.c
index be534aa..24066ad 100755
--- a/src/bin/edje/edje_cc_out.c
+++ b/src/bin/edje/edje_cc_out.c
@@ -1065,7 +1065,7 @@ data_write_images(Eet_File *ef, int *image_num)
}
else
{
- if (iw) free(iw);
+ free(iw);
error_and_abort_image_load_error
(ef, img->entry, load_err);
exit(1); // ensure static analysis tools know we exit
--
