conf: Don't free the constructed string in virDomainGetBlkioParametersAssignFromDef

virTypedParameterAssign steals the string rather than copying it into
the typed parameter and thus freeing it leads to a crash when attempting
to serialize the results.

This was introduced in commit 9f50f6e2 and later made an universal
helper in 32e6339c.

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1351473
This commit is contained in:
Peter Krempa 2016-06-30 14:33:24 +02:00
parent d7c40d50d7
commit cbe4c049d8

View File

@ -24936,7 +24936,7 @@ virDomainGetBlkioParametersAssignFromDef(virDomainDefPtr def,
if (virTypedParameterAssign(&(params[(*nparams)++]), name, \
VIR_TYPED_PARAM_STRING, data) < 0) \
goto error; \
VIR_FREE(data); \
data = NULL; \
}
/* blkiotune.device_weight */