[M108 Migration] Fix svace warning 41/289241/5
authorjiangyuwei <yuwei.jiang@samsung.com>
Fri, 3 Mar 2023 06:44:55 +0000 (14:44 +0800)
committerBot Blink <blinkbot@samsung.com>
Mon, 13 Mar 2023 07:33:11 +0000 (07:33 +0000)
Release the value from app_control_get_extra_data.

Reference:
 - https://review.tizen.org/gerrit/#/c/267099/
 - https://review.tizen.org/gerrit/#/c/265555/

Change-Id: I6697cccaea5c10272440b583aed8432f76b1fc6d
Signed-off-by: jiangyuwei <yuwei.jiang@samsung.com>
tizen_src/ewk/efl_integration/public/ewk_context.cc

index 3df8af5..4e0e4a1 100644 (file)
@@ -1014,6 +1014,8 @@ Eina_Bool ewk_context_app_control_set(const Ewk_Context* context, void* app_cont
                                        &notification_id);
   if (ret != APP_CONTROL_ERROR_NONE) {
     LOG(ERROR) << "app_control_get_extra_data is failed with err " << ret;
+    if (notification_id)
+      free(notification_id);
     return EINA_FALSE;
   }
 
@@ -1023,11 +1025,13 @@ Eina_Bool ewk_context_app_control_set(const Ewk_Context* context, void* app_cont
       char* notification_origin = nullptr;
       app_control_get_extra_data(control, "notification_origin",
                                  &notification_origin);
-      cbce->GetNotificationController()->
-          PersistentNotificationClicked(context->browser_context(),
-                                        notification_id,
-                                        notification_origin);
+      if (notification_origin) {
+        cbce->GetNotificationController()->PersistentNotificationClicked(
+            context->browser_context(), notification_id, notification_origin);
+        free(notification_origin);
+      }
     }
+    free(notification_id);
   }
   return EINA_TRUE;
 #else