migration: fix leak of src file on dst
authorPeter Xu <peterx@redhat.com>
Fri, 19 May 2017 06:43:29 +0000 (14:43 +0800)
committerJuan Quintela <quintela@redhat.com>
Thu, 1 Jun 2017 16:48:58 +0000 (18:48 +0200)
The return path channel is possibly leaked. Fix it.

Signed-off-by: Peter Xu <peterx@redhat.com>
Reviewed-by: Juan Quintela <quintela@redhat.com>
Signed-off-by: Juan Quintela <quintela@redhat.com>
migration/migration.c

index c3218cd4b604dc1b829241f999b222d6e1a6ee5a..b90e39992e61802a0af7d9c88581e373a0c46afe 100644 (file)
@@ -133,6 +133,11 @@ void migration_incoming_state_destroy(void)
 {
     struct MigrationIncomingState *mis = migration_incoming_get_current();
 
+    if (mis->to_src_file) {
+        qemu_fclose(mis->to_src_file);
+        mis->to_src_file = NULL;
+    }
+
     qemu_event_destroy(&mis->main_thread_load_event);
 }