Merge pull request #265 from coroot/issue_260

def · web-flow · commit 28105e53d14a · 2026-01-05T10:40:50.000-03:00
fix panic in python cmdline parsing
diff --git a/containers/process.go b/containers/process.go
@@ -110,7 +110,11 @@ func (p *Process) instrumentPython(cmdline []byte, tracer *ebpftracer.Tracer) {
 	if len(cmd) == 0 {
 		return
 	}
-	cmd = bytes.TrimSuffix(bytes.Fields(cmd)[0], []byte{':'})
+	cmdFields := bytes.Fields(cmd)
+	if len(cmdFields) == 0 {
+		return
+	}
+	cmd = bytes.TrimSuffix(cmdFields[0], []byte{':'})
 	if !pythonCmd.Match(cmd) {
 		return
 	}

Original file line number	Diff line number	Diff line change
`@@ -110,7 +110,11 @@ func (p Process) instrumentPython(cmdline []byte, tracer ebpftracer.Tracer) {`
`110`	`110`	`if len(cmd) == 0 {`
`111`	`111`	`return`
`112`	`112`	`}`
`113`		`- cmd = bytes.TrimSuffix(bytes.Fields(cmd)[0], []byte{':'})`
	`113`	`+ cmdFields := bytes.Fields(cmd)`
	`114`	`+ if len(cmdFields) == 0 {`
	`115`	`+ return`
	`116`	`+ }`
	`117`	`+ cmd = bytes.TrimSuffix(cmdFields[0], []byte{':'})`
`114`	`118`	`if !pythonCmd.Match(cmd) {`
`115`	`119`	`return`
`116`	`120`	`}`