|
From: | Juan Quintela |
Subject: | Re: [Qemu-devel] [PATCH RFC 3/6] migration: fix leak of src file on dst |
Date: | Wed, 31 May 2017 11:51:55 +0200 |
User-agent: | Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) |
Peter Xu <address@hidden> wrote: > The return path channel is possibly leaked. Fix it. > > Signed-off-by: Peter Xu <address@hidden> > --- > migration/migration.c | 5 +++++ > 1 file changed, 5 insertions(+) > Queued this one. Reviewed-by: Juan Quintela <address@hidden> > diff --git a/migration/migration.c b/migration/migration.c > index e4f4c8c..92617fc 100644 > --- a/migration/migration.c > +++ b/migration/migration.c > @@ -130,6 +130,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); > loadvm_free_handlers(mis); > }
[Prev in Thread] | Current Thread | [Next in Thread] |