From: Chuck Lever Date: Tue, 31 Aug 2010 19:30:14 +0000 (-0400) Subject: libexport.a: IPv6 support for client_init_subnet() X-Git-Tag: nfs-utils-1-2-3-rc6~25 X-Git-Url: https://git.decadent.org.uk/gitweb/?p=nfs-utils.git;a=commitdiff_plain;h=965b15b855c0c621462256b0ab687fc32644255a;hp=2824097496f6b154befbf3b3d15dacf237b07f83 libexport.a: IPv6 support for client_init_subnet() To parse and store an IPv6 host or subnet address, init_netmask() needs to handle 128 bit subnet masks. Unfortunately what once was a pretty simple little function has grown much larger. This logic must now not only parse IPv6 addresses correctly, but must also distinguish between IPv4 and IPv6. To avoid code duplication, I'm "bending" the cardinal rule of not using "#ifdef" inside functions. Signed-off-by: Chuck Lever Signed-off-by: Steve Dickson --- diff --git a/support/export/client.c b/support/export/client.c index 780c74d..4c6cd69 100644 --- a/support/export/client.c +++ b/support/export/client.c @@ -66,6 +66,12 @@ init_netmask(nfs_client *clp, const char *slash, const sa_family_t family) }; unsigned long prefixlen; uint32_t shift; +#ifdef IPV6_SUPPORTED + struct sockaddr_in6 sin6 = { + .sin6_family = AF_INET6, + }; + int i; +#endif /* No slash present; assume netmask is all ones */ if (slash == NULL) { @@ -73,6 +79,11 @@ init_netmask(nfs_client *clp, const char *slash, const sa_family_t family) case AF_INET: prefixlen = 32; break; +#ifdef IPV6_SUPPORTED + case AF_INET6: + prefixlen = 128; + break; +#endif default: goto out_badfamily; } @@ -87,6 +98,14 @@ init_netmask(nfs_client *clp, const char *slash, const sa_family_t family) set_addrlist_in(clp, 1, &sin); return 1; } +#ifdef IPV6_SUPPORTED + if (strchr(slash + 1, ':')) { + if (!inet_pton(AF_INET6, slash + 1, &sin6.sin6_addr)) + goto out_badmask; + set_addrlist_in6(clp, 1, &sin6); + return 1; + } +#endif /* A prefixlen was given */ prefixlen = strtoul(slash + 1, &endptr, 10); @@ -102,6 +121,19 @@ init_netmask(nfs_client *clp, const char *slash, const sa_family_t family) sin.sin_addr.s_addr = htonl((uint32_t)~0 << shift); set_addrlist_in(clp, 1, &sin); return 1; +#ifdef IPV6_SUPPORTED + case AF_INET6: + if (prefixlen > 128) + goto out_badprefix; + for (i = 0; prefixlen > 32; i++) { + sin6.sin6_addr.s6_addr32[i] = 0xffffffff; + prefixlen -= 32; + } + shift = 32 - (uint32_t)prefixlen; + sin6.sin6_addr.s6_addr32[i] = htonl((uint32_t)~0 << shift); + set_addrlist_in6(clp, 1, &sin6); + return 1; +#endif } out_badfamily: