Laine Stump wrote:
There is a potential problem here. When this function was renamed
from
brAddInetAddress(), it was incorrectly given the "IPv4" moniker. In
fact, it is used to add either IPv4 or IPv6 addresses to the interface.
In order for this to work correctly, you'll need to check the family of
the virSocketAddr and use either "inet" or "inet6" accordingly.
(and this function should really be renamed to virNetDevAddIPAddress,
since it is v4 or v6, and multiple addresses can be added.
virNetDevClearIPv4Address should similarly be changed to
virNetDevDelIPAddress (or maybe they should be
virNetDevIPAddress(Add|Del) ) (That's a separate issue, though).
Thanks for suggestion, I have added a check.
BTW, doc string for virNetDevSetIPv4Address() references brDelInetAddress() which
doesn't exist (I guess it's a leftover after rename as well).
Roman Bogorodskiy