Apply patch from Scott Lamb fixing some memory and resource leaks.

diff --git a/lib/pidfile.c b/lib/pidfile.c
index 61a802f69713c78e78dedfb4bf81680eac884402..08d96dfe6c02303000bcb1545d96cca793a144c2 100644 (file)
--- a/lib/pidfile.c
+++ b/lib/pidfile.c
@@ -84,8 +84,13 @@ pid_t write_pid (char *pidfile)
   int fd;
   pid_t pid;
 
-  if ( ((fd = open(pidfile, O_RDWR|O_CREAT, 0644)) == -1)
-       || ((f = fdopen(fd, "r+")) == NULL) ) {
+  if ((fd = open(pidfile, O_RDWR|O_CREAT, 0644)) == -1) {
+      close(fd);
+      return 0;
+  }
+
+  if ((f = fdopen(fd, "r+")) == NULL) {
+      fclose(f);
       return 0;
   }
   
@@ -98,18 +103,18 @@ pid_t write_pid (char *pidfile)
 
   pid = getpid();
   if (!fprintf(f,"%ld\n", (long)pid)) {
-      close(fd);
+      fclose(f);
       return 0;
   }
   fflush(f);
 
 #ifdef HAVE_FLOCK
   if (flock(fd, LOCK_UN) == -1) {
-      close(fd);
+      fclose(f);
       return 0;
   }
 #endif
-  close(fd);
+  fclose(f);
 
   return pid;
 }

diff --git a/src/process.c b/src/process.c
index 11e8b4fc7b98920fadb981e4fe1c7cc74eced74d..30ff82ad5ec700d86de5af753a65cbc388d4f00e 100644 (file)
--- a/src/process.c
+++ b/src/process.c
@@ -379,8 +379,10 @@ bool execute_script(const char *name, char **envp)
 
        /* First check if there is a script */
 
-       if(stat(scriptname + 1, &s))
+       if(stat(scriptname + 1, &s)) {
+               free(scriptname);
                return true;
+       }
 
        ifdebug(STATUS) logger(LOG_INFO, _("Executing script %s"), name);