Attachment 88460 Details for Bug 60859 – [PATCH 1/2] Use SIGHUP without check in UNIX environment

[patch] [PATCH 1/2] Use SIGHUP without check in UNIX environment

0001-Use-SIGHUP-without-check-in-UNIX-environment.patch (text/plain), 1.06 KB, created by Chengwei Yang on 2013-11-01 08:37:30 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Chengwei Yang

Created: 2013-11-01 08:37:30 UTC

Size: 1.06 KB

patch

obsolete

>From ac957139584b3a3986e5e648244563e8b7c050c7 Mon Sep 17 00:00:00 2001
>From: Chengwei Yang <chengwei.yang@intel.com>
>Date: Fri, 1 Nov 2013 16:23:27 +0800
>Subject: [PATCH 1/2] Use SIGHUP without check in UNIX environment
>
>As Simon's comment
>  https://bugs.freedesktop.org/show_bug.cgi?id=66068#c8
>we can do this in UNIX environment.
>---
> bus/main.c |    4 ----
> 1 file changed, 4 deletions(-)
>
>diff --git a/bus/main.c b/bus/main.c
>index 472458e..e060baa 100644
>--- a/bus/main.c
>+++ b/bus/main.c
>@@ -61,7 +61,6 @@ signal_handler (int sig)
> {
>   switch (sig)
>     {
>-#ifdef SIGHUP
>     case SIGHUP:
>       {
>         DBusString str;
>@@ -94,7 +93,6 @@ signal_handler (int sig)
>           }
>       }
>       break;
>-#endif
> 
>     case SIGTERM:
>       {
>@@ -642,9 +640,7 @@ main (int argc, char **argv)
>    * no point in trying to make the handler portable to non-Unix. */
> 
>   _dbus_set_signal_handler (SIGTERM, signal_handler);
>-#ifdef SIGHUP
>   _dbus_set_signal_handler (SIGHUP, signal_handler);
>-#endif
> #endif /* DBUS_UNIX */
> 
>   _dbus_verbose ("We are on D-Bus...\n");
>-- 
>1.7.9.5
>

Actions: View | Splinter Review

Attachments on bug 60859: 74834 | 74835 | 74836 | 74837 | 74838 | 74839 | 74840 | 88459 | 88460 | 88461

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.