External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2025-03-07 17:28:15 +00:00
Code

Fix crash free'ing securityDriverNames in QEMU driver

Browse Source 
The previous fix to avoid leaking securityDriverNames forgot to
handle the case of securityDriverNames being NULL, leading to
a crash

Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
This commit is contained in:
Daniel P. Berrange 2013-01-24 18:36:04 +00:00
parent 2349d51fbe
commit bbc663b1c3
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/qemu/qemu_driver.c
View File

@ -1134,7 +1134,8 @@ qemuShutdown(void) {
VIR_FREE(qemu_driver->saveImageFormat); VIR_FREE(qemu_driver->saveImageFormat);
VIR_FREE(qemu_driver->dumpImageFormat); VIR_FREE(qemu_driver->dumpImageFormat);
for (i = 0 ; qemu_driver->securityDriverNames[i] != NULL ; i++) for (i = 0 ; (qemu_driver->securityDriverNames != NULL &&
qemu_driver->securityDriverNames[i] != NULL) ; i++)
VIR_FREE(qemu_driver->securityDriverNames[i]); VIR_FREE(qemu_driver->securityDriverNames[i]);
VIR_FREE(qemu_driver->securityDriverNames); VIR_FREE(qemu_driver->securityDriverNames);
virSecurityManagerFree(qemu_driver->securityManager); virSecurityManagerFree(qemu_driver->securityManager);