]> xenbits.xensource.com Git - libvirt.git/commit
qemu: don't set/clear NetDef IP addresses in qemuConnectDomainXMLToNative()
authorLaine Stump <laine@laine.org>
Tue, 7 Jun 2016 23:59:10 +0000 (19:59 -0400)
committerLaine Stump <laine@laine.org>
Sun, 26 Jun 2016 23:33:08 +0000 (19:33 -0400)
commit910450928976ec1dc0f1c4047789356f8acc6a4f
tree968445fa6bc1b56f515d5b023b1cc35e268568fc
parent7cfbaad189aa635112770cffe70e2bfb2be288c7
qemu: don't set/clear NetDef IP addresses in qemuConnectDomainXMLToNative()

This patch removes the expanded and duplicated code that all sprung
out of two well-intentioned-but-useless settings of
net->data.(bridge|ethernet).ipaddr.

qemu has never supported even a single IP address in the interface
config, much less a list of them. All of the instances of "clearing
out the IP addresses" that are now in this function originated with
commit d8dbd6 "Basic domain XML conversions for Xen/QEMU drivers" in
May 2009, but even then the single "ipaddr" in the struct for
type='ethernet' and type='bridge' wasn't used in the qemu driver (only
in xen and openvz). Since then anyone who added a new interface type
also tacked on another unnecessary clearing of ipaddr, and when it was
made into a list of IPs (so far supported only by the LXC driver) this
simple setting was turned into a loop (well, multiple loops) to clear
them all.
src/qemu/qemu_driver.c