Mon Feb 26 10:22:30 1996 Roland McGrath <roland@charlie-brown.gnu.ai.mit.edu>
authorroland <roland>
Tue, 27 Feb 1996 00:53:30 +0000 (00:53 +0000)
committerroland <roland>
Tue, 27 Feb 1996 00:53:30 +0000 (00:53 +0000)
* sysdeps/unix/dirstream.h: Rewritten.
* sysdeps/unix/readdir.c: Rewritten.
* sysdeps/unix/telldir.c: Rewritten.
* sysdeps/unix/closedir.c: Use <...> instead of "..." for dirstream.h.
Remove __ from DIR struct member names.
* sysdeps/unix/dirfd.c: Likewise.
* sysdeps/unix/seekdir.c: Likewise.
* sysdeps/unix/rewinddir.c: Likewise.
* sysdeps/unix/opendir.c: Likewise.  Don't allocate extra space after
DIR structure.
* sysdeps/stub/direct.h: File removed.
* sysdeps/unix/bsd/bsd4.4/direct.h: File removed.
* sysdeps/unix/bsd/direct.h: File removed.
* sysdeps/unix/common/direct.h: File removed.
* sysdeps/unix/sysv/irix4/direct.h: File removed.
* sysdeps/unix/sysv/isc3/direct.h: File removed.
* sysdeps/unix/sysv/sco3.2.4/direct.h: File removed.
* sysdeps/unix/sysv/sysv4/solaris2/direct.h: File removed.
* sysdeps/unix/common/direntry.h: New file.
* sysdeps/unix/bsd/direntry.h: New file.
* sysdeps/unix/bsd/bsd4.4/direntry.h: New file.
* sysdeps/unix/sysv/direntry.h: New file.
* sysdeps/stub/direntry.h: New file.
* dirent/dirent.h (struct dirent): Type removed.  Include <direntry.h>
to define it.
(_D_EXACT_NAMLEN, _D_ALLOC_NAMLEN): New macros.
* dirent/Makefile (headers): Add direntry.h.
(distribute): Remove direct.h.
* sysdeps/posix/getcwd.c: Use new macros instead of d_namlen.
* dirent/scandir.c: Likewise.
* io/fts.c (fts_build): Likewise.
* io/ftw.c (ftw_dir): Likewise.
* sysdeps/posix/ttyname.c: Likewise.

13 files changed:
dirent/Makefile
dirent/dirent.h
dirent/scandir.c
sysdeps/unix/bsd/direntry.h [new file with mode: 0644]
sysdeps/unix/common/direntry.h [new file with mode: 0644]
sysdeps/unix/dirfd.c
sysdeps/unix/dirstream.h
sysdeps/unix/opendir.c
sysdeps/unix/readdir.c
sysdeps/unix/rewinddir.c
sysdeps/unix/seekdir.c
sysdeps/unix/sysv/direntry.h [new file with mode: 0644]
sysdeps/unix/telldir.c

index 532020d..b6a4219 100644 (file)
@@ -1,4 +1,4 @@
-# Copyright (C) 1991, 1992, 1993, 1994, 1995 Free Software Foundation, Inc.
+# Copyright (C) 1991, 92, 93, 94, 95, 96 Free Software Foundation, Inc.
 # This file is part of the GNU C Library.
 
 # The GNU C Library is free software; you can redistribute it and/or
 #
 subdir         := dirent
 
-headers                := dirent.h
+headers                := dirent.h direntry.h
 routines       := opendir closedir readdir rewinddir \
                   seekdir telldir scandir alphasort \
                   getdents dirfd
-distribute := direct.h dirstream.h
+distribute := dirstream.h
 
 tests     := list tst-seekdir
 
index 5b99437..d93a146 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (C) 1991, 1992, 1993, 1994, 1995 Free Software Foundation, Inc.
+/* Copyright (C) 1991, 92, 93, 94, 95, 96 Free Software Foundation, Inc.
 This file is part of the GNU C Library.
 
 The GNU C Library is free software; you can redistribute it and/or
@@ -29,30 +29,48 @@ __BEGIN_DECLS
 
 #include <gnu/types.h>
 
+/* This file defines `struct dirent'.
 
-/* Directory entry structure.
+   It defines the macro `_DIRENT_HAVE_D_NAMLEN' iff there is a `d_namlen'
+   member that gives the length of `d_name'.
 
-   This structure is laid out identically to the `struct direct' that
-   represents directory entries in the GNU Hurd and in BSD 4.4 (and
-   incidentally, on disk in the Berkeley fast file system).  The `readdir'
-   implementations for GNU and BSD know this; you must change them if you
-   change this structure.  */
+   It defines the macro `_DIRENT_HAVE_D_RECLEN' iff there is a `d_reclen'
+   member that gives the size of the entire directory entry.  */
 
-struct dirent
-  {
-    __ino_t d_fileno;          /* File serial number.  */
-    unsigned short int d_reclen; /* Length of the whole `struct dirent'.  */
-    unsigned char d_type;      /* File type, possibly unknown.  */
-    unsigned char d_namlen;    /* Length of the file name.  */
-
-    /* Only this member is in the POSIX standard.  */
-    char d_name[1];            /* File name (actually longer).  */
-  };
+#include <direntry.h>
 
 #if defined(__USE_BSD) || defined(__USE_MISC)
 #define        d_ino           d_fileno /* Backward compatibility.  */
 #endif
 
+/* These macros extract size information from a `struct dirent *'.
+   They may evaluate their argument multiple times, so it must not
+   have side effects.  Each of these may involve a relatively costly
+   call to `strlen' on some systems, so these values should be cached.
+
+   _D_EXACT_NAMLEN (DP)        returns the length of DP->d_name, not including
+   its terminating null character.
+
+   _D_ALLOC_NAMLEN (DP)        returns a size at least (_D_EXACT_NAMLEN (DP) + 1);
+   that is, the allocation size needed to hold the DP->d_name string.
+   Use this macro when you don't need the exact length, just an upper bound.
+   This macro is less likely to require calling `strlen' than _D_EXACT_NAMLEN.
+   */
+
+#ifdef _DIRENT_HAVE_D_NAMLEN
+#define _D_EXACT_NAMLEN(d) ((d)->d_namlen)
+#define _D_ALLOC_NAMLEN(d) (_D_EXACT_NAMLEN (d) + 1)
+#else
+#define _D_EXACT_NAMLEN(d) (strlen ((d)->d_name))
+#ifdef _DIRENT_HAVE_D_RECLEN
+#define _D_ALLOC_NAMLEN(d) (((char *) (d) + (d)->d_reclen) - &(d)->d_name[0])
+#else
+#define _D_ALLOC_NAMLEN(d) (sizeof (d)->d_name > 1 ? sizeof (d)->d_name : \
+                           _D_EXACT_NAMLEN (d) + 1)
+#endif
+#endif
+
+
 #ifdef __USE_BSD
 /* File types for `d_type'.  */
 enum
index e2ef63a..6907f08 100644 (file)
@@ -64,7 +64,7 @@ DEFUN(scandir, (dir, namelist, select, cmp),
            v = new;
          }
 
-       dsize = &d->d_name[d->d_namlen + 1] - (char *) d;
+       dsize = &d->d_name[_D_ALLOC_NAMLEN (d)] - (char *) d;
        v[i] = (struct dirent *) malloc (dsize);
        if (v[i] == NULL)
          goto lose;
diff --git a/sysdeps/unix/bsd/direntry.h b/sysdeps/unix/bsd/direntry.h
new file mode 100644 (file)
index 0000000..eb55d57
--- /dev/null
@@ -0,0 +1,29 @@
+/* Directory entry structure `struct dirent'.  4.2BSD version.
+Copyright (C) 1996 Free Software Foundation, Inc.
+This file is part of the GNU C Library.
+
+The GNU C Library is free software; you can redistribute it and/or
+modify it under the terms of the GNU Library General Public License as
+published by the Free Software Foundation; either version 2 of the
+License, or (at your option) any later version.
+
+The GNU C Library is distributed in the hope that it will be useful,
+but WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+Library General Public License for more details.
+
+You should have received a copy of the GNU Library General Public
+License along with the GNU C Library; see the file COPYING.LIB.  If
+not, write to the Free Software Foundation, Inc., 675 Mass Ave,
+Cambridge, MA 02139, USA.  */
+
+struct dirent
+  {
+    unsigned int d_fileno;     /* 32 bits.  */
+    unsigned short int d_reclen; /* 16 bits.  */
+    unsigned short int d_namlen; /* 16 bits.  */
+    char d_name[0];            /* Variable length.  */
+  };
+
+#define _DIRENT_HAVE_D_RECLEN 1
+#define _DIRENT_HAVE_D_NAMLEN 1
diff --git a/sysdeps/unix/common/direntry.h b/sysdeps/unix/common/direntry.h
new file mode 100644 (file)
index 0000000..5decd56
--- /dev/null
@@ -0,0 +1,29 @@
+/* Directory entry structure `struct dirent'.  SVR4/Linux version.
+Copyright (C) 1996 Free Software Foundation, Inc.
+This file is part of the GNU C Library.
+
+The GNU C Library is free software; you can redistribute it and/or
+modify it under the terms of the GNU Library General Public License as
+published by the Free Software Foundation; either version 2 of the
+License, or (at your option) any later version.
+
+The GNU C Library is distributed in the hope that it will be useful,
+but WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+Library General Public License for more details.
+
+You should have received a copy of the GNU Library General Public
+License along with the GNU C Library; see the file COPYING.LIB.  If
+not, write to the Free Software Foundation, Inc., 675 Mass Ave,
+Cambridge, MA 02139, USA.  */
+
+struct dirent
+  {
+    unsigned int d_fileno;
+    int d_off;                 /* Position in directory of following entry. */
+    unsigned short int d_reclen;
+    char d_name[0];            /* Variable length.  */
+  };
+
+#define _DIRENT_HAVE_D_RECLEN 1
+#define _DIRENT_HAVE_D_OFF 1
index 7733634..d02c12a 100644 (file)
@@ -26,5 +26,5 @@ int
 dirfd (dirp)
      DIR *dirp;
 {
-  return dirp->__fd;
+  return dirp->fd;
 }
index 72fd67c..87e78b6 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (C) 1993, 1995 Free Software Foundation, Inc.
+/* Copyright (C) 1993, 1995, 1996 Free Software Foundation, Inc.
 This file is part of the GNU C Library.
 
 The GNU C Library is free software; you can redistribute it and/or
@@ -20,26 +20,25 @@ Cambridge, MA 02139, USA.  */
 
 #define        _DIRSTREAM_H    1
 
-#define __need_size_t
-#include <stddef.h>
+#include <sys/types.h>
 
 /* Directory stream type.
 
    The miscellaneous Unix `readdir' implementations read directory data
-   into a buffer and fill in a `struct dirent' copy in the `DIR' object. */
+   into a buffer and return `struct dirent *' pointers into it.  */
 
 struct __dirstream
   {
-    int __fd;                  /* File descriptor.  */
+    int fd;                    /* File descriptor.  */
 
-    char *__data;              /* Directory block.  */
-    size_t __allocation;       /* Space allocated for the block.  */
-    size_t __offset;           /* Current offset into the block.  */
-    size_t __size;             /* Total valid data in the block.  */
+    char *data;                        /* Directory block.  */
+    size_t allocation;         /* Space allocated for the block.  */
+    size_t size;               /* Total valid data in the block.  */
+    size_t offset;             /* Current offset into the block.  */
 
-    struct dirent __entry;     /* Returned by `readdir'.  */
+    off_t filepos;             /* Position of next entry to read.  */
   };
 
-#define _DIR_dirfd(dirp)       ((dirp)->__fd)
+#define _DIR_dirfd(dirp)       ((dirp)->fd)
 
 #endif /* dirstream.h */
index 5cd6206..df20b9c 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (C) 1991, 1992, 1993, 1994, 1995 Free Software Foundation, Inc.
+/* Copyright (C) 1991, 92, 93, 94, 95, 96 Free Software Foundation, Inc.
 This file is part of the GNU C Library.
 
 The GNU C Library is free software; you can redistribute it and/or
@@ -16,7 +16,6 @@ License along with the GNU C Library; see the file COPYING.LIB.  If
 not, write to the Free Software Foundation, Inc., 675 Mass Ave,
 Cambridge, MA 02139, USA.  */
 
-#include <ansidecl.h>
 #include <errno.h>
 #include <limits.h>
 #include <stddef.h>
@@ -28,12 +27,11 @@ Cambridge, MA 02139, USA.  */
 #include <unistd.h>
 #include <stdio.h>
 
-#include "direct.h"            /* This file defines `struct direct'.  */
-#include "dirstream.h"
+#include <dirstream.h>
 
 /* Open a directory stream on NAME.  */
 DIR *
-DEFUN(opendir, (name), CONST char *name)
+opendir (const char *name)
 {
   DIR *dirp;
   struct stat statbuf;
@@ -62,7 +60,7 @@ DEFUN(opendir, (name), CONST char *name)
       goto lose;
     }
 
-  dirp = (DIR *) calloc (1, sizeof (DIR) + NAME_MAX); /* Zero-fill.  */
+  dirp = (DIR *) calloc (1, sizeof (DIR)); /* Zero-fill.  */
   if (dirp == NULL)
   lose:
     {
@@ -73,24 +71,24 @@ DEFUN(opendir, (name), CONST char *name)
     }
 
 #ifdef _STATBUF_ST_BLKSIZE
-  if (statbuf.st_blksize < sizeof (struct direct))
-    dirp->__allocation = sizeof (struct direct);
+  if (statbuf.st_blksize < sizeof (struct dirent))
+    dirp->allocation = sizeof (struct dirent);
   else
-    dirp->__allocation = statbuf.st_blksize;
+    dirp->allocation = statbuf.st_blksize;
 #else
-  dirp->__allocation = (BUFSIZ < sizeof (struct direct) ?
-                       sizeof (struct direct) : BUFSIZ);
+  dirp->allocation = (BUFSIZ < sizeof (struct dirent) ?
+                     sizeof (struct dirent) : BUFSIZ);
 #endif
-  dirp->__data = (char *) malloc (dirp->__allocation);
-  if (dirp->__data == NULL)
+  dirp->data = (char *) malloc (dirp->allocation);
+  if (dirp->data == NULL)
     {
       int save = errno;
-      free ((PTR) dirp);
+      free (dirp);
       (void) __close (fd);
       errno = save;
       return NULL;
     }
 
-  dirp->__fd = fd;
+  dirp->fd = fd;
   return dirp;
 }
index 1842948..84ac033 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (C) 1991, 1992, 1993, 1994, 1995 Free Software Foundation, Inc.
+/* Copyright (C) 1991, 92, 93, 94, 95, 96 Free Software Foundation, Inc.
 This file is part of the GNU C Library.
 
 The GNU C Library is free software; you can redistribute it and/or
@@ -16,7 +16,6 @@ License along with the GNU C Library; see the file COPYING.LIB.  If
 not, write to the Free Software Foundation, Inc., 675 Mass Ave,
 Cambridge, MA 02139, USA.  */
 
-#include <ansidecl.h>
 #include <errno.h>
 #include <limits.h>
 #include <stddef.h>
@@ -24,60 +23,77 @@ Cambridge, MA 02139, USA.  */
 #include <dirent.h>
 #include <unistd.h>
 #include <sys/types.h>
+#include <assert.h>
 
-#include "direct.h"            /* This file defines `struct direct'.  */
-#include "dirstream.h"
-
-/* direct.h may have an alternate definition for this.  */
-#ifndef D_RECLEN
-#define D_RECLEN(dp)   ((dp)->d_reclen)
-#endif
+#include <dirstream.h>
 
 
 /* Read a directory entry from DIRP.  */
 struct dirent *
-DEFUN(readdir, (dirp), DIR *dirp)
+readdir (DIR *dirp)
 {
-  if (dirp == NULL || dirp->__data == NULL)
+  struct dirent *dp;
+
+  if (dirp == NULL || dirp->data == NULL)
     {
       errno = EINVAL;
       return NULL;
     }
 
-  while (1)
+  do
     {
-      struct direct *dp;
+      size_t reclen;
 
-      if (dirp->__offset >= dirp->__size)
+      if (dirp->offset >= dirp->size)
        {
          /* We've emptied out our buffer.  Refill it.  */
 
+         size_t maxread;
          off_t base;
-         ssize_t bytes = __getdirentries (dirp->__fd, dirp->__data,
-                                          dirp->__allocation, &base);
+         ssize_t bytes;
+
+         if (sizeof (dp->d_name) > 1)
+           /* Fixed-size struct; must read one at a time (see below).  */
+           maxread = sizeof *dp;
+         else
+           maxread = dirp->allocation;
+
+         base = dirp->filepos;
+         bytes = __getdirentries (dirp->fd, dirp->data, maxread, &base);
          if (bytes <= 0)
            return NULL;
-         dirp->__size = (size_t) bytes;
+         dirp->size = (size_t) bytes;
 
          /* Reset the offset into the buffer.  */
-         dirp->__offset = 0;
+         dirp->offset = 0;
        }
 
-      dp = (struct direct *) &dirp->__data[dirp->__offset];
-      dirp->__offset += D_RECLEN (dp);
+      dp = (struct dirent *) &dirp->data[dirp->offset];
 
-      if (dp->d_ino != 0)
-       {
-         /* Not a deleted file.  */
-         register struct dirent *d = &dirp->__entry;
-         register const char *p;
-         d->d_fileno = (ino_t) dp->d_ino;
-         /* On some systems the name length does not actually mean much.
-            But we always use it as a maximum.  */
-         p = memchr ((PTR) dp->d_name, '\0', D_NAMLEN (dp) + 1);
-         d->d_namlen = (p != NULL) ? p - dp->d_name : D_NAMLEN (dp);
-         memcpy (d->d_name, dp->d_name, d->d_namlen + 1);
-         return d;
-       }
-    }
+#ifdef _DIRENT_HAVE_D_RECLEN
+      reclen = dp->d_reclen;
+#else
+      /* The only version of `struct dirent' that lacks `d_reclen'
+        is fixed-size.  */
+      assert (sizeof dp->d_name > 1);
+      reclen = sizeof *dp;
+      /* The name is not terminated if it is the largest possible size.
+        Clobber the following byte to ensure proper null termination.  We
+        read jst one entry at a time above so we know that byte will not
+        be used later.  */
+      dp->d_name[sizeof dp->d_name] = '\0';
+#endif
+
+      dirp->offset += reclen;
+
+#ifdef _DIRENT_HAVE_D_OFF
+      dirp->filepos = dp->d_off;
+#else
+      dirp->filepos += reclen;
+#endif
+
+      /* Skip deleted files.  */
+    } while (dp->d_ino == 0);
+
+  return dp;
 }
index 7ad7bc1..791ecc1 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (C) 1991, 1995 Free Software Foundation, Inc.
+/* Copyright (C) 1991, 1995, 1996 Free Software Foundation, Inc.
 This file is part of the GNU C Library.
 
 The GNU C Library is free software; you can redistribute it and/or
@@ -21,13 +21,13 @@ Cambridge, MA 02139, USA.  */
 #include <dirent.h>
 #include <sys/types.h>
 #include <unistd.h>
-#include "dirstream.h"
+#include <dirstream.h>
 
 /* Rewind DIRP to the beginning of the directory.  */
 void
 DEFUN(rewinddir, (dirp), DIR *dirp)
 {
-  (void) lseek(dirp->__fd, (off_t) 0, SEEK_SET);
-  dirp->__offset = 0;
-  dirp->__size = 0;
+  (void) lseek(dirp->fd, (off_t) 0, SEEK_SET);
+  dirp->offset = 0;
+  dirp->size = 0;
 }
index 018f697..9ce332b 100644 (file)
@@ -21,13 +21,13 @@ Cambridge, MA 02139, USA.  */
 #include <stddef.h>
 #include <dirent.h>
 #include <unistd.h>
-#include "dirstream.h"
+#include <dirstream.h>
 
 /* Seek to position POS in DIRP.  */
 void
 DEFUN(seekdir, (dirp, pos), DIR *dirp AND __off_t pos)
 {
-  (void) __lseek(dirp->__fd, pos, SEEK_SET);
-  dirp->__size = 0;
-  dirp->__offset = 0;
+  (void) __lseek(dirp->fd, pos, SEEK_SET);
+  dirp->size = 0;
+  dirp->offset = 0;
 }
diff --git a/sysdeps/unix/sysv/direntry.h b/sysdeps/unix/sysv/direntry.h
new file mode 100644 (file)
index 0000000..5035f14
--- /dev/null
@@ -0,0 +1,24 @@
+/* Directory entry structure `struct dirent'.  Old System V version.
+Copyright (C) 1996 Free Software Foundation, Inc.
+This file is part of the GNU C Library.
+
+The GNU C Library is free software; you can redistribute it and/or
+modify it under the terms of the GNU Library General Public License as
+published by the Free Software Foundation; either version 2 of the
+License, or (at your option) any later version.
+
+The GNU C Library is distributed in the hope that it will be useful,
+but WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+Library General Public License for more details.
+
+You should have received a copy of the GNU Library General Public
+License along with the GNU C Library; see the file COPYING.LIB.  If
+not, write to the Free Software Foundation, Inc., 675 Mass Ave,
+Cambridge, MA 02139, USA.  */
+
+struct dirent
+  {
+    unsigned short int d_fileno;
+    char d_name[14];
+  };
index 2b611b0..d9b8d11 100644 (file)
@@ -16,28 +16,13 @@ License along with the GNU C Library; see the file COPYING.LIB.  If
 not, write to the Free Software Foundation, Inc., 675 Mass Ave,
 Cambridge, MA 02139, USA.  */
 
-#include <ansidecl.h>
-#include <errno.h>
-#include <stddef.h>
 #include <dirent.h>
-#include <unistd.h>
-#include <sys/types.h>
-#include "dirstream.h"
+
+#include <dirstream.h>
 
 /* Return the current position of DIRP.  */
 off_t
-DEFUN(telldir, (dirp), DIR *dirp)
+telldir (DIR *dirp)
 {
-  register off_t pos;
-
-  if (dirp == NULL)
-    {
-      errno = EINVAL;
-      return (off_t) -1;
-    }
-
-  pos = __lseek(dirp->__fd, (off_t) 0, SEEK_CUR);
-  if (pos == (off_t) -1)
-    return (off_t) -1;
-  return pos + dirp->__size - dirp->__offset;
+  return dirp->filepos;
 }