[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v6 5/7] qdev: make get_pointer() handle temporary st
From: |
Stefan Hajnoczi |
Subject: |
[Qemu-devel] [PATCH v6 5/7] qdev: make get_pointer() handle temporary strings |
Date: |
Mon, 3 Mar 2014 11:30:06 +0100 |
From: Igor Mammedov <address@hidden>
get_pointer()'s print() callback might return a heap allocated
string, to avoid adding dedicated get_pointer_foo for this case
convert current print() callbacks to return temporary heap
allocated string and make get_pointer() free it.
Cc: "Andreas Färber" <address@hidden>
Signed-off-by: Igor Mammedov <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>
---
hw/core/qdev-properties-system.c | 19 +++++++++++--------
1 file changed, 11 insertions(+), 8 deletions(-)
diff --git a/hw/core/qdev-properties-system.c b/hw/core/qdev-properties-system.c
index 5f5957e..878c3bc 100644
--- a/hw/core/qdev-properties-system.c
+++ b/hw/core/qdev-properties-system.c
@@ -20,15 +20,16 @@
#include "sysemu/char.h"
static void get_pointer(Object *obj, Visitor *v, Property *prop,
- const char *(*print)(void *ptr),
+ char *(*print)(void *ptr),
const char *name, Error **errp)
{
DeviceState *dev = DEVICE(obj);
void **ptr = qdev_get_prop_ptr(dev, prop);
char *p;
- p = (char *) (*ptr ? print(*ptr) : "");
+ p = *ptr ? print(*ptr) : g_strdup("");
visit_type_str(v, &p, name, errp);
+ g_free(p);
}
static void set_pointer(Object *obj, Visitor *v, Property *prop,
@@ -91,9 +92,9 @@ static void release_drive(Object *obj, const char *name, void
*opaque)
}
}
-static const char *print_drive(void *ptr)
+static char *print_drive(void *ptr)
{
- return bdrv_get_device_name(ptr);
+ return g_strdup(bdrv_get_device_name(ptr));
}
static void get_drive(Object *obj, Visitor *v, void *opaque,
@@ -145,11 +146,12 @@ static void release_chr(Object *obj, const char *name,
void *opaque)
}
-static const char *print_chr(void *ptr)
+static char *print_chr(void *ptr)
{
CharDriverState *chr = ptr;
+ const char *val = chr->label ? chr->label : "";
- return chr->label ? chr->label : "";
+ return g_strdup(val);
}
static void get_chr(Object *obj, Visitor *v, void *opaque,
@@ -224,11 +226,12 @@ err:
return ret;
}
-static const char *print_netdev(void *ptr)
+static char *print_netdev(void *ptr)
{
NetClientState *netdev = ptr;
+ const char *val = netdev->name ? netdev->name : "";
- return netdev->name ? netdev->name : "";
+ return g_strdup(val);
}
static void get_netdev(Object *obj, Visitor *v, void *opaque,
--
1.8.5.3
- [Qemu-devel] [PATCH v6 0/7] dataplane: switch to N:M devices-per-thread model, Stefan Hajnoczi, 2014/03/03
- [Qemu-devel] [PATCH v6 1/7] object: add object_get_canonical_path_component(), Stefan Hajnoczi, 2014/03/03
- [Qemu-devel] [PATCH v6 2/7] rfifolock: add recursive FIFO lock, Stefan Hajnoczi, 2014/03/03
- [Qemu-devel] [PATCH v6 4/7] iothread: add I/O thread object, Stefan Hajnoczi, 2014/03/03
- [Qemu-devel] [PATCH v6 3/7] aio: add aio_context_acquire() and aio_context_release(), Stefan Hajnoczi, 2014/03/03
- [Qemu-devel] [PATCH v6 5/7] qdev: make get_pointer() handle temporary strings,
Stefan Hajnoczi <=
- [Qemu-devel] [PATCH v6 6/7] iothread: add "iothread" qdev property type, Stefan Hajnoczi, 2014/03/03
- [Qemu-devel] [PATCH v6 7/7] dataplane: replace internal thread with IOThread, Stefan Hajnoczi, 2014/03/03
- Re: [Qemu-devel] [PATCH v6 0/7] dataplane: switch to N:M devices-per-thread model, Stefan Hajnoczi, 2014/03/11