Don't use %m in fprintf().
[tinc] / src / process.c
index 22943c2..bf3d6b7 100644 (file)
@@ -1,7 +1,7 @@
 /*
     process.c -- process management functions
-    Copyright (C) 1999,2000 Ivo Timmermans <itimmermans@bigfoot.com>,
-                       2000 Guus Sliepen <guus@sliepen.warande.net>
+    Copyright (C) 1999-2001 Ivo Timmermans <itimmermans@bigfoot.com>,
+                  2000,2001 Guus Sliepen <guus@sliepen.warande.net>
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
@@ -17,7 +17,7 @@
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
-    $Id: process.c,v 1.1.2.19 2001/01/07 15:25:45 guus Exp $
+    $Id: process.c,v 1.1.2.23 2001/07/24 20:03:40 guus Exp $
 */
 
 #include "config.h"
@@ -164,7 +164,7 @@ cp
     {
       if(daemon(0, 0) < 0)
        {
-         fprintf(stderr, _("Couldn't detach from terminal: %m"));
+         fprintf(stderr, _("Couldn't detach from terminal: %s"), strerror(errno));
          return -1;
        }
 
@@ -221,7 +221,7 @@ cp
   /* No return on success */
   
   if(errno != ENOENT)  /* Ignore if the file does not exist */
-    exit(-1);          /* Some error while trying execl(). */
+    exit(1);           /* Some error while trying execl(). */
   else
     exit(0);
 }
@@ -309,7 +309,7 @@ sigsegv_square(int a, siginfo_t *info, void *b)
 {
   syslog(LOG_ERR, _("Got another SEGV signal: not restarting"));
   cp_trace();
-  exit(0);
+  exit(1);
 }
 
 RETSIGTYPE
@@ -375,6 +375,16 @@ unexpected_signal_handler(int a, siginfo_t *info, void *b)
   cp_trace();
 }
 
+RETSIGTYPE
+ignore_signal_handler(int a, siginfo_t *info, void *b)
+{
+  if(debug_lvl >= DEBUG_SCARY_THINGS)
+  {
+    syslog(LOG_WARNING, _("Got unexpected signal %d (%s)"), a, strsignal(a));
+    cp_trace();
+  }
+}
+
 struct {
   int signal;
   void (*handler)(int, siginfo_t *, void *);
@@ -383,11 +393,12 @@ struct {
   { SIGTERM, sigterm_handler },
   { SIGQUIT, sigquit_handler },
   { SIGSEGV, sigsegv_handler },
-  { SIGPIPE, NULL },
+  { SIGPIPE, ignore_signal_handler },
   { SIGINT, sigint_handler },
   { SIGUSR1, sigusr1_handler },
   { SIGUSR2, sigusr2_handler },
-  { SIGCHLD, NULL },
+  { SIGCHLD, ignore_signal_handler },
+  { SIGALRM, ignore_signal_handler },
   { 0, NULL }
 };
 
@@ -407,7 +418,7 @@ setup_signals(void)
   for(i = 0; i < NSIG; i++) 
     {
       act.sa_sigaction = unexpected_signal_handler;
-      sigaction(sighandlers[i].signal, &act, NULL);
+      sigaction(i, &act, NULL);
     }
 
   /* Then, for each known signal that we want to catch, assign a
@@ -416,7 +427,7 @@ setup_signals(void)
     {
       act.sa_sigaction = sighandlers[i].handler;
       if(sigaction(sighandlers[i].signal, &act, NULL) < 0)
-       fprintf(stderr, _("Installing signal handler for signal %d (%s) failed: %m\n"),
-               sighandlers[i].signal, strsignal(sighandlers[i].signal));
+       fprintf(stderr, _("Installing signal handler for signal %d (%s) failed: %s\n"),
+               sighandlers[i].signal, strsignal(sighandlers[i].signal), strerror(errno));
     }
 }