Merge pull request #17 from papertrail/13-no-blank-pid-files

No blank pid files
papertrail · Jul 15, 2014 · bacddd7 · bacddd7
2 parents 39552e6 + b649bf7
commit bacddd7
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/config_manager.go b/config_manager.go
@@ -309,7 +309,14 @@ func (cm *ConfigManager) defaultPidFile() string {
 		if err != nil {
 			continue
 		}
+		tmpPidFile := fd.Name()
 		fd.Close()
+		err = os.Remove(tmpPidFile)
+		if err != nil {
+			// Die because the lockfile lib can't handle an existing blank file
+			log.Criticalf("Can't remove temp pid file. Details: %s", err.Error())
+			os.Exit(1)
+		}
 		return f
 	}
 	return "/tmp/remote_syslog.pid"