From ea436445ad774179d6b3196e102907272f403da8 Mon Sep 17 00:00:00 2001 From: Emmanuel Roullit Date: Wed, 3 Apr 2013 21:07:32 +0200 Subject: [PATCH] Perform no operation on nl_object_free(NULL). Passing a NULL pointer would cause a NULL pointer dereference within nl_object_free(). Returning early on NULL pointer is the behavior free(3) and other nl*_free() functions. Signed-off-by: Emmanuel Roullit --- lib/object.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/lib/object.c b/lib/object.c index 17b98f6..b1ebe51 100644 --- a/lib/object.c +++ b/lib/object.c @@ -165,7 +165,12 @@ int nl_object_update(struct nl_object *dst, struct nl_object *src) */ void nl_object_free(struct nl_object *obj) { - struct nl_object_ops *ops = obj_ops(obj); + struct nl_object_ops *ops; + + if (!obj) + return; + + ops = obj_ops(obj); if (obj->ce_refcnt > 0) NL_DBG(1, "Warning: Freeing object in use...\n"); -- 2.40.0