[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH for 2.1 06/14] qom: object: Ignore refs/unrefs of NU
From: |
Paolo Bonzini |
Subject: |
[Qemu-devel] [PATCH for 2.1 06/14] qom: object: Ignore refs/unrefs of NULL |
Date: |
Wed, 25 Jun 2014 14:25:50 +0200 |
From: Peter Crosthwaite <address@hidden>
Just do nothing if passed NULL for a ref or unref. This avoids
call sites that manage a combination of NULL or non-NULL pointers
having to add iffery around every ref and unref.
Signed-off-by: Peter Crosthwaite <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>
---
qom/object.c | 14 ++++++++------
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/qom/object.c b/qom/object.c
index a3b8cf3..31bf8b9 100644
--- a/qom/object.c
+++ b/qom/object.c
@@ -711,11 +711,17 @@ GSList *object_class_get_list(const char *implements_type,
void object_ref(Object *obj)
{
+ if (!obj) {
+ return;
+ }
atomic_inc(&obj->ref);
}
void object_unref(Object *obj)
{
+ if (!obj) {
+ return;
+ }
g_assert(obj->ref > 0);
/* parent always holds a reference to its children */
@@ -1160,13 +1166,9 @@ static void object_set_link_property(Object *obj,
Visitor *v, void *opaque,
return;
}
- if (new_target) {
- object_ref(new_target);
- }
+ object_ref(new_target);
*child = new_target;
- if (old_target != NULL) {
- object_unref(old_target);
- }
+ object_unref(old_target);
}
static Object *object_resolve_link_property(Object *parent, void *opaque,
const gchar *part)
--
1.8.3.1
- [Qemu-devel] [PATCH for 2.1 00/14] My pending qom and memory QOMification patches, Paolo Bonzini, 2014/06/25
- [Qemu-devel] [PATCH for 2.1 01/14] qom: add object_property_add_alias(), Paolo Bonzini, 2014/06/25
- [Qemu-devel] [PATCH for 2.1 03/14] qom: allow creating an alias of a child<> property, Paolo Bonzini, 2014/06/25
- [Qemu-devel] [PATCH for 2.1 02/14] qom: add a generic mechanism to resolve paths, Paolo Bonzini, 2014/06/25
- [Qemu-devel] [PATCH for 2.1 05/14] qom: object: remove parent pointer when unparenting, Paolo Bonzini, 2014/06/25
- [Qemu-devel] [PATCH for 2.1 04/14] mc146818rtc: add "rtc-time" link to "/machine/rtc", Paolo Bonzini, 2014/06/25
- [Qemu-devel] [PATCH for 2.1 06/14] qom: object: Ignore refs/unrefs of NULL,
Paolo Bonzini <=
- [Qemu-devel] [PATCH for 2.1 08/14] memory: MemoryRegion: use /machine as default owner, Paolo Bonzini, 2014/06/25
- [Qemu-devel] [PATCH for 2.1 07/14] libqtest: escape strings in QMP commands, fix leak, Paolo Bonzini, 2014/06/25
- [Qemu-devel] [PATCH for 2.1 10/14] memory: MemoryRegion: replace owner field with QOM parent, Paolo Bonzini, 2014/06/25
- [Qemu-devel] [PATCH for 2.1 09/14] memory: MemoryRegion: QOMify, Paolo Bonzini, 2014/06/25
- [Qemu-devel] [PATCH for 2.1 12/14] memory: MemoryRegion: Add may-overlap and priority props, Paolo Bonzini, 2014/06/25
- [Qemu-devel] [PATCH for 2.1 11/14] memory: MemoryRegion: Add container and addr props, Paolo Bonzini, 2014/06/25
- [Qemu-devel] [PATCH for 2.1 13/14] memory: MemoryRegion: Add size property, Paolo Bonzini, 2014/06/25
- [Qemu-devel] [PATCH for 2.1 14/14] memory: do not give a name to the internal exec.c regions, Paolo Bonzini, 2014/06/25