This is the mail archive of the
mailing list for the Cygwin project.
Re: [Patch] cygcheck: warn about trailing (back)slash on mount entries
Christopher Faylor wrote:
> On Wed, Oct 06, 2004 at 11:30:17AM -0400, Pierre A. Humblet wrote:
> >Christopher Faylor wrote:
> >> On Wed, Oct 06, 2004 at 03:12:45PM +0200, Bas van Gompel wrote:
> >> >Another (hopefully trivial) patch, to help in trouble-shooting.
> >> Wasn't there another problem where "foo\/bar" type of entries were
> >> showing up? Could you add a check for that, too?
> >I while ago I have modified Cygwin to accept this kind of syntax.
> >Is there a remaining problem in the current release?
> >Otherwise I don't see the need to alarm the user.
> It's just a warning. This really shouldn't be in the mount table
> and it really should be corrected.
I don't think it's checking the mount table, it's checking the registry.
The entry will be cleaned up by the time it gets to the mount table.
What would be useful is a check that ::add_item will accept the registry
entry, i.e. won't return EINVAL or perhaps "path too long".
The relevant part of add_item is pasted below. It shows when EINVAL
mount_info::add_item (const char *native, const char *posix, unsigned mountflags, int reg_p)
char *nativetail, *posixtail, error = "error";
int nativeerr, posixerr;
/* Something's wrong if either path is NULL or empty, or if it's
not a UNC or absolute path. */
if (native == NULL || !isabspath (native) ||
!(is_unc_share (native) || isdrive (native)))
nativeerr = EINVAL;
nativeerr = normalize_win32_path (native, nativetmp, &nativetail);
if (posix == NULL || !isabspath (posix) ||
is_unc_share (posix) || isdrive (posix))
posixerr = EINVAL;
posixerr = normalize_posix_path (posix, posixtmp, &posixtail);
debug_printf ("%s[%s], %s[%s], %p",
native, nativeerr ? error : nativetmp,
posix, posixerr ? error : posixtmp, mountflags);
if (nativeerr || posixerr)
/* Make sure both paths do not end in /. */
if (nativetail > nativetmp + 1 && nativetail[-1] == '\\')
nativetail[-1] = '\0';
if (posixtail > posixtmp + 1 && posixtail[-1] == '/')
posixtail[-1] = '\0';