[PULL 24/39] migration: set file error on subsection loading

Juan Quintela quintela at redhat.com
Tue Oct 24 13:12:50 UTC 2023


From: Marc-André Lureau <marcandre.lureau at redhat.com>

commit 13cde50889237 ("vmstate: Return error in case of error") sets
QemuFile error to stop reading from it and report to the caller (checked
by unit tests). We should do the same on subsection loading error.

Signed-off-by: Marc-André Lureau <marcandre.lureau at redhat.com>
Reviewed-by: Juan Quintela <quintela at redhat.com>
Signed-off-by: Juan Quintela <quintela at redhat.com>
Message-ID: <20231024084043.2926316-8-marcandre.lureau at redhat.com>
---
 migration/vmstate.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/migration/vmstate.c b/migration/vmstate.c
index 16e33a5d34..9c36803c8a 100644
--- a/migration/vmstate.c
+++ b/migration/vmstate.c
@@ -179,6 +179,7 @@ int vmstate_load_state(QEMUFile *f, const VMStateDescription *vmsd,
     assert(field->flags == VMS_END);
     ret = vmstate_subsection_load(f, vmsd, opaque);
     if (ret != 0) {
+        qemu_file_set_error(f, ret);
         return ret;
     }
     if (vmsd->post_load) {
-- 
2.41.0



More information about the libvir-list mailing list