diff options
author | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2009-02-24 20:07:36 +0000 |
---|---|---|
committer | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2009-02-24 20:07:36 +0000 |
commit | 888f601479c424bb4b0378d14f19b33c826f9cfa (patch) | |
tree | a21b1b05238c845d0d248969d0fc2423b342ee6f | |
parent | 05eff19e0f159334c01570f0ab2b4eface7d5be0 (diff) |
Merged revisions 178342 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk
........
r178342 | tilghman | 2009-02-24 14:06:48 -0600 (Tue, 24 Feb 2009) | 2 lines
Use a SIGPIPE to kill the process, instead of depending upon the astcanary process being inherited by init.
........
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.0@178343 f38db490-d61c-443f-a65b-d21fe96a405b
-rw-r--r-- | main/asterisk.c | 18 | ||||
-rw-r--r-- | utils/astcanary.c | 4 |
2 files changed, 19 insertions, 3 deletions
diff --git a/main/asterisk.c b/main/asterisk.c index ffc85a9cf..b69f046e4 100644 --- a/main/asterisk.c +++ b/main/asterisk.c @@ -260,6 +260,7 @@ static int restartnow; static pthread_t consolethread = AST_PTHREADT_NULL; static int canary_pid = 0; static char canary_filename[128]; +static int canary_pipe = -1; static char randompool[256]; @@ -3113,8 +3114,17 @@ int main(int argc, char *argv[]) if (isroot) { ast_set_priority(ast_opt_high_priority); if (ast_opt_high_priority) { + int cpipe[2]; + + /* PIPE signal ensures that astcanary dies when Asterisk dies */ + pipe(cpipe); + canary_pipe = cpipe[0]; + snprintf(canary_filename, sizeof(canary_filename), "%s/alt.asterisk.canary.tweet.tweet.tweet", ast_config_AST_RUN_DIR); + /* Don't let the canary child kill Asterisk, if it dies immediately */ + signal(SIGPIPE, SIG_IGN); + canary_pid = fork(); if (canary_pid == 0) { char canary_binary[128], *lastslash; @@ -3122,9 +3132,14 @@ int main(int argc, char *argv[]) /* Reset signal handler */ signal(SIGCHLD, SIG_DFL); + signal(SIGPIPE, SIG_DFL); - for (fd = 0; fd < 100; fd++) + dup2(cpipe[1], 100); + close(cpipe[1]); + + for (fd = 0; fd < 100; fd++) { close(fd); + } execlp("astcanary", "astcanary", canary_filename, (char *)NULL); @@ -3139,6 +3154,7 @@ int main(int argc, char *argv[]) _exit(1); } else if (canary_pid > 0) { pthread_t dont_care; + close(cpipe[1]); ast_pthread_create_detached(&dont_care, NULL, canary_thread, NULL); } diff --git a/utils/astcanary.c b/utils/astcanary.c index bb3492af7..68557beaa 100644 --- a/utils/astcanary.c +++ b/utils/astcanary.c @@ -89,7 +89,7 @@ int main(int argc, char *argv[]) int fd; /* Run at normal priority */ setpriority(PRIO_PROCESS, 0, 0); - for (; getppid() != 1;) { + for (;;) { /* Update the modification times (checked from Asterisk) */ if (utime(argv[1], NULL)) { /* Recreate the file if it doesn't exist */ @@ -108,7 +108,7 @@ int main(int argc, char *argv[]) sleep(5); } - /* Reached if asterisk (our parent process) dies - its chldren are inherited by the init process (pid is 1). */ + /* Never reached */ return 0; } |