From 784dcbc125c3c6090e4778f05f7276a3d2f1fc87 Mon Sep 17 00:00:00 2001 From: Donald Buczek Date: Wed, 5 Jul 2017 12:25:24 +0200 Subject: [PATCH] mxqd: Remove reset_signals It is no longer necessary to reset signals, because we no longer change them. The signals may stay blocked for the reaper process and are explicitly unblocked for the user process. --- mxqd.c | 19 ------------------- 1 file changed, 19 deletions(-) diff --git a/mxqd.c b/mxqd.c index e8449552..f5964a01 100644 --- a/mxqd.c +++ b/mxqd.c @@ -701,21 +701,6 @@ int server_init(struct mxq_server *server, int argc, char *argv[]) return 0; } -static void reset_signals() -{ - signal(SIGINT, SIG_DFL); - signal(SIGTERM, SIG_DFL); - signal(SIGQUIT, SIG_DFL); - signal(SIGHUP, SIG_DFL); - signal(SIGTSTP, SIG_DFL); - signal(SIGTTIN, SIG_DFL); - signal(SIGTTOU, SIG_DFL); - signal(SIGCHLD, SIG_DFL); - signal(SIGPIPE, SIG_DFL); - signal(SIGUSR1, SIG_DFL); - signal(SIGUSR2, SIG_DFL); -} - static int init_child_process(struct mxq_group_list *glist, struct mxq_job *job) { struct mxq_server *server; @@ -733,8 +718,6 @@ static int init_child_process(struct mxq_group_list *glist, struct mxq_job *job) server = glist->user->server; group = &glist->group; - reset_signals(); - sigprocmask(SIG_UNBLOCK,&all_signals,NULL); passwd = getpwuid(group->user_uid); @@ -1059,8 +1042,6 @@ int reaper_process(struct mxq_server *server,struct mxq_group_list *glist, struc group = &glist->group; - reset_signals(); - res = setsid(); if (res < 0) { mx_log_warning("reaper_process setsid: %m");