coverity 156861 context destroy dereference context before NULL check
authorAndy Green <andy.green@linaro.org>
Tue, 19 Jan 2016 13:29:34 +0000 (21:29 +0800)
committerAndy Green <andy.green@linaro.org>
Tue, 19 Jan 2016 13:33:52 +0000 (21:33 +0800)
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 <andy.green@linaro.org>
lib/context.c

index 02b3f1f..26aac35 100644 (file)
@@ -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));