From 41045a11867303e5949f557dd37061923c37c202 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Sun, 7 Feb 2021 08:51:47 +0000 Subject: ipc: do not die inside wq_worker child process die() in a child zips up the stack into the parent, which is undesirable behavior. We're going to exit anyways, just warn and let exit(1) happen due to $@ being set. --- lib/PublicInbox/IPC.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'lib') diff --git a/lib/PublicInbox/IPC.pm b/lib/PublicInbox/IPC.pm index b936c27a..8f6f4ded 100644 --- a/lib/PublicInbox/IPC.pm +++ b/lib/PublicInbox/IPC.pm @@ -129,7 +129,7 @@ sub ipc_worker_spawn { local %SIG = %SIG; ipc_worker_loop($self, $r_req, $w_res); }; - die "worker $ident PID:$$ died: $@\n" if $@; + warn "worker $ident PID:$$ died: $@\n" if $@; undef $end; # trigger exit } PublicInbox::DS::sig_setmask($sigset) unless $oldset; -- cgit v1.2.3-24-ge0c7