[Nmh-commits] nmh ChangeLog uip/folder.c

[Peter Maydell]

CVSROOT:        /sources/nmh
Module name:    nmh
Changes by:     Peter Maydell <pm215>   08/08/05 19:09:03

Modified files:
        .              : ChangeLog 
        uip            : folder.c 

Log message:
        Dump hacky overoptimisation in addir -- it doesn't actually get the case
        of symlinks to directories right. Patch from Eric Gillespie.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/nmh/ChangeLog?cvsroot=nmh&r1=1.284&r2=1.285
http://cvs.savannah.gnu.org/viewcvs/nmh/uip/folder.c?cvsroot=nmh&r1=1.14&r2=1.15

Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/nmh/nmh/ChangeLog,v
retrieving revision 1.284
retrieving revision 1.285
diff -u -b -r1.284 -r1.285
--- ChangeLog   5 Aug 2008 19:05:23 -0000       1.284
+++ ChangeLog   5 Aug 2008 19:09:03 -0000       1.285
@@ -1,6 +1,12 @@
 2008-08-04  Eric Gillespie  <address@hidden>
 
-       * uip/folder.c: Simplify dodir/addir/addfold.
+       * uip/folder.c: Simplify dodir/addir/addfold.  Dump hacky
+       over-optimization in addir that tried to avoid readdir after all
+       child directories had been read; this was also trying to support
+       symlinks to directories, but would have been failing (because
+       nlink may have gone to 0 with symlinks to directories remaining)
+       had the lstat usage been correct (lstat doesn't fail for normal
+       directories; should have used S_ISLNK).
 
 2008-08-03  Peter Maydell  <address@hidden>
 

Index: uip/folder.c
===================================================================
RCS file: /sources/nmh/nmh/uip/folder.c,v
retrieving revision 1.14
retrieving revision 1.15
diff -u -b -r1.14 -r1.15
--- uip/folder.c        5 Aug 2008 19:05:23 -0000       1.14
+++ uip/folder.c        5 Aug 2008 19:09:03 -0000       1.15
@@ -4,7 +4,7 @@
  *             -- push/pop a folder onto/from the folder stack
  *             -- list the folder stack
  *
- * $Id: folder.c,v 1.14 2008/08/05 19:05:23 pm215 Exp $
+ * $Id: folder.c,v 1.15 2008/08/05 19:09:03 pm215 Exp $
  *
  * This code is Copyright (c) 2002, by the authors of nmh.  See the
  * COPYRIGHT file in the root directory of the nmh distribution for
@@ -665,16 +665,11 @@
 static void
 addir (char *name)
 {
-    int nlink;
     char *prefix, *child;
     struct stat st;
     struct dirent *dp;
     DIR * dd;
 
-   /* short-cut to see if directory has any sub-directories */
-    if (stat (name, &st) != -1 && st.st_nlink == 2)
-        return;
- 
     if (!(dd = opendir (name))) {
        admonish (name, "unable to read directory ");
        return;
@@ -686,25 +681,12 @@
        prefix = concat (name, "/", (void *)NULL);
     }
 
-    /*
-     * Keep track of the number of directories we've seen
-     * so we can quit stat'ing early, if we've seen them all.
-     */
-    nlink = st.st_nlink;
-
-    while (nlink && (dp = readdir (dd))) {
+    while ((dp = readdir (dd))) {
        if (!strcmp (dp->d_name, ".") || !strcmp (dp->d_name, "..")) {
-           nlink--;
            continue;
        }
        child = concat (prefix, dp->d_name, (void *)NULL);
        if (stat (child, &st) != -1 && S_ISDIR(st.st_mode)) {
-           /*
-            * Check if this was really a symbolic link pointing at
-            * a directory.  If not, then decrement link count.
-            */
-           if (lstat (child, &st) == -1)
-               nlink--;
            /* addfold saves child in the list, don't free it */
            addfold (child);
        } else {