Fix some more bugs in signal handlers and process shutdown logic.
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 1 Feb 2014 21:21:00 +0000 (16:21 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 1 Feb 2014 21:21:30 +0000 (16:21 -0500)
commit6f1a4077310e97fe32379263918a31b95f85e2ed
treecce96a6bb9a0589c1258f78c4d39fc45a3025e94
parent27942baf49eb3a939dde14cb81f254fbdf2b1816
Fix some more bugs in signal handlers and process shutdown logic.

WalSndKill was doing things exactly backwards: it should first clear
MyWalSnd (to stop signal handlers from touching MyWalSnd->latch),
then disown the latch, and only then mark the WalSnd struct unused by
clearing its pid field.

Also, WalRcvSigUsr1Handler and worker_spi_sighup failed to preserve
errno, which is surely a requirement for any signal handler.

Per discussion of recent buildfarm failures.  Back-patch as far
as the relevant code exists.
contrib/worker_spi/worker_spi.c
src/backend/replication/walreceiver.c
src/backend/replication/walsender.c