fts: don't operate on an invalid file descriptor after failed dup
authorJim Meyering <meyering@redhat.com>
Mon, 13 Sep 2010 05:29:18 +0000 (07:29 +0200)
committerJim Meyering <meyering@redhat.com>
Mon, 13 Sep 2010 05:29:18 +0000 (07:29 +0200)
* lib/fts.c (fts_build): Don't call set_cloexec_flag on a
negative file descriptor.

ChangeLog
lib/fts.c

index 8e6c3c2..9910da2 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2010-09-13  Jim Meyering  <meyering@redhat.com>
+
+       fts: don't operate on an invalid file descriptor after failed dup
+       * lib/fts.c (fts_build): Don't call set_cloexec_flag on a
+       negative file descriptor.
+
 2010-09-12  Paul Eggert  <eggert@cs.ucla.edu>
 
        savedir: add streamsavedir, deprecate fdsavedir
index a308a8c..4b89ee7 100644 (file)
--- a/lib/fts.c
+++ b/lib/fts.c
@@ -1305,7 +1305,8 @@ fts_build (register FTS *sp, int type)
                 if (ISSET(FTS_CWDFD) && 0 <= dir_fd)
                   {
                     dir_fd = dup (dir_fd);
-                    set_cloexec_flag (dir_fd, true);
+                    if (0 <= dir_fd)
+                      set_cloexec_flag (dir_fd, true);
                   }
                 if (dir_fd < 0 || fts_safe_changedir(sp, cur, dir_fd, NULL)) {
                         if (nlinks && type == BREAD)