commit e8c9b8a8e340a612b4a0a24901526982aa17d61f
author Bertrand SIMONNET <bsimonnet@google.com> Fri Dec 18 16:15:25 2015 -0800
committer Bertrand SIMONNET <bsimonnet@google.com> Fri Dec 18 16:17:33 2015 -0800
tree 58fca58532ab5edfc9ef442779dd2913c37a1e1f
parent beeafcd9a08682077dc086fe1d96dee6f3b60fd2

dbus: Raise SIGTERM when the connection drops.

Instead of exiting directly when a client notices that the connection
got dropped, raise a SIGTERM signal to allow the client to clean up and
exit cleanly.

Bug: 26267381
Test: stop dbus. Clients exit cleanly.

Change-Id: I7e1faf1aa8b761b1b05dfde82a48882f61124acb

dbus/dbus-connection.c

diff --git a/dbus/dbus-connection.c b/dbus/dbus-connection.c
index fb52955..0362288 100644
--- a/dbus/dbus-connection.c
+++ b/dbus/dbus-connection.c
@@ -22,6 +22,7 @@
  */
 
 #include <config.h>
+#include <signal.h>
 #include "dbus-shared.h"
 #include "dbus-connection.h"
 #include "dbus-list.h"
@@ -4284,8 +4285,13 @@
           CONNECTION_UNLOCK (connection);            
           
           _dbus_verbose ("Exiting on Disconnected signal\n");
-          _dbus_exit (1);
-          _dbus_assert_not_reached ("Call to exit() returned");
+          if (raise (SIGTERM) != 0)
+            {
+              _dbus_verbose ("Failed to raise a SIGTERM signal. Exiting\n");
+              _dbus_exit (1);
+              _dbus_assert_not_reached ("Call to exit() returned");
+            }
+          return;
         }
     }