From 670cfb14fcf03ba25221c630d5237750cca52acb Mon Sep 17 00:00:00 2001 From: Marcel Hollerbach Date: Tue, 24 Mar 2020 15:45:22 +0100 Subject: [PATCH] eo: do not check this here as coverity points out, _obj_is_override is already dereferencing obj->opt so this was the wrong spot to check this. CID 1422014 CID 1422013 Reviewed-by: Stefan Schmidt Differential Revision: https://phab.enlightenment.org/D11579 --- src/lib/eo/eo.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/lib/eo/eo.c b/src/lib/eo/eo.c index 63a73fae51..2bdf284f5f 100644 --- a/src/lib/eo/eo.c +++ b/src/lib/eo/eo.c @@ -1255,10 +1255,9 @@ _eo_free(_Eo_Object *obj, Eina_Bool manual_free EINA_UNUSED) } } #endif - if (_obj_is_override(obj)) + if (obj->opt && _obj_is_override(obj)) { - if (obj->opt) - _vtable_free(obj->opt->vtable, &obj->klass->vtable); + _vtable_free(obj->opt->vtable, &obj->klass->vtable); EO_OPTIONAL_COW_SET(obj, vtable, NULL); }