Patch to allow trailing dots on managed mounts

Pierre A. Humblet pierre@phumblet.no-ip.org
Thu Dec 16 16:56:00 GMT 2004


cgf wrote:

> Is it correct to assume that only fhandler_base::open cares about
>trailing dots?

Good point. This bring back memories.

The initial motivation was to fix problems introduced by the 
use of NtCreateFile 

http://www.cygwin.com/ml/cygwin/2004-04/msg01250.html
and there were successive changes

2004-04-30  Corinna Vinschen  <corinna@vinschen.de>

        * path.cc (normalize_posix_path): Remove trailing dots and spaces.

http://cygwin.com/ml/cygwin-patches/2004-q2/msg00053.html

2004-05-06  Pierre Humblet <pierre.humblet@ieee.org>

        * path.cc (path_conv::check): Strip trailing dots and spaces and
        return error if the final component had only dots and spaces.
        (normalize_posix_path): Revert 2004-04-30.

However, as a side effect, checking the tail in :check
also cleanly fixed longstanding dormant issues with the path hash
and with chroot (at least).

So checking the tail in fhandler_base::open() is too late.
It should be done before exiting  :check(), perhaps only in the
case where the path refers to a disk file, preferably
with little processing overhead.

Although it wasn't done before 2004/04, we should also make sure
(I have no free time for the moment) that nothing goes wrong 
inside :check() while we lookup symbolic links.

Pierre



More information about the Cygwin-patches mailing list