It should no longer be necessary to ignore SIGCHLD because it is now

blocked around daemon(), and doing so causes trouble with newer libevent
(it cannot restore the original handler). Reported by Azat Khuzhin in
GitHub issue 3626.

diff --git a/usr.bin/tmux/client.c b/usr.bin/tmux/client.c
index 16373e5..f819509 100644 (file)
--- a/usr.bin/tmux/client.c
+++ b/usr.bin/tmux/client.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: client.c,v 1.160 2023/07/10 09:35:46 nicm Exp $ */
+/* $OpenBSD: client.c,v 1.161 2023/07/10 12:00:08 nicm Exp $ */
 
 /*
  * Copyright (c) 2007 Nicholas Marriott <nicholas.marriott@gmail.com>
@@ -246,9 +246,6 @@ client_main(struct event_base *base, int argc, char **argv, uint64_t flags,
        u_int                    ncaps = 0;
        struct args_value       *values;
 
-       /* Ignore SIGCHLD now or daemon() in the server will leave a zombie. */
-       signal(SIGCHLD, SIG_IGN);
-
        /* Set up the initial command. */
        if (shell_command != NULL) {
                msg = MSG_SHELL;