Avoid propagating shutdown signal to all subprocesses

This was a sort-of-regression with 7233f68

While this behavior might seem desirable in some cases, multi-user
installs like public servers would rather not kill children while
upgrading.

Turns out that pipes are inherited by forks, and writing in one side
means there might be more than one listener that calls
bitlbee_shutdown(). If the parent gets it, the children will get it
too. If a child gets it, the parent and the other children get it too.

This adds a sighandler_shutdown_setup() function that closes any
previously existing pipes and disconnects the events from them, to
create a new one. This is called again after forking each child process.

While I'm sure this fixes the issue, I still don't understand why it
*didn't* kill the forked processes in some cases. Worrying.

1 files changed, 2 insertions, 0 deletions

diff --git a/bitlbee.h b/bitlbee.h
index 49a5496a..447e2e64 100644
--- a/bitlbee.h
+++ b/bitlbee.h
@@ -157,6 +157,8 @@ typedef struct global {
 	int restart;
 } global_t;
 
+void sighandler_shutdown_setup(void);
+
 int bitlbee_daemon_init(void);
 int bitlbee_inetd_init(void);