From 30363956f1d1cb8d0457ccf784400baa2ff8f5c8 Mon Sep 17 00:00:00 2001 From: Chris Michael Date: Thu, 25 Jun 2015 14:59:28 -0400 Subject: [PATCH] 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 --- 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 be534aab5f..24066ad2ef 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