From 2e3bf0613977ab0ae859cd7e7cceeedc3abf0c52 Mon Sep 17 00:00:00 2001 From: Andy Green Date: Tue, 19 Jan 2016 21:29:34 +0800 Subject: [PATCH] coverity 156861 context destroy dereference context before NULL check Context could be NULL only if context creation failed in the first place and user error path is bad... no network connectivity at that point... Signed-off-by: Andy Green --- lib/context.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/context.c b/lib/context.c index 02b3f1fa..26aac353 100644 --- a/lib/context.c +++ b/lib/context.c @@ -326,13 +326,14 @@ lws_context_destroy(struct lws_context *context) { const struct lws_protocols *protocol = NULL; struct lws wsi; - int n, m = context->count_threads; + int n, m; lwsl_notice("%s\n", __func__); if (!context) return; + m = context->count_threads; context->being_destroyed = 1; memset(&wsi, 0, sizeof(wsi));