Fix panic in cgroups monitoring

Gosigar's cgroups GetStatsForProcesses can return a nil Stats pointer and no error when the ["blkio", "cpu", "cpuacct", "memory"] subsystems are on the root cgroup. Related elastic#21113
adriansr · Sep 28, 2020 · 1a2af5c · 1a2af5c
1 parent ddfe085
commit 1a2af5c
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/libbeat/cmd/instance/metrics.go b/libbeat/cmd/instance/metrics.go
@@ -285,6 +285,10 @@ func reportBeatCgroups(_ monitoring.Mode, V monitoring.Visitor) {
 		logp.Err("error getting group status: %v", err)
 		return
 	}
+	// GetStatsForProcess returns a nil selfStats and no error when there's no stats
+	if selfStats == nil {
+		return
+	}
 
 	if cpu := selfStats.CPU; cpu != nil {
 		monitoring.ReportNamespace(V, "cpu", func() {