From: Greg Kurz <gr...@kaod.org> This is the only error path that needs to free the previously allocated ov1.
Reported-by: Coverity (CID 1421924) Fixes: cbd0d7f36322 "spapr: Fail CAS if option vector table cannot be parsed" Signed-off-by: Greg Kurz <gr...@kaod.org> Message-Id: <158481206205.336182.16106097429336044843.st...@bahia.lan> Signed-off-by: David Gibson <da...@gibson.dropbear.id.au> Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com> --- hw/ppc/spapr_hcall.c | 1 + 1 file changed, 1 insertion(+) diff --git a/hw/ppc/spapr_hcall.c b/hw/ppc/spapr_hcall.c index 40c86e91eb..0d50fc9117 100644 --- a/hw/ppc/spapr_hcall.c +++ b/hw/ppc/spapr_hcall.c @@ -1726,6 +1726,7 @@ static target_ulong h_client_architecture_support(PowerPCCPU *cpu, } ov5_guest = spapr_ovec_parse_vector(ov_table, 5); if (!ov5_guest) { + spapr_ovec_cleanup(ov1_guest); warn_report("guest didn't provide option vector 5"); return H_PARAMETER; } -- 2.25.1