On Tue, May 01, 2012 at 04:09:55PM +0000, Adamson, Dros wrote:
>
> On Apr 27, 2012, at 7:25 AM, J. Bruce Fields wrote:
>
> > On Tue, Apr 24, 2012 at 10:59:37AM -0400, Weston Andros Adamson wrote:
> >> The fs_location->hosts list is split on colons, but this doesn't work when
> >> IPv6 addresses are used (they contain colons).
> >> This patch adds the function nfsd4_encode_components_esc() to
> >> allow the caller to specify escape characters when splitting on 'sep'.
> >> In order to fix referrals, this patch must be used with the mountd patch
> >> that similarly fixes IPv6 [] escaping.
> >
> > Thanks, looks fine, applying with one trivial change:
> >
> >> - for (; *end && (*end != sep); end++)
> >> - ; /* Point to end of component */
> >> + bool found_esc = false;
> >> +
> >> + /* try to parse as esc_start, ..., esc_end, sep */
> >> + if (*str == esc_enter) {
> >> + for (; *end && (*end != esc_exit); end++);
> >
> > I kinda like keeping the semicolon on its own line here. Tastes may
> > differ.
>
> Yeah, as Jim pointed out checkpatch will complain if it's not -- you already found my updated diff.
>
> > (Also: how did you test this?)
>
> I testsed this patch along with the corresponding mountd patch using two servers:
>
> [fc00::10] - exporting /export, /export/refer @ [fc00:41]:/export
> [fc00::41] - exporting /export
>
> On a client machine (with recently posted client-side fix), I mount [fc00::10]:/export on /mnt, then cd to /mnt/refer.
>
> Without these patches, wireshark sees a list of referral locations: "[fc00", "", "10]", which is obviously wrong.
> With these patches the cd to /mnt/refer works as [fc00::41]:/export is mounted at /mnt/refer.
Sounds good. Thanks!
--b.
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
