From: Paweł Stawicki Date: Tue, 20 Mar 2018 16:19:37 +0000 (+0100) Subject: [prevent][38682] Fix for resource leak X-Git-Tag: accepted/tizen/unified/20180405.005254~2 X-Git-Url: http://review.tizen.org/git/?p=platform%2Fupstream%2Fat-spi2-core.git;a=commitdiff_plain;h=d3c1fc8c5aca4fca78c0c04580c9e4532eba8ff2 [prevent][38682] Fix for resource leak Change-Id: Iea61c399f59d2ddb2b55768f325c3005e01cfcf3 --- diff --git a/atspi/atspi-event-listener.c b/atspi/atspi-event-listener.c index be04ad2..4f59939 100644 --- a/atspi/atspi-event-listener.c +++ b/atspi/atspi-event-listener.c @@ -766,6 +766,7 @@ atspi_event_listener_deregister_from_callback (AtspiEventListenerCB callback, GPtrArray *matchrule_array; gint i; GList *l; + gboolean result = TRUE; if (!convert_event_type_to_dbus (event_type, &category, &name, &detail, &matchrule_array)) { @@ -800,8 +801,10 @@ atspi_event_listener_deregister_from_callback (AtspiEventListenerCB callback, atspi_path_registry, atspi_interface_registry, "DeregisterEvent"); - if (!message) - return FALSE; + if (!message) { + result = FALSE; + break; + } dbus_message_append_args (message, DBUS_TYPE_STRING, &event_type, DBUS_TYPE_INVALID); reply = _atspi_dbus_send_with_reply_and_block (message, error); if (reply) @@ -817,7 +820,7 @@ atspi_event_listener_deregister_from_callback (AtspiEventListenerCB callback, for (i = 0; i < matchrule_array->len; i++) g_free (g_ptr_array_index (matchrule_array, i)); g_ptr_array_free (matchrule_array, TRUE); - return TRUE; + return result; } /**