]> git.ipfire.org Git - thirdparty/glibc.git/commitdiff
(__getdents): The Linux kernel does not allow relative seeks on descriptors
authorUlrich Drepper <drepper@redhat.com>
Fri, 7 Apr 2000 06:51:32 +0000 (06:51 +0000)
committerUlrich Drepper <drepper@redhat.com>
Fri, 7 Apr 2000 06:51:32 +0000 (06:51 +0000)
associated with directories.  Go back to get the position in the directory
every time we enter the function.  Return -1 and set errno if one entry could
be read from the kernel but does not fit into the buffer passed in by the user.

sysdeps/unix/sysv/linux/getdents.c

index 9a2255c0da1c061729dfe4c09044619b4ff61303..d142843cbe1786e8ce33bdba4de82c6332aa3436 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (C) 1993, 95, 96, 97, 98, 99 Free Software Foundation, Inc.
+/* Copyright (C) 1993, 95, 96, 97, 98, 99, 2000 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
@@ -67,7 +67,7 @@ ssize_t
 internal_function
 __getdents (int fd, char *buf, size_t nbytes)
 {
-  off_t last_offset = 0;
+  off_t last_offset = __lseek (fd, 0, SEEK_CUR);
   size_t red_nbytes;
   struct kernel_dirent *skdp, *kdp;
   struct dirent *dp;
@@ -96,13 +96,17 @@ __getdents (int fd, char *buf, size_t nbytes)
       if ((char *) dp + new_reclen > buf + nbytes)
        {
          /* Our heuristic failed.  We read too many entries.  Reset
-            the stream.  `last_offset' contains the last known
-            position.  If it is zero this is the first record we are
-            reading.  In this case do a relative search.  */
-         if (last_offset == 0)
-           __lseek (fd, -retval, SEEK_CUR);
-         else
-           __lseek (fd, last_offset, SEEK_SET);
+            the stream.  */
+         __lseek (fd, last_offset, SEEK_SET);
+
+         if ((char *) dp == buf)
+           {
+             /* The buffer the user passed in is too small to hold even
+                one entry.  */
+             __set_errno (EINVAL);
+             return -1;
+           }
+
          break;
        }