summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDaniel Henrique Barboza2020-08-25 23:57:47 +0200
committerDavid Gibson2020-09-08 02:08:42 +0200
commit90d282d0858cf5a38f3e8a7e201aeab2a0ccbe88 (patch)
tree425d2e9644843cfabd1efa5a7e0f47349d1791b8
parentspapr/xive: Allocate vCPU IPIs from the vCPU contexts (diff)
downloadqemu-90d282d0858cf5a38f3e8a7e201aeab2a0ccbe88.tar.gz
qemu-90d282d0858cf5a38f3e8a7e201aeab2a0ccbe88.tar.xz
qemu-90d282d0858cf5a38f3e8a7e201aeab2a0ccbe88.zip
ppc/spapr_nvdimm: use g_autofree in spapr_nvdimm_validate_opts()
Since we're using the string just once, just use g_autofree and avoid leaking it without calling g_free(). Signed-off-by: Daniel Henrique Barboza <danielhb413@gmail.com> Message-Id: <20200825215749.213536-2-danielhb413@gmail.com> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
-rw-r--r--hw/ppc/spapr_nvdimm.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/hw/ppc/spapr_nvdimm.c b/hw/ppc/spapr_nvdimm.c
index 81410aa63f..9a20a65640 100644
--- a/hw/ppc/spapr_nvdimm.c
+++ b/hw/ppc/spapr_nvdimm.c
@@ -33,7 +33,7 @@
void spapr_nvdimm_validate_opts(NVDIMMDevice *nvdimm, uint64_t size,
Error **errp)
{
- char *uuidstr = NULL;
+ g_autofree char *uuidstr = NULL;
QemuUUID uuid;
int ret;
@@ -54,7 +54,6 @@ void spapr_nvdimm_validate_opts(NVDIMMDevice *nvdimm, uint64_t size,
&error_abort);
ret = qemu_uuid_parse(uuidstr, &uuid);
g_assert(!ret);
- g_free(uuidstr);
if (qemu_uuid_is_null(&uuid)) {
error_setg(errp, "NVDIMM device requires the uuid to be set");