==> Regarding Re: [autofs] Re: [ANNOUNCE] Important update - autofs 4.1.4 
release; Ian Kent <[EMAIL PROTECTED]> adds:

raven> On Thu, 14 Apr 2005, Jeff Moyer wrote:
>> ==> Regarding Re: [autofs] Re: [ANNOUNCE] Important update - autofs 4.1.4 
>> release; Ian Kent <[EMAIL PROTECTED]> adds:
>> 
raven> On Thu, 14 Apr 2005, Ian Kent wrote:
>> >> On Wed, 13 Apr 2005, Jeff Moyer wrote:
>> >> 
>> >> > 
>> >> > This new version has regressed since beta2 for /net mounts in my 
>> >> environment.  A simple ls of /net/<filer> will now show the contents of
>> >> > /net/<filer>/exporteddir.  So, for example, we have devfiler which
>> >> exports /vol/vol1.  When I do an ls of /net/devfiler, I get the
>> >> contents of devfiler:/vol/vol1.
>> >> > 
>> >> > Okay, after further debugging, it seems we're not identifying this as
>> >> a multimount entry.  It's getting late, so I'm going to turn this over
>> >> to you.  I think that the checking of first_chunk is bogus.  For
>> >> instance, with the below change, everything works as expected for me.
>> >>
>> >> 
>> >> Cool, I'll check this out and get back.
>> 
raven> Could I have the example map entry that breaks the parse please.
>> 
>> Sheesh, you'd think that I could generate a bug report by now... I'll blame
>> it on the late night thing.  =)
>> 
>> This is an auto.net entry.  Here's what auto.net spits out for this
>> particular host:
>> 
>> -fstype=nfs,hard,intr,nodev,nosuid \
>> /vol/vol1 devfiler:/vol/vol1
>> 
>> I'm guessing you didn't test auto.net against a host that only exported one
>> dir?  Just a guess.  Below are the debug logs.
>> 

raven> I think this should do the trick.

Yes, you're probably right.  Please consider renaming the first_chunk
variable, too!

-Jeff

raven> --- autofs-4.1.4/modules/parse_sun.c.first-multi Fri Apr 15 14:30:18 2005
raven> +++ autofs-4.1.4/modules/parse_sun.c     Fri Apr 15 14:42:04 2005
raven> @@ -768,6 +768,10 @@ static int check_is_multi(const char *ma
raven>          int multi = 0;
raven>          int first_chunk = 0;
 
raven> +        /* If first character is "/" it's a multi-mount */
raven> +        if (*p == '/')
raven> +                return 1;
raven> +
raven>          while (*p) {
raven>                  p = skipspace(p);
 

_______________________________________________
autofs mailing list
autofs@linux.kernel.org
http://linux.kernel.org/mailman/listinfo/autofs

Reply via email to