Le 04/02/2026 à 08:57, Sergei Heifetz a écrit :
Reorder the code so the assertion of mis->from_src_file occurs before
the call to migration_ioc_unregister_yank_from_file, which dereferences
it in qemu_file_get_ioc.

Fixes: 39675ffffb3394 ("migration: Move the yank unregister of channel_close 
out")
Signed-off-by: Sergei Heifetz <[email protected]>
---
  migration/savevm.c | 3 ++-
  1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/migration/savevm.c b/migration/savevm.c
index 3dc812a7bb..930a3391e3 100644
--- a/migration/savevm.c
+++ b/migration/savevm.c
@@ -2885,13 +2885,14 @@ static bool 
postcopy_pause_incoming(MigrationIncomingState *mis)
assert(migrate_postcopy_ram()); + assert(mis->from_src_file);
+
      /*
       * Unregister yank with either from/to src would work, since ioc behind it
       * is the same
       */
      migration_ioc_unregister_yank_from_file(mis->from_src_file);
- assert(mis->from_src_file);
      qemu_file_shutdown(mis->from_src_file);
      qemu_fclose(mis->from_src_file);
      mis->from_src_file = NULL;

Reviewed-by: Laurent Vivier <[email protected]>

Reply via email to