[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH qom-next] qom: make object cast assert if NULL o
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH qom-next] qom: make object cast assert if NULL object is passed as argument |
Date: |
Wed, 30 May 2012 18:24:49 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:12.0) Gecko/20120421 Thunderbird/12.0 |
Am 30.05.2012 18:19, schrieb Igor Mammedov:
> without assert it will crash at following point:
> OBJECT_CHECK(type, obj, name) \
> ((type *)object_dynamic_cast_assert(OBJECT(obj), (name)))
> => object_dynamic_cast(obj, typename)
> => object_is_type(obj, target_type)
> => type_is_ancestor(obj->class->type, target_type);
> ^^^
> so abort earlier and print nice message instead of SIGSEGV
>
> Signed-off-by: Igor Mammedov <address@hidden>
Acked-by: Andreas Färber <address@hidden>
If nobody objects, I'll queue it on qom-next.
Andreas
> ---
> qom/object.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/qom/object.c b/qom/object.c
> index 00bb3b0..444e2fc 100644
> --- a/qom/object.c
> +++ b/qom/object.c
> @@ -481,6 +481,8 @@ Object *object_dynamic_cast_assert(Object *obj, const
> char *typename)
> {
> Object *inst;
>
> + g_assert(obj != NULL);
> +
> inst = object_dynamic_cast(obj, typename);
>
> if (!inst) {
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg