fix failure of fchmodat to report EOPNOTSUPP in the race path
authorRich Felker <dalias@aerifal.cx>
Fri, 6 Feb 2015 04:34:27 +0000 (23:34 -0500)
committerRich Felker <dalias@aerifal.cx>
Fri, 6 Feb 2015 04:34:27 +0000 (23:34 -0500)
in the case where a non-symlink file was replaced by a symlink during
the fchmodat operation with AT_SYMLINK_NOFOLLOW, mode change on the
new symlink target was successfully suppressed, but the error was not
reported. instead, fchmodat simply returned 0.

src/stat/fchmodat.c

index a894cb696bbad9852260a805a0a6a79d298cd18f..d94667aed5e1a4aa1e924bd89a8ba3aa3ec6aa16 100644 (file)
@@ -29,8 +29,10 @@ int fchmodat(int fd, const char *path, mode_t mode, int flag)
 
        __procfdname(proc, fd2);
        ret = __syscall(SYS_fstatat, AT_FDCWD, proc, &st, 0);
-       if (!ret && !S_ISLNK(st.st_mode))
-               ret = __syscall(SYS_fchmodat, AT_FDCWD, proc, mode);
+       if (!ret) {
+               if (S_ISLNK(st.st_mode)) ret = -EOPNOTSUPP;
+               else ret = __syscall(SYS_fchmodat, AT_FDCWD, proc, mode);
+       }
 
        __syscall(SYS_close, fd2);
        return __syscall_ret(ret);