Re: Automounting of smb shares with dollar sign

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 2016-07-01 at 11:15 +0800, Ian Kent wrote:
> On Thu, 2016-06-30 at 13:21 +0300, Yury Krivopalov wrote:
> > Hello,
> > I found a problem, that started to reproduce on my new environment
> > (archlinux), before that I used Debian Wheezy and everything worked fine.
> > The point is that I can't automount smb shares with dollar sign (Ex.:
> > //hostname/C$).
> >  
> > It happend because two facts:
> > 1. Autofs reads '$' as a special symbol
> > 2. mount.cifs reads '\\' as a delimiter
> >  
> > So in my case if I returns "//hostname/C$" from auto.smb, then automount
> > daemon replace '$' to '' and if I returns "//hostname/C\\$" then mount.cifs
> > crops "\\$" and uses '$' as a prefixpath mount option.
> 
> Escaping of cifs mount entries has been a long standing problem over and over
> again.
> 
> Mostly fixes seem to always to have bad side effects, dropping them of trying
> to
> fix them often results in the problems returning.
> 
> I'll have a look at this and try and work out if it would have side effects
> for
> other than cifs mounts.
> 
> >  
> > I attached simple fix for autofs-5.1.2, that drops back slash after
> > arguments
> > parsing. It works in my case, 

Actually, now I look closer, this might be a bug in expandsunent().

The expandsunent() function is called before parsing the map entry not after so
changing this could have unexpected side effects.

I'll run parser tests with this change and see what happens, when I get a
chance.

> >  
> 
> Thanks for our patch.
> Ian
> --
> To unsubscribe from this list: send the line "unsubscribe autofs" in
--
To unsubscribe from this list: send the line "unsubscribe autofs" in



[Index of Archives]     [Linux Filesystem Development]     [Linux Ext4]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux