closefrom() is overkill, the only extra fds are the ones we opened so

just close those in the child.

diff --git a/sudo_edit.c b/sudo_edit.c
index a5fefc4d265cc111e87fcdcbc076e97b33cb95c1..95a1e8bd0c8c75e31488b9c7ecfb2457117cad6a 100644 (file)
--- a/sudo_edit.c
+++ b/sudo_edit.c
@@ -213,7 +213,8 @@ int sudo_edit(argc, argv)
        (void) sigaction(SIGINT, &saved_sa_int, NULL);
        (void) sigaction(SIGQUIT, &saved_sa_quit, NULL);
        (void) sigaction(SIGCHLD, &saved_sa_chld, NULL);
-       closefrom(STDERR_FILENO + 1);
+       for (i = 0; i < argc - 1; i++)
+           close(tf[i].tfd);
        set_perms(PERM_FULL_USER);
        execvp(nargv[0], nargv);
        warn("unable to execute %s", nargv[0]);