diff options
author | Andres Freund | 2015-01-13 15:44:09 +0000 |
---|---|---|
committer | Andres Freund | 2015-01-13 19:45:03 +0000 |
commit | 782acc75cbb540ec31a93f104d849679a108ed2b (patch) | |
tree | eb1a48fe4e13a0dcb18f0a055b9de7d737758489 | |
parent | d6d391620c5e6a6cc5e456cb4fcfa843c095b8c8 (diff) |
Since commit 626eb021988a2 has introduced the auxiliary process
infrastructure bootstrap_signals() was never used when forked from
postmaster.
Remove the IsUnderPostmaster specific code, and add a appropriate
assertion.
-rw-r--r-- | src/backend/bootstrap/bootstrap.c | 41 |
1 files changed, 6 insertions, 35 deletions
diff --git a/src/backend/bootstrap/bootstrap.c b/src/backend/bootstrap/bootstrap.c index 4c650fb0a4..a17f3ff39a 100644 --- a/src/backend/bootstrap/bootstrap.c +++ b/src/backend/bootstrap/bootstrap.c @@ -505,42 +505,13 @@ BootstrapModeMain(void) static void bootstrap_signals(void) { - if (IsUnderPostmaster) - { - /* - * Properly accept or ignore signals the postmaster might send us - */ - pqsignal(SIGHUP, SIG_IGN); - pqsignal(SIGINT, SIG_IGN); /* ignore query-cancel */ - pqsignal(SIGTERM, die); - pqsignal(SIGQUIT, quickdie); - pqsignal(SIGALRM, SIG_IGN); - pqsignal(SIGPIPE, SIG_IGN); - pqsignal(SIGUSR1, SIG_IGN); - pqsignal(SIGUSR2, SIG_IGN); - - /* - * Reset some signals that are accepted by postmaster but not here - */ - pqsignal(SIGCHLD, SIG_DFL); - pqsignal(SIGTTIN, SIG_DFL); - pqsignal(SIGTTOU, SIG_DFL); - pqsignal(SIGCONT, SIG_DFL); - pqsignal(SIGWINCH, SIG_DFL); + Assert(!IsUnderPostmaster); - /* - * Unblock signals (they were blocked when the postmaster forked us) - */ - PG_SETMASK(&UnBlockSig); - } - else - { - /* Set up appropriately for interactive use */ - pqsignal(SIGHUP, die); - pqsignal(SIGINT, die); - pqsignal(SIGTERM, die); - pqsignal(SIGQUIT, die); - } + /* Set up appropriately for interactive use */ + pqsignal(SIGHUP, die); + pqsignal(SIGINT, die); + pqsignal(SIGTERM, die); + pqsignal(SIGQUIT, die); } /* |