Merge pull request #49 from Andreagit97/fix-fill-from-stat

dd-mergequeue[bot] · Andreagit97 · brycekahle · web-flow · commit 0057e4d0813a · 2025-11-12T16:25:44.000Z
fix: do not read under `/proc/pid/stat` if empty

Co-authored-by: Andreagit97 &lt;andreaterzolo3@gmail.com&gt;
Co-authored-by: bryce.kahle &lt;bryce.kahle@datadoghq.com&gt;
diff --git a/process/process_linux.go b/process/process_linux.go
@@ -867,6 +867,9 @@ func (p *Process) fillFromStat() (int32, int32, *cpu.TimesStat, int64, int32, er
 		return 0, 0, nil, 0, 0, err
 	}
 	fields := strings.Fields(string(contents))
+	if len(fields) == 0 {
+		return 0, 0, nil, 0, 0, fmt.Errorf("stat file is empty for pid %d", pid)
+	}
 	timestamp := time.Now().Unix()
 
 	i := 1

Original file line number	Diff line number	Diff line change
`@@ -867,6 +867,9 @@ func (p Process) fillFromStat() (int32, int32, cpu.TimesStat, int64, int32, er`
`867`	`867`	`return 0, 0, nil, 0, 0, err`
`868`	`868`	`}`
`869`	`869`	`fields := strings.Fields(string(contents))`
	`870`	`+ if len(fields) == 0 {`
	`871`	`+ return 0, 0, nil, 0, 0, fmt.Errorf("stat file is empty for pid %d", pid)`
	`872`	`+ }`
`870`	`873`	`timestamp := time.Now().Unix()`
`871`	`874`
`872`	`875`	`i := 1`