After previous commits, the cleanup label shrank to plain
'return' statement. There's no point in having such label, so
drop it.
Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
bool reset_nvram)
{
g_autoptr(virQEMUDriverConfig) cfg = virQEMUDriverGetConfig(driver);
- int ret = -1;
VIR_AUTOCLOSE srcFD = -1;
virDomainLoaderDef *loader = vm->def->os.loader;
const char *master_nvram_path;
virReportError(VIR_ERR_OPERATION_FAILED,
_("unable to find any master var store for "
"loader: %s"), loader->path);
- goto cleanup;
+ return -1;
}
if ((srcFD = virFileOpenAs(master_nvram_path, O_RDONLY,
virReportSystemError(-srcFD,
_("Failed to open file '%s'"),
master_nvram_path);
- goto cleanup;
+ return -1;
}
data.srcFD = srcFD;
cfg->user, cfg->group,
qemuPrepareNVRAMHelper,
&data) < 0) {
- goto cleanup;
+ return -1;
}
- ret = 0;
- cleanup:
- return ret;
+ return 0;
}