From: Samuel Thibault Date: Thu, 29 Mar 2018 19:47:29 +0000 (+0100) Subject: hurd: Fix O_DIRECTORY | O_NOFOLLOW X-Git-Tag: archive/raspbian/2.27-3+rpi1^2~68 X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=ee0e6ec72a30449ae2ecfa6bee6f85b979aab3e5;p=glibc.git hurd: Fix O_DIRECTORY | O_NOFOLLOW Appending / to the path to be looked up would make us always follow a final symlink, even with O_NOTRANS (since the final resolution is after the '/'). In the O_DIRECTORY | O_NOFOLLOW case, we thus have to really open the node and stat it, which we already do anyway, and check for directory type. Signed-off-by: Samuel Thibault Gbp-Pq: Topic hurd-i386 Gbp-Pq: Name git-NOFOLLOW-DIRECTORY.diff --- diff --git a/hurd/hurdlookup.c b/hurd/hurdlookup.c index 1861d5879..a642c4900 100644 --- a/hurd/hurdlookup.c +++ b/hurd/hurdlookup.c @@ -72,7 +72,7 @@ __hurd_file_name_lookup (error_t (*use_init_port) if (flags & O_NOFOLLOW) /* See lookup-retry.c about O_NOFOLLOW. */ flags |= O_NOTRANS; - if (flags & O_DIRECTORY) + if (flags & O_DIRECTORY && (flags & O_NOFOLLOW) == 0) { /* The caller wants to require that the file we look up is a directory. We can do this without an extra RPC by appending a trailing slash diff --git a/hurd/lookup-retry.c b/hurd/lookup-retry.c index 31376ef56..1115d4b92 100644 --- a/hurd/lookup-retry.c +++ b/hurd/lookup-retry.c @@ -147,6 +147,8 @@ __hurd_file_name_lookup_retry (error_t (*use_init_port) err = __io_stat (*result, &st); if (!err) { + if (flags & O_DIRECTORY && !S_ISDIR(st.st_mode)) + err = ENOTDIR; if (S_ISLNK(st.st_mode)) err = ELOOP; else if (st.st_mode & (S_IPTRANS|S_IATRANS))