]> xenbits.xensource.com Git - libvirt.git/commit
util: Clear output broadcast address before filling it in
authorMartin Kletzander <mkletzan@redhat.com>
Wed, 3 Jun 2015 10:46:36 +0000 (12:46 +0200)
committerMartin Kletzander <mkletzan@redhat.com>
Wed, 3 Jun 2015 10:54:32 +0000 (12:54 +0200)
commit99cc11b924548ae58b60e80e14467387baf61e42
tree4a671dc97ae90278f96abcffc162636f11b2f3dd
parente9507fd41c9c6b73093cc0a4ce568bf0d8204854
util: Clear output broadcast address before filling it in

Since commit 55ace7c4789c8a7408139460f4b639cee00e5125, the sockettest
fails without VIR_TEST_DEBUG set.  The problem is found by test number
42 (co-incidence?), which tests range '192.168.122.1' -
'192.168.122.255' in network '192.168.122.0/24'.  That is supposed to
fail because the end address is equal to the broadcast address.

When comparing these two in 'virSocketAddrEqual(end, &broadcast)',
there is a check for sin_addr as well as for sin_port.  That port,
however, is different when we do not enable test debugging.  With the
testing enabled, the port is 0 (correctly initialized), but without that
it has a random number there.  And that's because the structure is not
initialized anywhere.

By zeroing the structure before filling in the info, we make sure we
return only the address and not any information that was not requested.
And the test work once again.

Signed-off-by: Martin Kletzander <mkletzan@redhat.com>
src/util/virsocketaddr.c