]> xenbits.xensource.com Git - libvirt.git/commitdiff
qemu: replace VIR_FREE with g_free in qemuFirmware*FreeContent()
authorLaine Stump <laine@redhat.com>
Thu, 4 Feb 2021 02:07:20 +0000 (21:07 -0500)
committerLaine Stump <laine@redhat.com>
Fri, 5 Feb 2021 05:22:09 +0000 (00:22 -0500)
These functions are all only called as a part of qemuFirmwareFree(),
which frees the qemuFirmware object before return, so we can be sure
none of the pointers is referenced after freeing (and thus there is no
need to clear any of them).

Signed-off-by: Laine Stump <laine@redhat.com>
Reviewed-by: Daniel Henrique Barboza <danielhb413@gmail.com>
src/qemu/qemu_firmware.c

index aad39ee038f2ff16ab00b395b4fb58f4abb3474c..e6541d505ed2916e9bad2deb8ec0ae2e05f32366 100644 (file)
@@ -190,8 +190,8 @@ G_DEFINE_AUTOPTR_CLEANUP_FUNC(qemuFirmwareOSInterface, qemuFirmwareOSInterfaceFr
 static void
 qemuFirmwareFlashFileFreeContent(qemuFirmwareFlashFilePtr flash)
 {
-    VIR_FREE(flash->filename);
-    VIR_FREE(flash->format);
+    g_free(flash->filename);
+    g_free(flash->format);
 }
 
 
@@ -206,14 +206,14 @@ qemuFirmwareMappingFlashFreeContent(qemuFirmwareMappingFlashPtr flash)
 static void
 qemuFirmwareMappingKernelFreeContent(qemuFirmwareMappingKernelPtr kernel)
 {
-    VIR_FREE(kernel->filename);
+    g_free(kernel->filename);
 }
 
 
 static void
 qemuFirmwareMappingMemoryFreeContent(qemuFirmwareMappingMemoryPtr memory)
 {
-    VIR_FREE(memory->filename);
+    g_free(memory->filename);
 }