Merge pull request #42 from snorwin/add-read-lock

read lock cmd array

Author: snorwin

main.go

@@ -18,7 +18,7 @@ import (
 var (
 	executable string
 	cmds       []*exec.Cmd
-	l          sync.Mutex
+	l          sync.RWMutex
 	terminated bool
 )
 
@@ -100,13 +100,15 @@ func main() {
 			terminated = true
 
 			// propagate signal to child processes
+			l.RLock()
 			for i := range cmds {
 				if cmds[i].Process != nil {
 					if err := cmds[i].Process.Signal(sig); err != nil {
 						log.Warning(fmt.Sprintf("propagating signal %d to process %d failed", sig, cmds[i].Process.Pid))
 					}
 				}
 			}
+			l.RUnlock()
 		}
 	}
 }
@@ -131,9 +133,11 @@ func runInstance() {
 
 	// launch the actual haproxy including the previous pids to terminate
 	args := os.Args[1:]
+	l.RLock()
 	if len(cmds) > 0 {
 		args = append(args, []string{"-x", utils.LookupHAProxySocketPath(), "-sf", pids()}...)
 	}
+	l.RUnlock()
 
 	cmd := exec.Command(executable, args...)
 	cmd.Stdout = os.Stdout