From: Bastien Roucariès Date: Tue, 10 May 2011 09:30:19 +0000 (+0200) Subject: openat: correct new comment X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=38ed8cbdaac9ed0f1c8cbedfac78ca3f10991528;p=pspp openat: correct new comment * lib/openat-proc.c (openat_proc_name): Correct the comment. --- diff --git a/ChangeLog b/ChangeLog index 5a89d18d5b..74919bb30c 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2011-05-10 Bastien Roucariès + + openat: correct new comment + * lib/openat-proc.c (openat_proc_name): Correct the comment. + 2011-05-10 Jim Meyering openat: add comments diff --git a/lib/openat-proc.c b/lib/openat-proc.c index c326ffcbef..2e2e966bc7 100644 --- a/lib/openat-proc.c +++ b/lib/openat-proc.c @@ -80,10 +80,11 @@ openat_proc_name (char buf[OPENAT_BUFFER_SIZE], int fd, char const *file) proc_status = -1; else { - /* Detect whether /proc/self/fd/../fd exists. On Linux, that name - resolves to /proc/self/fd, which was opened above. However, on - Solaris, it may resolve to /proc/self/fd/fd, which cannot exist, - since all names in /proc/self/fd are numeric. */ + /* Detect whether /proc/self/fd/%i/../fd exists, where %i is the + number of a file descriptor open on /proc/self/fd. On Linux, + that name resolves to /proc/self/fd, which was opened above. + However, on Solaris, it may resolve to /proc/self/fd/fd, which + cannot exist, since all names in /proc/self/fd are numeric. */ char dotdot_buf[PROC_SELF_FD_NAME_SIZE_BOUND (sizeof "../fd" - 1)]; sprintf (dotdot_buf, PROC_SELF_FD_FORMAT, proc_self_fd, "../fd"); proc_status = access (dotdot_buf, F_OK) ? -1 : 1;