]> git.proxmox.com Git - mirror_qemu.git/commitdiff
savevm: Fix memory leak of compat struct
authorAlex Williamson <alex.williamson@redhat.com>
Wed, 21 Jul 2010 14:35:31 +0000 (08:35 -0600)
committerAurelien Jarno <aurelien@aurel32.net>
Fri, 30 Jul 2010 21:00:56 +0000 (23:00 +0200)
Forgot to check for and free these.

Found-by: Zachary Amsden <zamsden@redhat.com>
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
savevm.c

index 7a1de3c5323f22f24547a2c0c3b44313356d941b..4c0e5d3b102fbe59533040b5c664f54083da7fb7 100644 (file)
--- a/savevm.c
+++ b/savevm.c
@@ -1139,6 +1139,9 @@ void unregister_savevm(DeviceState *dev, const char *idstr, void *opaque)
     QTAILQ_FOREACH_SAFE(se, &savevm_handlers, entry, new_se) {
         if (strcmp(se->idstr, id) == 0 && se->opaque == opaque) {
             QTAILQ_REMOVE(&savevm_handlers, se, entry);
+            if (se->compat) {
+                qemu_free(se->compat);
+            }
             qemu_free(se);
         }
     }
@@ -1206,6 +1209,9 @@ void vmstate_unregister(DeviceState *dev, const VMStateDescription *vmsd,
     QTAILQ_FOREACH_SAFE(se, &savevm_handlers, entry, new_se) {
         if (se->vmsd == vmsd && se->opaque == opaque) {
             QTAILQ_REMOVE(&savevm_handlers, se, entry);
+            if (se->compat) {
+                qemu_free(se->compat);
+            }
             qemu_free(se);
         }
     }