close_fds: fix close of external fds

commit 0f092d5 introduced the regression. Closes: containers#240 Signed-off-by: Giuseppe Scrivano <[email protected]>
giuseppe · Feb 24, 2021 · d862266 · d862266
1 parent 7310bf1
commit d862266
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/src/close_fds.c b/src/close_fds.c
@@ -32,7 +32,6 @@ static void __attribute__((constructor)) init()
 	ssize_t size = 0;
 	DIR *d;
 
-	/* Store how many FDs were open before the Go runtime kicked in.  */
 	d = opendir("/proc/self/fd");
 	if (!d)
 		return;
@@ -74,10 +73,8 @@ void close_other_fds()
 {
 	int fd;
 
-	for (fd = 3; fd < open_files_max_fd; fd++) {
+	for (fd = 3; fd <= open_files_max_fd; fd++) {
 		if (open_files_set == NULL || FD_ISSET(fd % FD_SETSIZE, &(open_files_set[fd / FD_SETSIZE])))
-			if (fd == sync_pipe_fd || fd == attach_pipe_fd || fd == dev_null_r || fd == dev_null_w || fd == oom_cgroup_fd
-			    || fd == oom_event_fd)
-				close(fd);
+			close(fd);
 	}
 }