namei() returns EISDIR for "/" (Re: svn commit: r203990 - head/lib/libc/sys)

Alexander Best alexbestms at wwu.de
Sun Feb 28 19:18:39 UTC 2010


Bruce Evans said that the doesn't like this comment you added in your patchset

        /*
         * Check for "" which is a way of talking about the root directory.
         * We can't provide a parent node for CREATE, DELETE and RENAME
         * operations.
         */

and would like to keep "/" the way it is instead of stripping the slash.
however this comment

        /*
         * Replace multiple slashes by a single slash and trailing slashes
         * by a null.  This must be done before VOP_LOOKUP() because some
         * fs's don't know about trailing slashes.  Remember if there were
         * trailing slashes to handle symlinks, existing non-directories
         * and non-existing files that won't be directories specially later.
         */

says that some fs's can't handle trailing slashes. how exactly can the root
dir be expressed on these fs's? is it in fact ""? and which fs's exactly are
lacking support for trailing slashes?

cheers.
alex


More information about the freebsd-hackers mailing list