From: Eric Wong <e@80x24.org>
Date: Sun, 7 Feb 2021 08:51:47 +0000 (+0000)
Subject: ipc: do not die inside wq_worker child process
X-Git-Tag: v1.7.0~1180
X-Git-Url: http://www.git.stargrave.org/?a=commitdiff_plain;h=41045a11867303e5949f557dd37061923c37c202;p=public-inbox.git

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.
---

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;