[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] fchmodat, lchmod: port to buggy Linux filesystems
From: |
Pádraig Brady |
Subject: |
Re: [PATCH] fchmodat, lchmod: port to buggy Linux filesystems |
Date: |
Mon, 9 Mar 2020 17:30:06 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:73.0) Gecko/20100101 Thunderbird/73.0 |
On 13/02/2020 18:42, Paul Eggert wrote:
Problem reported by Florian Weimer in:
https://www.sourceware.org/ml/libc-alpha/2020-02/msg00534.html
A very similar "ENOTSUP" problem is being reported with coreutils-8.32
with `mknod -m 666 /dev/random c 1 8` when trying to build Fedora rawhide in a
chroot.
https://bugzilla.redhat.com/1811038
BTW I see a possible small issue in lchmod.c in this code:
+ if (chmod_errno != ENOENT)
+ {
+ errno = chmod_errno;
+ return chmod_result;
+ }
Shouldn't that also check for other lookup errors like ENOTDIR and ELOOP ?
Also mknod() in coreutils is calling lchmod().
Shouldn't it be just calling chmod() as if mknod() was passed a
symlink, then it would have already failed with EEXIST?
cheers,
Pádraig
- Re: [PATCH] fchmodat, lchmod: port to buggy Linux filesystems,
Pádraig Brady <=
- Re: [PATCH] fchmodat, lchmod: port to buggy Linux filesystems, Paul Eggert, 2020/03/09
- Re: [PATCH] fchmodat, lchmod: port to buggy Linux filesystems, Pádraig Brady, 2020/03/09
- Re: [PATCH] fchmodat, lchmod: port to buggy Linux filesystems, Florian Weimer, 2020/03/10
- Re: [PATCH] fchmodat, lchmod: port to buggy Linux filesystems, Kamil Dudka, 2020/03/10
- Re: [PATCH] fchmodat, lchmod: port to buggy Linux filesystems, Pádraig Brady, 2020/03/10
- Re: [PATCH] fchmodat, lchmod: port to buggy Linux filesystems, Florian Weimer, 2020/03/10
- Re: [PATCH] fchmodat, lchmod: port to buggy Linux filesystems, Paul Eggert, 2020/03/11
- Re: [PATCH] fchmodat, lchmod: port to buggy Linux filesystems, Florian Weimer, 2020/03/11
- Re: [PATCH] fchmodat, lchmod: port to buggy Linux filesystems, Paul Eggert, 2020/03/11
- Re: [PATCH] fchmodat, lchmod: port to buggy Linux filesystems, Paul Eggert, 2020/03/11