diff options
author | Ulrich Drepper <drepper@redhat.com> | 2000-04-07 06:50:46 +0000 |
---|---|---|
committer | Ulrich Drepper <drepper@redhat.com> | 2000-04-07 06:50:46 +0000 |
commit | f11b9da65497cfdb8e49cbc074a1d17a050358bc (patch) | |
tree | 63fe2be39015788f71d2bf8b51156ca9a39b4c97 /sysdeps/unix | |
parent | a0dc52061fd8d47c37bbd363533ae0a4307e507c (diff) | |
download | glibc-f11b9da65497cfdb8e49cbc074a1d17a050358bc.tar glibc-f11b9da65497cfdb8e49cbc074a1d17a050358bc.tar.gz glibc-f11b9da65497cfdb8e49cbc074a1d17a050358bc.tar.bz2 glibc-f11b9da65497cfdb8e49cbc074a1d17a050358bc.zip |
Update.
* sysdeps/unix/sysv/linux/getdents.c (__getdents): The Linux
kernel does not allow relative seeks on descriptors 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.
Diffstat (limited to 'sysdeps/unix')
-rw-r--r-- | sysdeps/unix/sysv/linux/getdents.c | 22 |
1 files changed, 13 insertions, 9 deletions
diff --git a/sysdeps/unix/sysv/linux/getdents.c b/sysdeps/unix/sysv/linux/getdents.c index 9a2255c0da..d142843cbe 100644 --- a/sysdeps/unix/sysv/linux/getdents.c +++ b/sysdeps/unix/sysv/linux/getdents.c @@ -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; } |