This method cannot fail since it merely allocates a single struct, so the only possible failure (ENOMEM) will cause an abort() already.
Signed-off-by: Daniel P. Berrangé <berra...@redhat.com> --- migration/savevm.c | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/migration/savevm.c b/migration/savevm.c index c18b7e6033..6a7b930b1c 100644 --- a/migration/savevm.c +++ b/migration/savevm.c @@ -2861,10 +2861,7 @@ bool save_snapshot(const char *name, bool overwrite, const char *vmstate, /* save the VM state */ f = qemu_fopen_bdrv(bs, 1); - if (!f) { - error_setg(errp, "Could not open VM state file"); - goto the_end; - } + ret = qemu_savevm_state(f, errp); vm_state_size = qemu_ftell(f); ret2 = qemu_fclose(f); @@ -3041,10 +3038,6 @@ bool load_snapshot(const char *name, const char *vmstate, /* restore the VM state */ f = qemu_fopen_bdrv(bs_vm_state, 0); - if (!f) { - error_setg(errp, "Could not open VM state file"); - goto err_drain; - } qemu_system_reset(SHUTDOWN_CAUSE_NONE); mis->from_src_file = f; -- 2.29.2