49 lines
1.5 KiB
Diff
49 lines
1.5 KiB
Diff
|
From 0538387bbdb426de88b54a8c45153c5644209698 Mon Sep 17 00:00:00 2001
|
||
|
|
From: Mark McLoughlin <markmc@redhat.com>
|
||
|
|
Date: Thu, 17 Sep 2009 15:32:45 +0100
|
||
|
|
Subject: [PATCH] Fix leak in PCI hostdev hot-unplug
|
||
|
|
|
||
|
|
* src/qemu_driver.c: sync the hostdev hot-unplug code with the disk/net
|
||
|
|
code.
|
||
|
|
|
||
|
|
Fedora-patch: libvirt-fix-pci-hostdev-hotunplug-leak.patch
|
||
|
|
---
|
||
|
|
src/qemu_driver.c | 20 +++++++++++++-------
|
||
|
|
1 files changed, 13 insertions(+), 7 deletions(-)
|
||
|
|
|
||
|
|
diff --git a/src/qemu_driver.c b/src/qemu_driver.c
|
||
|
|
index de31581..2ddcdc0 100644
|
||
|
|
--- a/src/qemu_driver.c
|
||
|
|
+++ b/src/qemu_driver.c
|
||
|
|
@@ -6206,14 +6206,20 @@ static int qemudDomainDetachHostPciDevice(virConnectPtr conn,
|
||
|
|
pciFreeDevice(conn, pci);
|
||
|
|
}
|
||
|
|
|
||
|
|
- if (i != --vm->def->nhostdevs)
|
||
|
|
- memmove(&vm->def->hostdevs[i],
|
||
|
|
- &vm->def->hostdevs[i+1],
|
||
|
|
- sizeof(*vm->def->hostdevs) * (vm->def->nhostdevs-i));
|
||
|
|
- if (VIR_REALLOC_N(vm->def->hostdevs, vm->def->nhostdevs) < 0) {
|
||
|
|
- virReportOOMError(conn);
|
||
|
|
- ret = -1;
|
||
|
|
+ if (vm->def->nhostdevs > 1) {
|
||
|
|
+ memmove(vm->def->hostdevs + i,
|
||
|
|
+ vm->def->hostdevs + i + 1,
|
||
|
|
+ sizeof(*vm->def->hostdevs) *
|
||
|
|
+ (vm->def->nhostdevs - (i + 1)));
|
||
|
|
+ vm->def->nhostdevs--;
|
||
|
|
+ if (VIR_REALLOC_N(vm->def->hostdevs, vm->def->nhostdevs) < 0) {
|
||
|
|
+ /* ignore, harmless */
|
||
|
|
+ }
|
||
|
|
+ } else {
|
||
|
|
+ VIR_FREE(vm->def->hostdevs);
|
||
|
|
+ vm->def->nhostdevs = 0;
|
||
|
|
}
|
||
|
|
+ virDomainHostdevDefFree(detach);
|
||
|
|
|
||
|
|
return ret;
|
||
|
|
}
|
||
|
|
--
|
||
|
|
1.6.2.5
|
||
|
|
|