tizen-platform-wrapper: fix resource leak 54/47754/2
authorImran Zaman <imran.zaman@intel.com>
Tue, 8 Sep 2015 10:19:35 +0000 (13:19 +0300)
committerSunmin Lee <sunm.lee@samsung.com>
Wed, 6 Jul 2016 06:41:39 +0000 (23:41 -0700)
'sum' is free'd when it is not in use anymore

Signed-off-by: Imran Zaman <imran.zaman@intel.com>
Change-Id: I3add11bc32bc496bd0b1069310a60245cdac77d0

src/toolbox.c

index 0c48f36..35dfd3d 100644 (file)
@@ -703,14 +703,19 @@ static int signup( FILE *output)
     term = ';';
     for (key = keys ; key != NULL ; key = key->next) {
         status = sha256sum_add_data(sum, key->name, strlen(key->name));
-        if (status < 0)
+        if (status < 0) {
+            sha256sum_destroy(sum);
             return status;
+        }
         status = sha256sum_add_data(sum, &term, 1);
-        if (status < 0)
+        if (status < 0) {
+            sha256sum_destroy(sum);
             return status;
+        }
     }
 
     status = sha256sum_get(sum, signup);
+    sha256sum_destroy(sum);
     if (status < 0)
         return status;