Merge "Document more clearly that we don't plan on using faccessat2(2)."

This commit is contained in:
Treehugger Robot 2021-06-25 20:04:28 +00:00 committed by Gerrit Code Review
commit b4b3166d7b

View file

@ -48,10 +48,10 @@ int faccessat(int dirfd, const char* pathname, int mode, int flags) {
// More details at http://permalink.gmane.org/gmane.linux.lib.musl.general/6952 // More details at http://permalink.gmane.org/gmane.linux.lib.musl.general/6952
// //
// AT_EACCESS isn't supported either. Android doesn't have setuid // AT_EACCESS isn't supported either. Android doesn't have setuid
// programs, and never runs code with euid!=uid. It could be // programs, and never runs code with euid!=uid.
// implemented in an expensive way, following the model at //
// https://gitlab.com/bminor/musl/commit/0a05eace163cee9b08571d2ff9d90f5e82d9c228 // We could use faccessat2(2) from Linux 5.8, but since we don't want the
// but not worth it. // first feature and don't need the second, we just reject such requests.
errno = EINVAL; errno = EINVAL;
return -1; return -1;
} }