From: Joseph T Date: Mon, 20 Sep 2010 20:59:29 +0000 (-0400) Subject: Merge with EFL svn repo: changeset 51544: eet: more check cleans (bogus actually... X-Git-Tag: 2.0_alpha~114 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=28c42b3f51062b9be36c05744193005c5c61df1f;p=framework%2Fuifw%2Feet.git Merge ... EFL svn repo: changeset 51544: eet: more check cleans (bogus actually too - but you guys like to shut up warnings don't you?) --- diff --git a/src/lib/eet_lib.c b/src/lib/eet_lib.c index b736fc0..3fc2ac2 100644 --- a/src/lib/eet_lib.c +++ b/src/lib/eet_lib.c @@ -958,9 +958,14 @@ eet_internal_read2(Eet_File *ef) /* out directory block is inconsistent - we have oveerun our */ /* dynamic block buffer before we finished scanning dir entries */ - efn = malloc (sizeof(Eet_File_Node)); + efn = malloc(sizeof(Eet_File_Node)); if (eet_test_close(!efn, ef)) - return NULL; + { + if (efn) free(efn); /* yes i know - we only get here if + * efn is null/0 -> trying to shut up + * warning tools like cppcheck */ + return NULL; + } /* get entrie header */ GET_INT(efn->offset, data, idx); @@ -1220,7 +1225,12 @@ eet_internal_read1(Eet_File *ef) /* allocate all the ram needed for this stored node accounting */ efn = malloc (sizeof(Eet_File_Node)); if (eet_test_close(!efn, ef)) - return NULL; + { + if (efn) free(efn); /* yes i know - we only get here if + * efn is null/0 -> trying to shut up + * warning tools like cppcheck */ + return NULL; + } /* get entrie header */ EXTRACT_INT(efn->offset, p, indexn);