mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2025-02-22 03:12:22 +00:00
qemu: fix build error with older platforms
Jim Fehlig reported on IRC that older gcc/glibc triggers this warning: cc1: warnings being treated as errors qemu/qemu_domain.c: In function 'qemuDomainDefFormatBuf': qemu/qemu_domain.c:1297: error: declaration of 'remove' shadows a global declaration [-Wshadow] /usr/include/stdio.h:157: error: shadowed declaration is here [-Wshadow] make[3]: *** [libvirt_driver_qemu_impl_la-qemu_domain.lo] Error 1 Fix it like we have done in the past (such as commit 2e6322a). * src/qemu/qemu_domain.c (qemuDomainDefFormatBuf): Avoid shadowing a function name. Signed-off-by: Eric Blake <eblake@redhat.com>
This commit is contained in:
parent
caf659a868
commit
b121584f58
@ -1294,7 +1294,7 @@ qemuDomainDefFormatBuf(virQEMUDriverPtr driver,
|
||||
|
||||
if ((flags & VIR_DOMAIN_XML_MIGRATABLE)) {
|
||||
int i;
|
||||
int remove = 0;
|
||||
int toremove = 0;
|
||||
virDomainControllerDefPtr usb = NULL, pci = NULL;
|
||||
|
||||
/* If only the default USB controller is present, we can remove it
|
||||
@ -1314,7 +1314,7 @@ qemuDomainDefFormatBuf(virQEMUDriverPtr driver,
|
||||
if (usb && usb->idx == 0 && usb->model == -1) {
|
||||
VIR_DEBUG("Removing default USB controller from domain '%s'"
|
||||
" for migration compatibility", def->name);
|
||||
remove++;
|
||||
toremove++;
|
||||
} else {
|
||||
usb = NULL;
|
||||
}
|
||||
@ -1335,15 +1335,15 @@ qemuDomainDefFormatBuf(virQEMUDriverPtr driver,
|
||||
pci->model == VIR_DOMAIN_CONTROLLER_MODEL_PCI_ROOT) {
|
||||
VIR_DEBUG("Removing default 'pci-root' from domain '%s'"
|
||||
" for migration compatibility", def->name);
|
||||
remove++;
|
||||
toremove++;
|
||||
} else {
|
||||
pci = NULL;
|
||||
}
|
||||
|
||||
if (remove) {
|
||||
if (toremove) {
|
||||
controllers = def->controllers;
|
||||
ncontrollers = def->ncontrollers;
|
||||
if (VIR_ALLOC_N(def->controllers, ncontrollers - remove) < 0) {
|
||||
if (VIR_ALLOC_N(def->controllers, ncontrollers - toremove) < 0) {
|
||||
controllers = NULL;
|
||||
virReportOOMError();
|
||||
goto cleanup;
|
||||
|
Loading…
x
Reference in New Issue
Block a user