diff options
Diffstat (limited to 'source3/printing/spoolssd.c')
| -rw-r--r-- | source3/printing/spoolssd.c | 24 |
1 files changed, 20 insertions, 4 deletions
diff --git a/source3/printing/spoolssd.c b/source3/printing/spoolssd.c index 774f67da1b..4379179f02 100644 --- a/source3/printing/spoolssd.c +++ b/source3/printing/spoolssd.c @@ -713,14 +713,27 @@ pid_t start_spoolssd(struct tevent_context *ev_ctx, DEBUG(1, ("Forking SPOOLSS Daemon\n")); + /* + * Block signals before forking child as it will have to + * set its own handlers. Child will re-enable SIGHUP as + * soon as the handlers are set up. + */ + BlockSignals(true, SIGTERM); + BlockSignals(true, SIGHUP); + pid = sys_fork(); if (pid == -1) { DEBUG(0, ("Failed to fork SPOOLSS [%s]\n", strerror(errno))); } + + /* parent or error */ if (pid != 0) { - /* parent or error */ + + /* Re-enable SIGHUP before returnig */ + BlockSignals(false, SIGTERM); + BlockSignals(false, SIGHUP); return pid; } @@ -738,6 +751,12 @@ pid_t start_spoolssd(struct tevent_context *ev_ctx, spoolss_reopen_logs(0); spoolss_prefork_config(); + spoolss_setup_sig_term_handler(ev_ctx); + spoolss_setup_sig_hup_handler(ev_ctx, msg_ctx); + + BlockSignals(false, SIGTERM); + BlockSignals(false, SIGHUP); + /* Publish nt printers, this requires a working winreg pipe */ pcap_cache_reload(ev_ctx, msg_ctx, &reload_printers); @@ -771,9 +790,6 @@ pid_t start_spoolssd(struct tevent_context *ev_ctx, &spoolss_children_main, NULL, &spoolss_pool); - spoolss_setup_sig_term_handler(ev_ctx); - spoolss_setup_sig_hup_handler(ev_ctx, msg_ctx); - if (!serverid_register(procid_self(), FLAG_MSG_GENERAL|FLAG_MSG_SMBD |FLAG_MSG_PRINT_GENERAL)) { |