hw/ppc/spapr_caps.c: use g_autofree in spapr_cap_set_string()
And get rid of the 'out' label since it's now unused. Signed-off-by: Daniel Henrique Barboza <danielhb413@gmail.com> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> Message-Id: <20220228175004.8862-4-danielhb413@gmail.com> [ clg: Fixed typo in commit log ] Signed-off-by: Cédric Le Goater <clg@kaod.org>
This commit is contained in:
parent
5f2b96b38e
commit
ea8464fa27
@ -120,7 +120,7 @@ static void spapr_cap_set_string(Object *obj, Visitor *v, const char *name,
|
||||
SpaprCapabilityInfo *cap = opaque;
|
||||
SpaprMachineState *spapr = SPAPR_MACHINE(obj);
|
||||
uint8_t i;
|
||||
char *val;
|
||||
g_autofree char *val = NULL;
|
||||
|
||||
if (!visit_type_str(v, name, &val, errp)) {
|
||||
return;
|
||||
@ -128,20 +128,18 @@ static void spapr_cap_set_string(Object *obj, Visitor *v, const char *name,
|
||||
|
||||
if (!strcmp(val, "?")) {
|
||||
error_setg(errp, "%s", cap->possible->help);
|
||||
goto out;
|
||||
return;
|
||||
}
|
||||
for (i = 0; i < cap->possible->num; i++) {
|
||||
if (!strcasecmp(val, cap->possible->vals[i])) {
|
||||
spapr->cmd_line_caps[cap->index] = true;
|
||||
spapr->eff.caps[cap->index] = i;
|
||||
goto out;
|
||||
return;
|
||||
}
|
||||
}
|
||||
|
||||
error_setg(errp, "Invalid capability mode \"%s\" for cap-%s", val,
|
||||
cap->name);
|
||||
out:
|
||||
g_free(val);
|
||||
}
|
||||
|
||||
static void spapr_cap_get_pagesize(Object *obj, Visitor *v, const char *name,
|
||||
|
Loading…
Reference in New Issue
Block a user