openat: correct new comment
authorBastien Roucariès <roucaries.bastien@gmail.com>
Tue, 10 May 2011 09:30:19 +0000 (11:30 +0200)
committerJim Meyering <meyering@redhat.com>
Tue, 10 May 2011 09:30:19 +0000 (11:30 +0200)
* lib/openat-proc.c (openat_proc_name): Correct the comment.

ChangeLog
lib/openat-proc.c

index 5a89d18..74919bb 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2011-05-10  Bastien Roucariès  <roucaries.bastien@gmail.com>
+
+       openat: correct new comment
+       * lib/openat-proc.c (openat_proc_name): Correct the comment.
+
 2011-05-10  Jim Meyering  <meyering@redhat.com>
 
        openat: add comments
index c326ffc..2e2e966 100644 (file)
@@ -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;