On Thu, 29 Nov 2012 20:02:08 -0500 Jeff Layton <jlayton@xxxxxxxxxx> wrote: > On Thu, 29 Nov 2012 18:17:14 -0600 > Steve French <smfrench@xxxxxxxxx> 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 <jlayton@xxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html