From: Greg Kurz <[email protected]> The string returned by object_property_get_str() is dynamically allocated.
(Spotted by Coverity, CID 1375942) Signed-off-by: Greg Kurz <[email protected]> Signed-off-by: David Gibson <[email protected]> (cherry picked from commit 8a9e0e7b890b2598da94646bf6a7272f3d3924de) Signed-off-by: Michael Roth <[email protected]> --- hw/ppc/spapr.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c index 5564f78..58c15ef 100644 --- a/hw/ppc/spapr.c +++ b/hw/ppc/spapr.c @@ -2547,8 +2547,11 @@ static void spapr_memory_pre_plug(HotplugHandler *hotplug_dev, DeviceState *dev, if (mem_dev && !kvmppc_is_mem_backend_page_size_ok(mem_dev)) { error_setg(errp, "Memory backend has bad page size. " "Use 'memory-backend-file' with correct mem-path."); - return; + goto out; } + +out: + g_free(mem_dev); } typedef struct sPAPRDIMMState { -- 2.7.4
