[Pkg-xfce-commits] r9581 - in /goodies/trunk/lightdm/debian: changelog patches/08_reset-SIGPIPE-before-exec.patch patches/series
Yves-Alexis Perez
corsac at moszumanska.debian.org
Wed Oct 19 18:51:57 UTC 2016
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
More information about the Pkg-xfce-commits
mailing list