On 01/16/2015 09:51 AM, Cédric Bosdonnat wrote:
> ---
> src/util/virsocketaddr.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/util/virsocketaddr.c b/src/util/virsocketaddr.c
> index 91bcadf..67ed330 100644
> --- a/src/util/virsocketaddr.c
> +++ b/src/util/virsocketaddr.c
> @@ -850,7 +850,7 @@ virSocketAddrGetIpPrefix(const virSocketAddr *address,
> }
> return -1;
> } else if (VIR_SOCKET_ADDR_IS_FAMILY(address, AF_INET6)) {
> - if (address->data.inet6.sin6_addr.s6_addr == 0)
> + if (virSocketAddrIsWildcard(address))
> return 0;
> return 64;
> }
According to sockettest.c, that function does work for "::", and
examining the macro that it uses (from /usr/include/netinet/in.h) I see
that it is doing exactly what we need (checks all 4 uint32s in the
address against 0). So ACK.
(Nice! We should have been using that all along, instead of parsing out
"::" and calling virSocketAddrEqual())
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list