diff options
author | Alex Williamson <alex.williamson@redhat.com> | 2010-07-21 08:35:31 -0600 |
---|---|---|
committer | Aurelien Jarno <aurelien@aurel32.net> | 2010-07-30 23:02:03 +0200 |
commit | 8f6e28789faeac4f01f8dbfdac147a3d3b635f24 (patch) | |
tree | 1063ecad274ca00100103dd5e0c85c382c1e0cd6 /savevm.c | |
parent | e14aad448b9afcd7a2b249715768bf50c3b02034 (diff) |
savevm: Fix memory leak of compat struct
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>
(cherry picked from commit 69e58af92cf90a1a0551c73880928afa6753fa5f)
Diffstat (limited to 'savevm.c')
-rw-r--r-- | savevm.c | 6 |
1 files changed, 6 insertions, 0 deletions
@@ -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); } } |