[PATCH] qemu: free migration structure if migration fails

[PATCH] qemu: free migration structure if migration fails

[Jim Paris]

Without this, memory is leaked and later attempts fail with
"Migration already active".

Signed-off-by: Jim Paris <jim-XrPbb/hENzg@public.gmane.org>
---
 qemu/migration.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/qemu/migration.c b/qemu/migration.c
index bbeed86..ed1fd29 100644
--- a/qemu/migration.c
+++ b/qemu/migration.c
@@ -190,6 +190,8 @@ static void migrate_finish(MigrationState *s)
         save_verify_memory(f, NULL);
 #endif /* MIGRATION_VERIFY */
         qemu_fclose(f);
+    } else {
+	    migrate_close(s);
     }
     status = *has_error;
     if (ret && !status)
-- 
1.5.3.rc4


-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/

Re: [PATCH] qemu: free migration structure if migration fails

[Avi Kivity]

Jim Paris wrote:
> Without this, memory is leaked and later attempts fail with
> "Migration already active".
>   

Applied, thanks.


-- 
error compiling committee.c: too many arguments to function


-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/