Let the output from an executed script in execute_script() go to

syslog, with proper error detection.

diff --git a/src/net.c b/src/net.c
index 0e0bddf..09d98f4 100644 (file)
--- a/src/net.c
+++ b/src/net.c
@@ -17,7 +17,7 @@
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 

-    $Id: net.c,v 1.35.4.71 2000/11/15 01:06:10 zarq Exp $
+    $Id: net.c,v 1.35.4.72 2000/11/15 01:28:21 zarq Exp $

 */
 
 #include "config.h"
 */
 
 #include "config.h"


@@ -101,6 +101,7 @@ int execute_script(const char *name)
   char *scriptname;
   pid_t pid;
   char *s;
   char *scriptname;
   pid_t pid;
   char *s;

+  int error;

 
   if((pid = fork()) < 0)
     {
 
   if((pid = fork()) < 0)
     {


@@ -116,6 +117,8 @@ int execute_script(const char *name)
 
   /* Child here */
 
 
   /* Child here */
 

+  error = 0;
+

   if(netname)
     {
       asprintf(&s, "NETNAME=%s", netname);
   if(netname)
     {
       asprintf(&s, "NETNAME=%s", netname);


@@ -128,18 +131,55 @@ int execute_script(const char *name)
     }
 #endif
 
     }
 #endif
 

-  chdir(confbase);     /* This cannot fail since we already read config files from this directory. */
+  if(chdir(confbase) < 0)
+    /* This cannot fail since we already read config files from this
+       directory. - Guus */
+    /* Yes this can fail, somebody could have removed this directory
+       when we didn't pay attention. - Ivo */
+    {
+      if(chdir("/") < 0)
+       /* Now if THIS fails, something wicked is going on. - Ivo */
+       syslog(LOG_ERR, _("Couldn't chdir to `/': %m"));
+
+      /* Continue anyway. */
+    }

   
   asprintf(&scriptname, "%s/%s", confbase, name);
   
   asprintf(&scriptname, "%s/%s", confbase, name);

-  execl(scriptname, NULL);

 
 

+  /* Close all file descriptors */
+  closelog();
+  fcloseall();
+
+  /* Open standard input */
+  if(open("/dev/null", O_RDONLY) < 0)
+    {
+      syslog(LOG_ERR, _("Opening `/dev/null' failed: %m"));
+      error = 1;
+    }
+
+  if(!error)
+    {
+      /* Standard output directly goes to syslog */
+      openlog(name, LOG_CONS | LOG_PID, LOG_DAEMON);
+      /* Standard error as well */
+      if(dup2(1, 2) < 0)
+       {
+         syslog(LOG_ERR, _("System call `%s' failed: %m"),
+                "dup2");
+         error = 1;
+       }
+    }
+  
+  if(error && debug_lvl > 1)
+    syslog(LOG_INFO, _("This means that any output the script generates will not be shown in syslog."));
+  
+  execl(scriptname, NULL);

   /* No return on success */
   
   if(errno != ENOENT)  /* Ignore if the file does not exist */
     syslog(LOG_WARNING, _("Error executing `%s': %m"), scriptname);
 
   /* No need to free things */
   /* No return on success */
   
   if(errno != ENOENT)  /* Ignore if the file does not exist */
     syslog(LOG_WARNING, _("Error executing `%s': %m"), scriptname);
 
   /* No need to free things */

-

   exit(0);
 }
 
   exit(0);
 }