e_dbus: do not print log when closing dbus connection. 20/209720/1 accepted/tizen_5.5_unified accepted/tizen_5.5_unified_mobile_hotfix accepted/tizen_5.5_unified_wearable_hotfix accepted/tizen_6.0_unified_hotfix accepted/tizen_6.5_unified accepted/tizen_7.0_unified accepted/tizen_7.0_unified_hotfix accepted/tizen_8.0_unified accepted/tizen_unified tizen tizen_5.5 tizen_5.5_mobile_hotfix tizen_5.5_tv tizen_5.5_wearable_hotfix tizen_6.0 tizen_6.0_hotfix tizen_6.5 tizen_7.0 tizen_7.0_hotfix tizen_8.0 accepted/tizen/5.5/unified/20191031.005227 accepted/tizen/5.5/unified/mobile/hotfix/20201027.062139 accepted/tizen/5.5/unified/wearable/hotfix/20201027.102423 accepted/tizen/6.0/unified/hotfix/20201102.235654 accepted/tizen/6.5/unified/20211028.224004 accepted/tizen/7.0/unified/20221110.060119 accepted/tizen/7.0/unified/hotfix/20221116.110726 accepted/tizen/8.0/unified/20231005.094811 accepted/tizen/unified/20190711.062857 submit/tizen/20190710.100447 submit/tizen_5.5/20191031.000006 submit/tizen_5.5_mobile_hotfix/20201026.185106 submit/tizen_5.5_wearable_hotfix/20201026.184306 submit/tizen_6.0/20201029.205501 submit/tizen_6.0_hotfix/20201102.192901 submit/tizen_6.0_hotfix/20201103.115101 submit/tizen_6.5/20211028.163301 tizen_5.5.m2_release tizen_6.0.m2_release tizen_6.5.m2_release tizen_7.0_m2_release tizen_8.0_m2_release
authorJongmin Lee <jm105.lee@samsung.com>
Wed, 10 Jul 2019 07:55:54 +0000 (16:55 +0900)
committerJongmin Lee <jm105.lee@samsung.com>
Wed, 10 Jul 2019 07:55:54 +0000 (16:55 +0900)
After calling e_dbus_shutdown(), dbus connection could be closed.
In this case, if edbus tries to print log, then edbus get crashed
because log domain is invalid (if abort_on_error option is enabled).

Change-Id: I227750fa288a4b97f68ca5001eba55b87f2e24d9

src/lib/dbus/e_dbus.c

index 47d84bb..627732f 100644 (file)
@@ -60,7 +60,6 @@ e_dbus_fd_handler_del(E_DBus_Handler_Data *hd)
 {
   if (!hd->fd_handler) return;
 
-  DBG("handler disabled");
   hd->cd->fd_handlers = eina_list_remove(hd->cd->fd_handlers, hd->fd_handler);
   ecore_main_fd_handler_del(hd->fd_handler);
   hd->fd_handler = NULL;
@@ -130,8 +129,7 @@ static void
 e_dbus_handler_data_free(void *data)
 {
   E_DBus_Handler_Data *hd = data;
-  
-  DBG("e_dbus_handler_data_free");
+
   if (hd->fd_handler)
   {
     hd->cd->fd_handlers = eina_list_remove(hd->cd->fd_handlers, hd->fd_handler);
@@ -192,7 +190,6 @@ e_dbus_connection_free(void *data)
   E_DBus_Connection *cd = data;
   Ecore_Fd_Handler *fd_handler;
   Ecore_Timer *timer;
-  DBG("e_dbus_connection free!");
 
   EINA_LIST_FREE(cd->fd_handlers, fd_handler)
     ecore_main_fd_handler_del(fd_handler);
@@ -217,7 +214,6 @@ cb_dispatch_status(DBusConnection *conn __UNUSED__, DBusDispatchStatus new_statu
 {
   E_DBus_Connection *cd;
 
-  DBG("dispatch status: %d!", new_status);
   cd = data;
 
   if (new_status == DBUS_DISPATCH_DATA_REMAINS && !cd->idler)
@@ -335,7 +331,6 @@ cb_watch_del(DBusWatch *watch, void *data __UNUSED__)
 {
   E_DBus_Handler_Data *hd;
 
-  DBG("cb_watch_del");
   hd = (E_DBus_Handler_Data *)dbus_watch_get_data(watch);
   e_dbus_fd_handler_del(hd);
 }
@@ -395,7 +390,6 @@ e_dbus_filter(DBusConnection *conn __UNUSED__, DBusMessage *message, void *user_
     default:
       break;
   }
-  DBG("-----------------");
 
   return DBUS_HANDLER_RESULT_NOT_YET_HANDLED;
 }
@@ -517,7 +511,6 @@ EAPI void
 e_dbus_connection_close(E_DBus_Connection *conn)
 {
   if (!conn) return;
-  DBG("e_dbus_connection_close");
 
   if (e_dbus_idler_active)
   {