From 2aae648148a6c32300d9709deb24e5ca26544dd1 Mon Sep 17 00:00:00 2001 From: Hwankyu Jhun Date: Wed, 17 Oct 2018 15:46:57 +0900 Subject: [PATCH 01/16] Release version 0.1.4 Changes: - Fixed a bug about finding callback function Change-Id: Idae3e93d4a170c6fa2266ca8809b8359831fa04f Signed-off-by: Hwankyu Jhun --- packaging/eventsystem.spec | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packaging/eventsystem.spec b/packaging/eventsystem.spec index 349e4a0..96962d0 100755 --- a/packaging/eventsystem.spec +++ b/packaging/eventsystem.spec @@ -1,6 +1,6 @@ Name: eventsystem Summary: Event system library -Version: 0.1.3 +Version: 0.1.4 Release: 1 Group: Application Framework/Libraries License: Apache-2.0 -- 2.7.4 From fc572e111cef665bb39a9c335033ed486792b666 Mon Sep 17 00:00:00 2001 From: Inkyun Kil Date: Thu, 24 Jan 2019 09:10:22 +0900 Subject: [PATCH 02/16] Support launch_on_event with user event Change-Id: I586a79c2389ddace9d2c0ffaf80f4fcadd1313bc Signed-off-by: Inkyun Kil Signed-off-by: jusung son --- include/eventsystem_internal.h | 1 + src/eventsystem.c | 134 ++++++++++++++++++++++++++++++++--------- 2 files changed, 105 insertions(+), 30 deletions(-) mode change 100755 => 100644 src/eventsystem.c diff --git a/include/eventsystem_internal.h b/include/eventsystem_internal.h index ccbc2de..044f06d 100644 --- a/include/eventsystem_internal.h +++ b/include/eventsystem_internal.h @@ -26,6 +26,7 @@ extern "C" { #define ESD_INTERFACE_NAME "tizen.system.event.app2esd" #define USER_EVENT_NAME_PREFIX "event." +#define SYSTEM_EVENT_NAME_PREFIX "event://" /** * system-event definitions diff --git a/src/eventsystem.c b/src/eventsystem.c old mode 100755 new mode 100644 index aea30f5..587ef35 --- a/src/eventsystem.c +++ b/src/eventsystem.c @@ -106,13 +106,15 @@ typedef struct sysevent_info { static sysevent_info_s s_info; static int __eventsystem_check_user_send_validation(const char *event_name); -static int __eventsystem_requet_destination_list(const char *event_name, GList **dest_list); +static int __eventsystem_request_destination_list(const char *event_name, GList **dest_list); static int __eventsystem_check_sender_validation(int sender_pid, const char *event_name, char **sender); static eventmap_s *__create_eventmap(const char *interface_name, const char *member_name, const char *event_name, int event_type, eventsystem_cb callback, void *user_data); static void __destroy_eventmap(gpointer data); +static int __eventsystem_launch_on_event_for_userevent(const char *event_name, + bundle *data, const bool trusted); static int __event_compare_reg_id_cb(gconstpointer a, gconstpointer b) { @@ -871,7 +873,6 @@ static int __eventsystem_send_trusted_event(GDBusConnection *conn, eventinfo_s * } tmp_list = g_list_next(tmp_list); } - g_list_free_full(dest_list, __eventsystem_free_trusted_list); } else { _E("dest_list is null"); } @@ -884,11 +885,13 @@ static int __eventsystem_send_trusted_event(GDBusConnection *conn, eventinfo_s * */ int eventsystem_send_user_event(const char *event_name, bundle *data, bool is_trusted) { - int ret = 0; + int ret = ES_R_OK; eventinfo_s *evti = NULL; struct last_data_item *item; bundle_raw *raw = NULL; int len; + GDBusConnection *conn = NULL; + GList *trusted_dest_list = NULL; /* check validation */ retvm_if(!event_name, ES_R_EINVAL, "Invalid argument : event_name is NULL"); @@ -910,53 +913,38 @@ int eventsystem_send_user_event(const char *event_name, bundle *data, bool is_tr if (!evti->event_name) { _E("memory alloc failed"); ret = ES_R_ENOMEM; - goto out_1; + goto out; } evti->interface_name = __get_encoded_interface_name(evti->event_name); if (!evti->interface_name) { _E("interface_name is NULL"); ret = ES_R_ERROR; - goto out_2; + goto out; } evti->member_name = strdup(EVENT_SYSTEM_MEMBER); if (!evti->member_name) { _E("memory alloc failed"); ret = ES_R_ENOMEM; - goto out_3; + goto out; } evti->object_path = __get_object_path(evti->interface_name); if (!evti->object_path) { _E("object_path is NULL"); ret = ES_R_ERROR; - goto out_4; + goto out; } evti->destination_name = NULL; evti->is_user_event = true; evti->is_trusted = is_trusted; - GDBusConnection *conn = NULL; if (__get_gdbus_shared_connection(&conn, G_BUS_TYPE_SESSION, ES_TYPE_USER) == ES_R_OK) { - if (is_trusted) { - GList *dest_list = NULL; - ret = __eventsystem_requet_destination_list(evti->event_name, &dest_list); - if (ret < 0) { - _E("failed to get dest list"); - ret = ES_R_ERROR; - } else { - ret = __eventsystem_send_trusted_event(conn, evti, - data, dest_list); - } - } else { - ret = __eventsystem_send_event(conn, evti, data); - } - if (bundle_encode(data, &raw, &len)) { _E("Unable to encode bundle"); ret = ES_R_ERROR; - goto out_5; + goto out; } item = (struct last_data_item *)g_hash_table_lookup(last_data_tbl, @@ -973,23 +961,45 @@ int eventsystem_send_user_event(const char *event_name, bundle *data, bool is_tr } bundle_free_encoded_rawdata(&raw); + + if (is_trusted) { + if (__eventsystem_request_destination_list(evti->event_name, &trusted_dest_list) < 0) { + _E("failed to get dest list"); + ret = ES_R_ERROR; + goto out; + } + if (__eventsystem_send_trusted_event(conn, evti, data, trusted_dest_list) < 0) { + _E("failed to send trusted event"); + ret = ES_R_ERROR; + goto out; + } + + if (__eventsystem_launch_on_event_for_userevent(evti->event_name, data, true) < 0) + _E("Failed to launch on event for userevent"); + } else { + if (__eventsystem_send_event(conn, evti, data) < 0) { + _E("failed to send event"); + ret = ES_R_ERROR; + goto out; + } + + if (__eventsystem_launch_on_event_for_userevent(evti->event_name, data, false) < 0) + _E("Failed to launch on event for userevent"); + } } else { _E("getting gdbus-connetion error"); ret = ES_R_ERROR; } +out: if (conn) g_object_unref(conn); - -out_5: + if (trusted_dest_list) + g_list_free_full(trusted_dest_list, __eventsystem_free_trusted_list); FREE_AND_NULL(evti->object_path); -out_4: FREE_AND_NULL(evti->member_name); -out_3: FREE_AND_NULL(evti->interface_name); -out_2: FREE_AND_NULL(evti->event_name); -out_1: FREE_AND_NULL(evti); return ret; @@ -1370,7 +1380,7 @@ out_1: return ret; } -static int __eventsystem_requet_destination_list(const char *event_name, GList **dest_list) +static int __eventsystem_request_destination_list(const char *event_name, GList **dest_list) { int ret = ES_R_ERROR; GDBusConnection *conn = NULL; @@ -1572,6 +1582,70 @@ out_1: return ret; } +static int __eventsystem_launch_on_event_for_userevent(const char *event_name, + bundle *data, const bool trusted) +{ + int ret = ES_R_EINVAL; + GDBusConnection *conn = NULL; + GError *error = NULL; + GDBusProxy *proxy = NULL; + GVariant *param = NULL; + GVariant *value = NULL; + gint result = 0; + bundle_raw *raw = NULL; + bundle *buf = data; + int len; + + if (!_initialized) + __initialize(); + + if (__get_gdbus_shared_connection(&conn, G_BUS_TYPE_SYSTEM, ES_TYPE_SYSTEM) < 0) { + _E("getting gdbus-connetion error"); + ret = ES_R_ERROR; + goto out_1; + } + + proxy = g_dbus_proxy_new_sync(conn, + G_DBUS_PROXY_FLAGS_DO_NOT_LOAD_PROPERTIES, NULL, + ESD_BUS_NAME, ESD_OBJECT_PATH, ESD_INTERFACE_NAME, + NULL, &error); + if (proxy == NULL) { + _E("failed to create new proxy, error(%s)", error->message); + g_error_free(error); + ret = ES_R_ERROR; + goto out_1; + } + + bundle_encode(buf, &raw, &len); + + param = g_variant_new("(ssib)", event_name, raw, len, trusted); + value = g_dbus_proxy_call_sync(proxy, "LaunchOnEventFromUserEvent", param, + G_DBUS_CALL_FLAGS_NONE, -1, NULL, &error); + if (error != NULL) { + _E("proxy call sync error(%s)", error->message); + g_error_free(error); + ret = ES_R_ERROR; + goto out_2; + } + + _D("Launch on event from user event (%s)", event_name); + + g_variant_get(value, "(i)", &result); + + _D("result(%d)", result); + + ret = result; + +out_2: + bundle_free_encoded_rawdata(&raw); + g_object_unref(proxy); + g_variant_unref(value); +out_1: + if (conn) + g_object_unref(conn); +} + + static void _send_last_user_event(const char *event_name, bundle *data, void *user_data) { -- 2.7.4 From 9832dc3864d4e15894b070696517b59a6e8820b3 Mon Sep 17 00:00:00 2001 From: Dariusz Michaluk Date: Wed, 15 May 2019 18:25:36 +0200 Subject: [PATCH 03/16] Remove unnecessary openssl dependency Change-Id: Idcc32c23266e33831bb1d61623af81ea0a0abca0 --- CMakeLists.txt | 2 +- eventsystem.pc.in | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 5a43a4b..da3d498 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -16,7 +16,7 @@ INCLUDE_DIRECTORIES(${CMAKE_SOURCE_DIR}/include ${CMAKE_SOURCE_DIR}/src) ### Required packages INCLUDE(FindPkgConfig) -pkg_check_modules(libpkgs REQUIRED dlog bundle openssl glib-2.0 capi-base-common) +pkg_check_modules(libpkgs REQUIRED dlog bundle glib-2.0 capi-base-common) #FIND_LIBRARY(LIB_DL dl) diff --git a/eventsystem.pc.in b/eventsystem.pc.in index 85881d4..5cc84bc 100644 --- a/eventsystem.pc.in +++ b/eventsystem.pc.in @@ -8,6 +8,6 @@ includedir=${prefix}/include Name: libeventsystem Description: event system library Version: @VERSION@ -Requires: bundle openssl glib-2.0 gio-2.0 +Requires: bundle glib-2.0 gio-2.0 Libs: -L${libdir} -leventsystem Cflags: -I${includedir} -- 2.7.4 From 65f10ab32c35be8ddcbcc9022e657f0a2996b86a Mon Sep 17 00:00:00 2001 From: Inkyun Kil Date: Fri, 5 Jul 2019 08:54:25 +0900 Subject: [PATCH 04/16] Release version 0.1.5 Changes: - Remove unnecessary openssl dependency - Support launch_on_event with user event Change-Id: I97bcabb572060003a60c7a09d9f7407bbe6258b5 Signed-off-by: Inkyun Kil --- packaging/eventsystem.spec | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packaging/eventsystem.spec b/packaging/eventsystem.spec index 96962d0..f9c4360 100755 --- a/packaging/eventsystem.spec +++ b/packaging/eventsystem.spec @@ -1,6 +1,6 @@ Name: eventsystem Summary: Event system library -Version: 0.1.4 +Version: 0.1.5 Release: 1 Group: Application Framework/Libraries License: Apache-2.0 -- 2.7.4 From 775c7918b4974410e3cf82d0c59399931b728a34 Mon Sep 17 00:00:00 2001 From: Inkyun Kil Date: Thu, 11 Jul 2019 11:13:01 +0900 Subject: [PATCH 05/16] Unsubscribing signal when registration fails Change-Id: Ifdbb9d641d634e4517c1fcaa5d059eada86ae6be Signed-off-by: Inkyun Kil --- src/eventsystem.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/eventsystem.c b/src/eventsystem.c index 587ef35..3f80264 100644 --- a/src/eventsystem.c +++ b/src/eventsystem.c @@ -1966,11 +1966,15 @@ int eventsystem_register_application_event(const char *event_name, unsigned int if (s_info.own_name_session_bus == NULL) { _E("session bus is not ready"); ret = ES_R_ERROR; + g_dbus_connection_signal_unsubscribe(conn, subscription_id); + __destroy_eventmap(em); } else { if (__eventsystem_setup_trusted_peer(event_name, s_info.own_name_session_bus) < 0) { _E("failed to setup trusted peer"); ret = ES_R_ERROR; + g_dbus_connection_signal_unsubscribe(conn, subscription_id); + __destroy_eventmap(em); } } } -- 2.7.4 From d4bf4b3257261aa68cd5d1f30d3f724f67c0ada1 Mon Sep 17 00:00:00 2001 From: Inkyun Kil Date: Tue, 16 Jul 2019 09:05:42 +0900 Subject: [PATCH 06/16] Release version 0.1.6 Changes: - Unsubscribing signal when registration fails Change-Id: Ifddd088a5feabc4305288e49c281cc32c7d43e69 Signed-off-by: Inkyun Kil --- packaging/eventsystem.spec | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packaging/eventsystem.spec b/packaging/eventsystem.spec index f9c4360..8dd7ab5 100755 --- a/packaging/eventsystem.spec +++ b/packaging/eventsystem.spec @@ -1,6 +1,6 @@ Name: eventsystem Summary: Event system library -Version: 0.1.5 +Version: 0.1.6 Release: 1 Group: Application Framework/Libraries License: Apache-2.0 -- 2.7.4 From 7120c0e282cd9e1ce9b976caa1a2c195cbe6e377 Mon Sep 17 00:00:00 2001 From: Inkyun Kil Date: Thu, 8 Aug 2019 09:11:00 +0900 Subject: [PATCH 07/16] Fix stack overflow gboolean is different from bool Change-Id: I352572d00b1ea68fa86f775b0b471ae6c23ad828 Signed-off-by: Inkyun Kil --- src/eventsystem.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/eventsystem.c b/src/eventsystem.c index 3f80264..b678c79 100644 --- a/src/eventsystem.c +++ b/src/eventsystem.c @@ -114,7 +114,7 @@ static eventmap_s *__create_eventmap(const char *interface_name, int event_type, eventsystem_cb callback, void *user_data); static void __destroy_eventmap(gpointer data); static int __eventsystem_launch_on_event_for_userevent(const char *event_name, - bundle *data, const bool trusted); + bundle *data, gboolean trusted); static int __event_compare_reg_id_cb(gconstpointer a, gconstpointer b) { @@ -974,7 +974,7 @@ int eventsystem_send_user_event(const char *event_name, bundle *data, bool is_tr goto out; } - if (__eventsystem_launch_on_event_for_userevent(evti->event_name, data, true) < 0) + if (__eventsystem_launch_on_event_for_userevent(evti->event_name, data, TRUE) < 0) _E("Failed to launch on event for userevent"); } else { if (__eventsystem_send_event(conn, evti, data) < 0) { @@ -983,7 +983,7 @@ int eventsystem_send_user_event(const char *event_name, bundle *data, bool is_tr goto out; } - if (__eventsystem_launch_on_event_for_userevent(evti->event_name, data, false) < 0) + if (__eventsystem_launch_on_event_for_userevent(evti->event_name, data, FALSE) < 0) _E("Failed to launch on event for userevent"); } } else { @@ -1583,7 +1583,7 @@ out_1: } static int __eventsystem_launch_on_event_for_userevent(const char *event_name, - bundle *data, const bool trusted) + bundle *data, gboolean trusted) { int ret = ES_R_EINVAL; GDBusConnection *conn = NULL; -- 2.7.4 From dca96ac699140b262f7b5149c87efffc18f591e3 Mon Sep 17 00:00:00 2001 From: Inkyun Kil Date: Thu, 8 Aug 2019 09:48:04 +0900 Subject: [PATCH 08/16] Release version 0.1.7 Changes: - Fix stack overflow Change-Id: I62fb573b3daa4f1417d45807723283c0d66a1adf Signed-off-by: Inkyun Kil --- packaging/eventsystem.spec | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packaging/eventsystem.spec b/packaging/eventsystem.spec index 8dd7ab5..84ec09d 100755 --- a/packaging/eventsystem.spec +++ b/packaging/eventsystem.spec @@ -1,6 +1,6 @@ Name: eventsystem Summary: Event system library -Version: 0.1.6 +Version: 0.1.7 Release: 1 Group: Application Framework/Libraries License: Apache-2.0 -- 2.7.4 From 839cdcbef9e5aaa54c54a76cfdfc89c241f650b7 Mon Sep 17 00:00:00 2001 From: Jusung Son Date: Fri, 30 Aug 2019 11:16:55 +0900 Subject: [PATCH 09/16] Add missing return Change-Id: I2bd5612f38a4be7348c2455e1b61307ef85452e1 Signed-off-by: Jusung Son --- src/eventsystem.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/eventsystem.c b/src/eventsystem.c index b678c79..c9239af 100644 --- a/src/eventsystem.c +++ b/src/eventsystem.c @@ -1643,6 +1643,8 @@ out_2: out_1: if (conn) g_object_unref(conn); + + return ret; } -- 2.7.4 From 4cc09fc07f61034a7d68f20d6f57b8317b7f6f9a Mon Sep 17 00:00:00 2001 From: Jusung Son Date: Fri, 30 Aug 2019 14:00:28 +0900 Subject: [PATCH 10/16] Release version 0.1.8 Changes: - Add missing return Change-Id: I36e832db0f7d1a83f5d9a9c8495e290f476d3a4d Signed-off-by: Jusung Son --- packaging/eventsystem.spec | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) mode change 100755 => 100644 packaging/eventsystem.spec diff --git a/packaging/eventsystem.spec b/packaging/eventsystem.spec old mode 100755 new mode 100644 index 84ec09d..ea24ba8 --- a/packaging/eventsystem.spec +++ b/packaging/eventsystem.spec @@ -1,6 +1,6 @@ Name: eventsystem Summary: Event system library -Version: 0.1.7 +Version: 0.1.8 Release: 1 Group: Application Framework/Libraries License: Apache-2.0 -- 2.7.4 From 914d609b3baf3077b6b8f9d68918c08d5e736302 Mon Sep 17 00:00:00 2001 From: Inkyun Kil Date: Tue, 8 Oct 2019 15:51:25 +0900 Subject: [PATCH 11/16] Fix bug for checking last data - own_name_session_bus is null when registering a system-event. Do not need to check last event data when registering a system-event. Change-Id: I1d202c5425ea4d2e6e3f5c881f0f77dab7b39be0 Signed-off-by: Inkyun Kil --- src/eventsystem.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/eventsystem.c b/src/eventsystem.c index c9239af..4559c99 100644 --- a/src/eventsystem.c +++ b/src/eventsystem.c @@ -1986,7 +1986,8 @@ int eventsystem_register_application_event(const char *event_name, unsigned int ret = ES_R_ERROR; } - __request_esd_for_last_data(event_name, true); + if (*event_type == ES_TYPE_USER) + __request_esd_for_last_data(event_name, true); end: FREE_AND_NULL(interface_name); FREE_AND_NULL(object_path); -- 2.7.4 From a5e5aa66694a4072f583fea854623eab2538738c Mon Sep 17 00:00:00 2001 From: Inkyun Kil Date: Tue, 8 Oct 2019 16:46:27 +0900 Subject: [PATCH 12/16] Handle error for GetEalierData Related patch : https://review.tizen.org/gerrit/c/platform/core/appfw/event-system/+/215448 Change-Id: I22bd99f4393cf3e7f4f15b15fc6bc2fa3d41f448 Signed-off-by: Inkyun Kil Signed-off-by: Jusung Son --- src/eventsystem.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/eventsystem.c b/src/eventsystem.c index 4559c99..6fe1de4 100644 --- a/src/eventsystem.c +++ b/src/eventsystem.c @@ -1498,17 +1498,17 @@ static int __eventsystem_request_earlier_data(const char *event_name, goto out_2; } - g_variant_get(value, "(iis)", &result, &len, &raw); + g_variant_get(value, "(ii&s)", &result, &len, &raw); _D("result(%d), len(%d)", result, len); if (!result && raw && len > 0) { callback(event_name, raw, len, user_data); - bundle_free_encoded_rawdata(&raw); + ret = ES_R_OK; + } else { + ret = ES_R_ERROR; } - ret = ES_R_OK; - out_2: g_object_unref(proxy); g_variant_unref(value); -- 2.7.4 From 566113430a23d04c4c4d7b0f2e971bdde208aa9f Mon Sep 17 00:00:00 2001 From: Inkyun Kil Date: Thu, 17 Oct 2019 16:30:00 +0900 Subject: [PATCH 13/16] Release version 0.1.9 Changes: - Handle error for GetEalierData - Fix bug for checking last data Change-Id: Ibf732535813e2489a9efcb635f3c7e7c2a3959f4 Signed-off-by: Inkyun Kil --- packaging/eventsystem.spec | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packaging/eventsystem.spec b/packaging/eventsystem.spec index ea24ba8..7289d79 100644 --- a/packaging/eventsystem.spec +++ b/packaging/eventsystem.spec @@ -1,6 +1,6 @@ Name: eventsystem Summary: Event system library -Version: 0.1.8 +Version: 0.1.9 Release: 1 Group: Application Framework/Libraries License: Apache-2.0 -- 2.7.4 From c78c7e7b6ae13160b221a39e02e38c917c16f2dd Mon Sep 17 00:00:00 2001 From: Junghoon Park Date: Tue, 12 Nov 2019 18:06:56 +0900 Subject: [PATCH 14/16] Add dependency with esd Change-Id: Iee9d5042d31d81e1f3a2b529c5ed876f9a825e7f Signed-off-by: Junghoon Park --- packaging/eventsystem.spec | 1 + 1 file changed, 1 insertion(+) diff --git a/packaging/eventsystem.spec b/packaging/eventsystem.spec index 7289d79..8c61d0f 100644 --- a/packaging/eventsystem.spec +++ b/packaging/eventsystem.spec @@ -15,6 +15,7 @@ BuildRequires: pkgconfig(capi-base-common) Requires(post): /sbin/ldconfig Requires(postun): /sbin/ldconfig +Recommends: esd %description Event System Library -- 2.7.4 From 1027c58bbf1da9c515f5d9e6cb52daad01fa3518 Mon Sep 17 00:00:00 2001 From: Junghoon Park Date: Tue, 12 Nov 2019 18:29:32 +0900 Subject: [PATCH 15/16] Release version 0.1.10 Changes: - Add dependency with esd Change-Id: Ibbedf9d0c13340af68b36817f8855d51039b57f0 Signed-off-by: Junghoon Park --- packaging/eventsystem.spec | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packaging/eventsystem.spec b/packaging/eventsystem.spec index 8c61d0f..f608d04 100644 --- a/packaging/eventsystem.spec +++ b/packaging/eventsystem.spec @@ -1,6 +1,6 @@ Name: eventsystem Summary: Event system library -Version: 0.1.9 +Version: 0.1.10 Release: 1 Group: Application Framework/Libraries License: Apache-2.0 -- 2.7.4 From aa78f8bdccd9e5ed69cc88d5993010ec1ca5f438 Mon Sep 17 00:00:00 2001 From: Inkyun Kil Date: Thu, 2 Jan 2020 16:39:40 +0900 Subject: [PATCH 16/16] Fix build warning based on GCC-9 Change-Id: Ic7258ef723d7de380f8aafebc400a44ab97a5fcd Signed-off-by: Inkyun Kil --- src/eventsystem.c | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/src/eventsystem.c b/src/eventsystem.c index 6fe1de4..7c4d4ec 100644 --- a/src/eventsystem.c +++ b/src/eventsystem.c @@ -611,7 +611,6 @@ static int __eventsystem_register_event_internal(const char *event_name, char *interface_name; char *object_path; char *member_name; - char *sender_name = NULL; GDBusSignalCallback filter; GBusType bus_type; guint subscription_id = 0; @@ -677,7 +676,6 @@ static int __eventsystem_register_event_internal(const char *event_name, ret = ES_R_ERROR; goto out_3; } - sender_name = NULL; bus_type = G_BUS_TYPE_SYSTEM; @@ -694,7 +692,7 @@ static int __eventsystem_register_event_internal(const char *event_name, } subscription_id = g_dbus_connection_signal_subscribe(conn, - sender_name, /* sender */ + NULL, /* sender */ interface_name, member_name, /* member */ object_path, /* object_path */ @@ -1842,7 +1840,6 @@ int eventsystem_register_application_event(const char *event_name, unsigned int char *interface_name; char *object_path; char *member_name; - char *sender_name = NULL; GDBusSignalCallback filter; GBusType bus_type; guint subscription_id = 0; @@ -1918,11 +1915,10 @@ int eventsystem_register_application_event(const char *event_name, unsigned int ret = ES_R_ERROR; goto end; } - sender_name = NULL; _D("interface_name(%s), object_path(%s)", interface_name, object_path); - _D(" member_name(%s), sender_name(%s), type(%d), bus_type(%d)", - member_name, sender_name, *event_type, bus_type); + _D(" member_name(%s), type(%d), bus_type(%d)", + member_name, *event_type, bus_type); if (__get_gdbus_shared_connection(&conn, bus_type, *event_type) < 0) { _E("getting gdbus-connetion error"); @@ -1942,7 +1938,7 @@ int eventsystem_register_application_event(const char *event_name, unsigned int } subscription_id = g_dbus_connection_signal_subscribe(conn, - sender_name, /* sender */ + NULL, /* sender */ interface_name, member_name, /* member */ object_path, /* object_path */ -- 2.7.4