Skip to content

Commit 10062d7

Browse files
committed
libct: Don't exclude BPF fds from leaks
Now the cgroups dependency has been upgraded and it doesn't cause these leaks anymore. Let's remove the exclude. Signed-off-by: Rodrigo Campos <rodrigo@amutable.com>
1 parent 892f500 commit 10062d7

1 file changed

Lines changed: 0 additions & 10 deletions

File tree

libcontainer/integration/exec_test.go

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1748,10 +1748,6 @@ func testFdLeaks(t *testing.T, systemd bool) {
17481748
}
17491749
// Show the extra opened files.
17501750

1751-
excludedPaths := []string{
1752-
"anon_inode:bpf-prog", // FIXME: see https://github.com/opencontainers/runc/issues/2366#issuecomment-776411392
1753-
}
1754-
17551751
count := 0
17561752

17571753
procSelfFd, closer, err := pathrs.ProcThreadSelfOpen("fd/", unix.O_DIRECTORY|unix.O_CLOEXEC)
@@ -1767,12 +1763,6 @@ next_fd:
17671763
}
17681764
}
17691765
dst, _ := linux.Readlinkat(procSelfFd, fd1)
1770-
for _, ex := range excludedPaths {
1771-
if ex == dst {
1772-
continue next_fd
1773-
}
1774-
}
1775-
17761766
count++
17771767
t.Logf("extra fd %s -> %s", fd1, dst)
17781768
}

0 commit comments

Comments
 (0)