Merge pull request #4913 from MegaManSec/k

AkihiroSuda · web-flow · commit bc432ce88cf8 · 2025-10-06T19:38:51.000+09:00
fix(seccompagent): close received FDs, not loop index
diff --git a/tests/cmd/seccompagent/seccompagent.go b/tests/cmd/seccompagent/seccompagent.go
@@ -32,8 +32,8 @@ var (
 )
 
 func closeStateFds(recvFds []int) {
-	for i := range recvFds {
-		unix.Close(i)
+	for _, fd := range recvFds {
+		_ = unix.Close(fd)
 	}
 }
 

Original file line number	Diff line number	Diff line change
`@@ -32,8 +32,8 @@ var (`
`32`	`32`	`)`
`33`	`33`
`34`	`34`	`func closeStateFds(recvFds []int) {`
`35`		`- for i := range recvFds {`
`36`		`- unix.Close(i)`
	`35`	`+ for _, fd := range recvFds {`
	`36`	`+ _ = unix.Close(fd)`
`37`	`37`	`}`
`38`	`38`	`}`
`39`	`39`