[bug #29655] linkat() fails because __file_name_lookup_at() problems

[Carl Fredrik Hammar]

Follow-up Comment #4, bug #29655 (project hurd):

This has already been said on IRC but I might as well repeat it here:
POSIX.1-2008 says that only `AT_SYMLINK_FOLLOW` is applicable to linkat()
(see http://www.opengroup.org/onlinepubs/9699919799/functions/linkat.html).
I would recommend testing it in `__file_name_lookup_at()` as I described
earlier.  Otherwise, you'd need to check it in all `at` functions
except `openat()', because they should fail when they're given
`AT_SYMLINK_FOLLOW`.

You are a bit too verbose in the ChangeLog, the first item is fine,
but the second item should just be "Updated comment."  or some such.
In the third item, you shouldn't mention motivation, i.e "Pass O_NOLINK
in flags" would be enough.  Such information should be placed in code
comments, as it already is.  Also there should be no blank lines between
related items, and they are all related:

    * item1 (foo): ...
    * item2 (bar): ...



    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?29655>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/