Re: [PATCH] systemd: Fix build on musl
On Thu, Jul 29, 2021 at 6:49 AM Luca Bocassi <luca.boccassi@...> wrote:
I wrote a long and verbose comment when I created the patch which
tries to document any differences in runtime behaviour.
Avoid using AT_SYMLINK_NOFOLLOW flag. It doesn't seem like the right thing to
do and it's not portable (not supported by musl). See:
Note that laccess() is never passing AT_EACCESS so a lot of the discussion in
the links above doesn't apply. Note also that (currently) all systemd callers
of laccess() pass mode as F_OK, so only check for existence of a file, not
access permissions. Therefore, in this case, the only distiction between
faccessat() with (flag == 0) and (flag == AT_SYMLINK_NOFOLLOW) is the
behaviour for broken symlinks; laccess() on a broken symlink will succeed
with (flag == AT_SYMLINK_NOFOLLOW) and fail (flag == 0).
The laccess() macros was added to systemd some time ago and it's not clear if
or why it needs to return success for broken symlinks. Maybe just historical
and not actually necessary or desired behaviour?
If that comment is now out of date or something is missing then please
send a patch to update it.
However looking at this patch again now, it appears to have got broken
during a past rebase:
The upstream code changed from:
#define laccess(path, mode) faccessat(AT_FDCWD, (path), (mode),
#define laccess(path, mode) \
(faccessat(AT_FDCWD, (path), (mode), AT_SYMLINK_NOFOLLOW) <
0 ? -errno : 0)
but the replacement version in the patch still returns the raw result
from faccessat(). That looks like an issue.