From ef9d7352272f08ce12d0a4bd22ff45946e1b8420 Mon Sep 17 00:00:00 2001
From: Rene Zbinden <rene.zbinden@postfinance.ch>
Date: Tue, 29 Mar 2016 22:22:38 +0200
Subject: [PATCH] fix race condition

---
 plugins/inputs/sysstat/sysstat.go | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/plugins/inputs/sysstat/sysstat.go b/plugins/inputs/sysstat/sysstat.go
index 2b605b3ad..e87ec1f41 100644
--- a/plugins/inputs/sysstat/sysstat.go
+++ b/plugins/inputs/sysstat/sysstat.go
@@ -175,6 +175,13 @@ func (s *Sysstat) collect() error {
 	if len(s.Activities) == 0 {
 		s.Activities = dfltActivities
 	}
+	if len(s.Sadf) == 0 {
+		sadf, err := exec.LookPath("sadf")
+		if err != nil {
+			return errors.New("sadf not in $PATH, configure path to sadf")
+		}
+		s.Sadf = sadf
+	}
 	options := []string{}
 	for _, act := range s.Activities {
 		options = append(options, "-S", act)
@@ -194,13 +201,6 @@ func (s *Sysstat) collect() error {
 //    Sadf -p -- -p <option> tmpFile
 // and parses the output to add it to the telegraf.Accumulator acc.
 func (s *Sysstat) parse(acc telegraf.Accumulator, option string, ts time.Time) error {
-	if len(s.Sadf) == 0 {
-		sadf, err := exec.LookPath("sadf")
-		if err != nil {
-			return errors.New("sadf not in $PATH, configure path to sadf")
-		}
-		s.Sadf = sadf
-	}
 	cmd := execCommand(s.Sadf, s.sadfOptions(option)...)
 	stdout, err := cmd.StdoutPipe()
 	if err != nil {