mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2025-01-04 20:15:19 +00:00
2abde0ac07
This bug resolves CVE-2012-3411, which is described in the following
bugzilla report:
https://bugzilla.redhat.com/show_bug.cgi?id=833033
The following report is specifically for libvirt on Fedora:
https://bugzilla.redhat.com/show_bug.cgi?id=874702
In short, a dnsmasq instance run with the intention of listening for
DHCP/DNS requests only on a libvirt virtual network (which is
constructed using a Linux host bridge) would also answer queries sent
from outside the virtualization host.
This patch takes advantage of a new dnsmasq option "--bind-dynamic",
which will cause the listening socket to be setup such that it will
only receive those requests that actually come in via the bridge
interface. In order for this behavior to actually occur, not only must
"--bind-interfaces" be replaced with "--bind-dynamic", but also all
"--listen-address" options must be replaced with a single
"--interface" option. Fully:
--bind-interfaces --except-interface lo --listen-address x.x.x.x ...
(with --listen-address possibly repeated) is replaced with:
--bind-dynamic --interface virbrX
Of course libvirt can't use this new option if the host's dnsmasq
doesn't have it, but we still want libvirt to function (because the
great majority of libvirt installations, which only have mode='nat'
networks using RFC1918 private address ranges (e.g. 192.168.122.0/24),
are immune to this vulnerability from anywhere beyond the local subnet
of the host), so we use the new dnsmasqCaps API to check if dnsmasq
supports the new option and, if not, we use the "old" option style
instead. In order to assure that this permissiveness doesn't lead to a
vulnerable system, we do check for non-private addresses in this case,
and refuse to start the network if both a) we are using the old-style
options, and b) the network has a publicly routable IP
address. Hopefully this will provide the proper balance of not being
disruptive to those not practically affected, and making sure that
those who *are* affected get their dnsmasq upgraded.
(--bind-dynamic was added to dnsmasq in upstream commit
54dd393f3938fc0c19088fbd319b95e37d81a2b0, which was included in
dnsmasq-2.63)
(cherry picked from commit
|
||
|---|---|---|
| .. | ||
| capabilityschemadata | ||
| commanddata | ||
| confdata | ||
| cputestdata | ||
| domainschemadata | ||
| domainsnapshotxml2xmlin | ||
| domainsnapshotxml2xmlout | ||
| interfaceschemadata | ||
| lxcxml2xmldata | ||
| networkxml2argvdata | ||
| networkxml2xmlin | ||
| networkxml2xmlout | ||
| nodedevschemadata | ||
| nodeinfodata | ||
| nwfilterxml2xmlin | ||
| nwfilterxml2xmlout | ||
| qemuhelpdata | ||
| qemuxml2argvdata | ||
| qemuxml2xmloutdata | ||
| qemuxmlnsdata | ||
| sexpr2xmldata | ||
| storagepoolxml2xmlin | ||
| storagepoolxml2xmlout | ||
| storagevolxml2xmlin | ||
| storagevolxml2xmlout | ||
| vmx2xmldata | ||
| xencapsdata | ||
| xmconfigdata | ||
| xml2sexprdata | ||
| xml2vmxdata | ||
| .valgrind.supp | ||
| capabilityschematest | ||
| commandhelper.c | ||
| commandtest.c | ||
| conftest.c | ||
| cpuset | ||
| cputest.c | ||
| daemon-conf | ||
| define-dev-segfault | ||
| domainschematest | ||
| domainsnapshotschematest | ||
| domainsnapshotxml2xmltest.c | ||
| esxutilstest.c | ||
| eventtest.c | ||
| int-overflow | ||
| interfaceschematest | ||
| interfacexml2xmltest.c | ||
| jsontest.c | ||
| libvirtd-fail | ||
| libvirtd-pool | ||
| lxcxml2xmltest.c | ||
| Makefile.am | ||
| networkschematest | ||
| networkxml2argvtest.c | ||
| networkxml2xmltest.c | ||
| nodedevschematest | ||
| nodedevxml2xmltest.c | ||
| nodeinfotest.c | ||
| nwfilterschematest | ||
| nwfilterxml2xmltest.c | ||
| object-locking.ml | ||
| oomtrace.pl | ||
| openvzutilstest.c | ||
| openvzutilstest.conf | ||
| pkix_asn1_tab.c | ||
| qemuargv2xmltest.c | ||
| qemuhelptest.c | ||
| qemumonitortest.c | ||
| qemuxml2argvtest.c | ||
| qemuxml2xmltest.c | ||
| qemuxmlnstest.c | ||
| read-bufsiz | ||
| read-non-seekable | ||
| reconnect.c | ||
| schematestutils.sh | ||
| seclabeltest.c | ||
| sexpr2xmltest.c | ||
| shunloadhelper.c | ||
| shunloadtest.c | ||
| sockettest.c | ||
| ssh.c | ||
| start | ||
| statstest.c | ||
| storagepoolschematest | ||
| storagepoolxml2xmltest.c | ||
| storagevolschematest | ||
| storagevolxml2xmltest.c | ||
| test_conf.sh | ||
| test-lib.sh | ||
| testutils.c | ||
| testutils.h | ||
| testutilslxc.c | ||
| testutilslxc.h | ||
| testutilsqemu.c | ||
| testutilsqemu.h | ||
| testutilsxen.c | ||
| testutilsxen.h | ||
| undefine | ||
| utiltest.c | ||
| vcpupin | ||
| virauthconfigtest.c | ||
| virbuftest.c | ||
| virhashdata.h | ||
| virhashtest.c | ||
| virkeyfiletest.c | ||
| virnetmessagetest.c | ||
| virnetsockettest.c | ||
| virnettlscontexttest.c | ||
| virsh-all | ||
| virsh-optparse | ||
| virsh-schedinfo | ||
| virsh-synopsis | ||
| virshtest.c | ||
| virt-aa-helper-test | ||
| virtimetest.c | ||
| viruritest.c | ||
| vmx2xmltest.c | ||
| xencapstest.c | ||
| xmconfigtest.c | ||
| xml2sexprtest.c | ||
| xml2vmxtest.c | ||