[Pkg-xfce-commits] r9581 - in /goodies/trunk/lightdm/debian: changelog patches/08_reset-SIGPIPE-before-exec.patch patches/series

[Yves-Alexis Perez]

Author: corsac
Date: Wed Oct 19 18:51:56 2016
New Revision: 9581

URL: http://svn.debian.org/wsvn/pkg-xfce/?sc=1&rev=9581
Log:
* debian/patches:
  - 08_reset-SIGPIPE-before-exec added, ignore SIGPIPE during LightDM life
    but reset it before exec()ing children.                   closes: #823460

Added:
    goodies/trunk/lightdm/debian/patches/08_reset-SIGPIPE-before-exec.patch
Modified:
    goodies/trunk/lightdm/debian/changelog
    goodies/trunk/lightdm/debian/patches/series

Modified: goodies/trunk/lightdm/debian/changelog
URL: http://svn.debian.org/wsvn/pkg-xfce/goodies/trunk/lightdm/debian/changelog?rev=9581&op=diff
==============================================================================
--- goodies/trunk/lightdm/debian/changelog	(original)
+++ goodies/trunk/lightdm/debian/changelog	Wed Oct 19 18:51:56 2016
@@ -2,6 +2,9 @@
 
   * debian/control:
     - add Suggests on xserver-xephyr.                           closes: #831298
+  * debian/patches:
+    - 08_reset-SIGPIPE-before-exec added, ignore SIGPIPE during LightDM life
+      but reset it before exec()ing children.                   closes: #823460
 
  -- Yves-Alexis Perez <corsac at debian.org>  Fri, 15 Jul 2016 21:44:54 +0200
 

Added: goodies/trunk/lightdm/debian/patches/08_reset-SIGPIPE-before-exec.patch
URL: http://svn.debian.org/wsvn/pkg-xfce/goodies/trunk/lightdm/debian/patches/08_reset-SIGPIPE-before-exec.patch?rev=9581&op=file
==============================================================================
--- goodies/trunk/lightdm/debian/patches/08_reset-SIGPIPE-before-exec.patch	(added)
+++ goodies/trunk/lightdm/debian/patches/08_reset-SIGPIPE-before-exec.patch	Wed Oct 19 18:51:56 2016
@@ -0,0 +1,45 @@
+=== modified file 'src/lightdm.c'
+--- lightdm/src/lightdm.c	2016-07-14 02:25:24 +0000
++++ lightdm/src/lightdm.c	2016-10-19 12:24:49 +0000
+@@ -1225,6 +1225,12 @@
+     };
+     GError *error = NULL;
+ 
++    /* Disable the SIGPIPE handler - this is a stupid Unix hangover behaviour.
++     * We will handle piples / sockets being closed instead of having the whole daemon be killed...
++     * http://stackoverflow.com/questions/8369506/why-does-sigpipe-exist
++     */
++    signal (SIGPIPE, SIG_IGN);
++
+     /* When lightdm starts sessions it needs to run itself in a new mode */
+     if (argc >= 2 && strcmp (argv[1], "--session-child") == 0)
+         return session_child_run (argc, argv);
+
+=== modified file 'src/process.c'
+--- lightdm/src/process.c	2015-10-20 07:50:44 +0000
++++ lightdm/src/process.c	2016-10-19 12:24:49 +0000
+@@ -236,6 +236,9 @@
+         for (i = 0; i < env_length; i++)
+             setenv (env_keys[i], env_values[i], TRUE);
+ 
++        /* Reset SIGPIPE handler so the child has default behaviour (we disabled it at LightDM start) */
++        signal (SIGPIPE, SIG_DFL);
++
+         execvp (argv[0], argv);
+         _exit (EXIT_FAILURE);
+     }
+
+=== modified file 'src/session-child.c'
+--- lightdm/src/session-child.c	2016-03-15 03:07:59 +0000
++++ lightdm/src/session-child.c	2016-10-19 12:24:49 +0000
+@@ -704,6 +704,9 @@
+             }
+         }
+ 
++        /* Reset SIGPIPE handler so the child has default behaviour (we disabled it at LightDM start) */
++        signal (SIGPIPE, SIG_DFL);
++
+         /* Run the command */
+         execve (command_argv[0], command_argv, pam_getenvlist (pam_handle));
+         _exit (EXIT_FAILURE);
+

Modified: goodies/trunk/lightdm/debian/patches/series
URL: http://svn.debian.org/wsvn/pkg-xfce/goodies/trunk/lightdm/debian/patches/series?rev=9581&op=diff
==============================================================================
--- goodies/trunk/lightdm/debian/patches/series	(original)
+++ goodies/trunk/lightdm/debian/patches/series	Wed Oct 19 18:51:56 2016
@@ -3,3 +3,4 @@
 05_debianize-pam-files.patch
 06_change-user-dirs.patch
 07-Remove-unused-unititialized-file-descriptors-could-c.patch
+08_reset-SIGPIPE-before-exec.patch