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é <berrange@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