MODULE_CONTEXT() now avoids directly returning NULL to make static analyzer happier.

--HG--
branch : HEAD

diff --git a/src/lib/module-context.h b/src/lib/module-context.h
index 2603cf06f71b3ae9e8fd49561f74ade6f041951c..5c0738a653674bb80d46e2dcda53818fdf19109c 100644 (file)
--- a/src/lib/module-context.h
+++ b/src/lib/module-context.h
@@ -46,10 +46,9 @@
        COMPILE_ERROR_IF_TYPES_NOT_COMPATIBLE(OBJ_REGISTER(obj), (id_ctx).reg)
 
 #define MODULE_CONTEXT(obj, id_ctx) \
-       (unlikely(!(id_ctx).id.module_id_set) ? NULL : \
        (*((void **)array_idx_modifiable(&(obj)->module_contexts, \
-                                       (id_ctx).id.module_id) + \
-        OBJ_REGISTER_COMPATIBLE(obj, id_ctx))))
+               module_get_context_id(&(id_ctx).id)) + \
+        OBJ_REGISTER_COMPATIBLE(obj, id_ctx)))
 
 #ifdef HAVE_TYPEOF
 #  define MODULE_CONTEXT_DEFINE(_name, _reg) \