This is the mail archive of the
libc-help@sourceware.org
mailing list for the glibc project.
Re: access call not calling faccessat correctly
- From: Florian Weimer <fweimer at redhat dot com>
- To: Michael Morrell <mmorrell at tachyum dot com>
- Cc: "libc-help\@sourceware.org" <libc-help at sourceware dot org>
- Date: Mon, 21 Oct 2019 20:07:52 +0200
- Subject: Re: access call not calling faccessat correctly
- References: <2da56ba636a8462393450dfb6e3f5c06@tachyum.com>
* Michael Morrell:
> I was seeing a failure in the chmod_1.f90 test that is part of gcc's validation testsuite and am pretty sure it is caused by some bad code in glibc.
>
> In sysdeps/unix/sysv/linux/access.c, a call to "access()" is implemented by either calling the access syscall or the faccessat syscall, depending on whether __NR_access is #defined. If __NR_access is not #defined and the faccessat syscall is used, it is only called with 3 arguments - no value for the flags argument is passed. This causes EINVAL to be returned in some cases, depending on whatever happens to be in the register that would be used to pass that argument.
>
> I think the change is simple:
>
> diff --git a/sysdeps/unix/sysv/linux/access.c b/sysdeps/unix/sysv/linux/access.c
> index 524594ac79..706f242adb 100644
> --- a/sysdeps/unix/sysv/linux/access.c
> +++ b/sysdeps/unix/sysv/linux/access.c
> @@ -26,7 +26,7 @@ __access (const char *file, int type)
> #ifdef __NR_access
> return INLINE_SYSCALL_CALL (access, file, type);
> #else
> - return INLINE_SYSCALL_CALL (faccessat, AT_FDCWD, file, type);
> + return INLINE_SYSCALL_CALL (faccessat, AT_FDCWD, file, type, 0);
> #endif
> }
> weak_alias (__access, access)
>
> I haven't filed a bug yet, but can do so.
The faccessat system call takes only three arguments. It's always been
this way.
What you are seeing must be the result of something else.
Thanks,
Florian