On Thu, 29 Nov 2012 20:02:08 -0500
Jeff Layton <[email protected]> wrote:

> On Thu, 29 Nov 2012 18:17:14 -0600
> Steve French <[email protected]> wrote:
> 
> > Jeff,
> > In your patch "cifs: clean up handling of unc= option       commit" aren't
> > you changing the parsing of not just the beginning of the UNC name to
> > change //server/share to \\server/share (e.g.) but did you mean to
> > change the parsing of the rest of the UNC name as well.  Doesn't
> > convert delimiter change all occurrences not just the first) ?
> > 
> 
> Yes, that's intentional.
> 

To clarify...

The mount helper has passed in the unc= option with '\\' delimiters for
a long time, regardless of what the user passes in. '/' is not an
allowable character in a UNC component, so there's no harm in
converting those characters unconditionally.

The prefixpath is different, of course...

-- 
Jeff Layton <[email protected]>
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to