aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--main/asterisk.c18
-rw-r--r--utils/astcanary.c4
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;
}