diff options
author | Paul Eggert <eggert@cs.ucla.edu> | 2020-02-13 10:41:10 -0800 |
---|---|---|
committer | Paul Eggert <eggert@cs.ucla.edu> | 2020-02-13 10:41:54 -0800 |
commit | d17d3a3cc55500709c22c7d0ade24a0845aa17b9 (patch) | |
tree | 7c1ebef1fa2efc343c37d1d93ea3643db8a504e5 | |
parent | 4fcedca004fd13aecb5c6f235a988a5548bcb9a4 (diff) | |
download | gnulib-d17d3a3cc55500709c22c7d0ade24a0845aa17b9.tar.gz |
fchmodat, lchmod: port to buggy Linux filesystems
Problem reported by Florian Weimer in:
https://www.sourceware.org/ml/libc-alpha/2020-02/msg00534.html
* lib/fchmodat.c (fchmodat):
* lib/lchmod.c (lchmod):
Don’t assume that chmod on the O_PATH-opened fd will do
the right thing on a symbolic link.
* lib/fchmodat.c (fchmodat):
Don’t attempt to special-case
any flag value other than AT_SYMLINK_NOFOLLOW.
-rw-r--r-- | ChangeLog | 13 | ||||
-rw-r--r-- | lib/fchmodat.c | 33 | ||||
-rw-r--r-- | lib/lchmod.c | 27 |
3 files changed, 61 insertions, 12 deletions
@@ -1,3 +1,16 @@ +2020-02-13 Paul Eggert <eggert@cs.ucla.edu> + + fchmodat, lchmod: port to buggy Linux filesystems + Problem reported by Florian Weimer in: + https://www.sourceware.org/ml/libc-alpha/2020-02/msg00534.html + * lib/fchmodat.c (fchmodat): + * lib/lchmod.c (lchmod): + Don’t assume that chmod on the O_PATH-opened fd will do + the right thing on a symbolic link. + * lib/fchmodat.c (fchmodat): + Don’t attempt to special-case + any flag value other than AT_SYMLINK_NOFOLLOW. + 2020-02-11 Paul Eggert <eggert@cs.ucla.edu> lchmod: pacify Coverity CID 1491216 diff --git a/lib/fchmodat.c b/lib/fchmodat.c index 87aa0d191a..02e2da956e 100644 --- a/lib/fchmodat.c +++ b/lib/fchmodat.c @@ -63,12 +63,31 @@ orig_fchmodat (int dir, char const *file, mode_t mode, int flags) int fchmodat (int dir, char const *file, mode_t mode, int flags) { - if (flags & AT_SYMLINK_NOFOLLOW) + if (flags == AT_SYMLINK_NOFOLLOW) { -# ifdef O_PATH + struct stat st; + +# if defined O_PATH && defined AT_EMPTY_PATH int fd = openat (dir, file, O_PATH | O_NOFOLLOW | O_CLOEXEC); if (fd < 0) return fd; + + /* Use fstatat because fstat does not work on O_PATH descriptors + before Linux 3.6. */ + if (fstatat (fd, "", &st, AT_EMPTY_PATH) != 0) + { + int stat_errno = errno; + close (fd); + errno = stat_errno; + return -1; + } + if (S_ISLNK (st.st_mode)) + { + close (fd); + errno = EOPNOTSUPP; + return -1; + } + static char const fmt[] = "/proc/self/fd/%d"; char buf[sizeof fmt - sizeof "%d" + INT_BUFSIZE_BOUND (int)]; sprintf (buf, fmt, fd); @@ -82,10 +101,8 @@ fchmodat (int dir, char const *file, mode_t mode, int flags) errno = chmod_errno; return chmod_result; } - /* /proc is not mounted; fall back on racy implementation. */ -# endif - - struct stat st; + /* /proc is not mounted. */ +# else int fstatat_result = fstatat (dir, file, &st, AT_SYMLINK_NOFOLLOW); if (fstatat_result != 0) return fstatat_result; @@ -94,7 +111,9 @@ fchmodat (int dir, char const *file, mode_t mode, int flags) errno = EOPNOTSUPP; return -1; } - flags &= ~AT_SYMLINK_NOFOLLOW; +# endif + /* Fall back on chmod, despite the race. */ + flags = 0; } return orig_fchmodat (dir, file, mode, flags); diff --git a/lib/lchmod.c b/lib/lchmod.c index 5fc658023c..c7191c07d8 100644 --- a/lib/lchmod.c +++ b/lib/lchmod.c @@ -37,10 +37,28 @@ lchmod (char const *file, mode_t mode) #if HAVE_FCHMODAT return fchmodat (AT_FDCWD, file, mode, AT_SYMLINK_NOFOLLOW); #else -# if defined O_PATH && defined AT_FDCWD +# if defined AT_FDCWD && defined O_PATH && defined AT_EMPTY_PATH int fd = openat (AT_FDCWD, file, O_PATH | O_NOFOLLOW | O_CLOEXEC); if (fd < 0) return fd; + + /* Use fstatat because fstat does not work on O_PATH descriptors + before Linux 3.6. */ + struct stat st; + if (fstatat (fd, "", &st, AT_EMPTY_PATH) != 0) + { + int stat_errno = errno; + close (fd); + errno = stat_errno; + return -1; + } + if (S_ISLNK (st.st_mode)) + { + close (fd); + errno = EOPNOTSUPP; + return -1; + } + static char const fmt[] = "/proc/self/fd/%d"; char buf[sizeof fmt - sizeof "%d" + INT_BUFSIZE_BOUND (int)]; sprintf (buf, fmt, fd); @@ -54,10 +72,8 @@ lchmod (char const *file, mode_t mode) errno = chmod_errno; return chmod_result; } - /* /proc is not mounted; fall back on racy implementation. */ -# endif - -# if HAVE_LSTAT + /* /proc is not mounted. */ +# elif HAVE_LSTAT struct stat st; int lstat_result = lstat (file, &st); if (lstat_result != 0) @@ -69,6 +85,7 @@ lchmod (char const *file, mode_t mode) } # endif + /* Fall back on chmod, despite the race. */ return chmod (file, mode); #endif } |