aboutsummaryrefslogtreecommitdiff
path: root/dirent
diff options
context:
space:
mode:
authorAurelien Jarno <aurelien@aurel32.net>2017-12-29 14:44:57 +0100
committerAurelien Jarno <aurelien@aurel32.net>2017-12-29 14:45:35 +0100
commit72f29e3aab56cf9bd11d60ae351fbf9d5c709ada (patch)
tree82192c5a742b395c645a983f9fbbc7c188db7601 /dirent
parent54e4efc2876b329ba80a6965a2583a906d99e694 (diff)
downloadglibc-72f29e3aab56cf9bd11d60ae351fbf9d5c709ada.zip
glibc-72f29e3aab56cf9bd11d60ae351fbf9d5c709ada.tar.gz
glibc-72f29e3aab56cf9bd11d60ae351fbf9d5c709ada.tar.bz2
scandir: fix wrong assumption about errno [BZ #17804]
malloc and realloc may set errno to ENOMEM even if they are successful. The scandir code wrongly assume that they do not change errno, this causes scandir to fail with ENOMEM even if malloc succeed. The code already handles that readdir might set errno by calling __set_errno (0) to clear the error. Move that part at the end of the loop to also take malloc and realloc into account. Changelog: [BZ #17804] * dirent/scandir-tail.c (SCANDIR_TAIL): Move __set_errno (0) at the end of the loop. Improve comments.
Diffstat (limited to 'dirent')
-rw-r--r--dirent/scandir-tail.c13
1 files changed, 8 insertions, 5 deletions
diff --git a/dirent/scandir-tail.c b/dirent/scandir-tail.c
index 068c644..a02c99a 100644
--- a/dirent/scandir-tail.c
+++ b/dirent/scandir-tail.c
@@ -53,16 +53,14 @@ SCANDIR_TAIL (DIR *dp,
{
int selected = (*select) (d);
- /* The SELECT function might have changed errno. It was
- zero before and it need to be again to make the later
- tests work. */
+ /* The SELECT function might have set errno to non-zero on
+ success. It was zero before and it needs to be again to
+ make the later tests work. */
__set_errno (0);
if (!selected)
continue;
}
- else
- __set_errno (0);
if (__glibc_unlikely (c.cnt == vsize))
{
@@ -81,6 +79,11 @@ SCANDIR_TAIL (DIR *dp,
if (vnew == NULL)
break;
v[c.cnt++] = (DIRENT_TYPE *) memcpy (vnew, d, dsize);
+
+ /* Ignore errors from readdir, malloc or realloc. These functions
+ might have set errno to non-zero on success. It was zero before
+ and it needs to be again to make the latter tests work. */
+ __set_errno (0);
}
if (__glibc_likely (errno == 0))