This is the mail archive of the cygwin-patches@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [Patch] cygcheck: warn about trailing (back)slash on mount entries


On Wed, Oct 06, 2004 at 12:00:15PM -0400, Pierre A. Humblet wrote:
>
>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,

Ok, sigh, yes, I "mispoke".  s/mount table/registry/.

>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 is returned.

cygcheck is not a cygwin program.  It has it's own implementation of
getmntent which reads the registry directly, and, it displays the \/
paths when you do a 'cygcheck -s'.  It can check for these types of
problems and it should warn about them.

cgf


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]