diff options
author | Chris Michael <cp.michael@samsung.com> | 2015-06-25 14:59:28 -0400 |
---|---|---|
committer | Chris Michael <cp.michael@samsung.com> | 2015-06-25 14:59:28 -0400 |
commit | 30363956f1d1cb8d0457ccf784400baa2ff8f5c8 (patch) | |
tree | 2d406ea5a6de04df740cb057a77f62ce0cc85ae8 /src/bin/edje | |
parent | 6949d1193eb7f3f7d9218157e78cdf1de5ba8c23 (diff) |
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.michael@samsung.com>
Diffstat (limited to 'src/bin/edje')
-rwxr-xr-x | src/bin/edje/edje_cc_out.c | 2 |
1 files changed, 1 insertions, 1 deletions
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) | |||
1065 | } | 1065 | } |
1066 | else | 1066 | else |
1067 | { | 1067 | { |
1068 | if (iw) free(iw); | 1068 | free(iw); |
1069 | error_and_abort_image_load_error | 1069 | error_and_abort_image_load_error |
1070 | (ef, img->entry, load_err); | 1070 | (ef, img->entry, load_err); |
1071 | exit(1); // ensure static analysis tools know we exit | 1071 | exit(1); // ensure static analysis tools know we exit |