commit 6e3da9c81c2112e72829ed45ac34c0916de71a47
author Christian Gmeiner <christian.gmeiner@gmail.com> Wed Feb 08 13:07:25 2017 +0100
committer Emil Velikov <emil.l.velikov@gmail.com> Wed Mar 01 13:34:41 2017 +0000
tree 749400c0b7ffacda6ac4cb2a58c8f2fc96c873f3
parent 42b73955808d7b33cf8f65fc8b4abe53aeaba135

etnaviv: move pctx initialisation to avoid a null dereference

In case ctx->stream == NULL the fail label gets executed where
pctx gets dereferenced - too bad pctx is NULL in that case.

Caught by Coverity, reported to me by imirkin.

Cc: "17.0" <mesa-stable@lists.freedesktop.org>
Signed-off-by: Christian Gmeiner <christian.gmeiner@gmail.com>
Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com>
(cherry picked from commit a0b16a08905d68da07668a42eeb464b4f30bf3e5)

src/gallium/drivers/etnaviv/etnaviv_context.c

diff --git a/src/gallium/drivers/etnaviv/etnaviv_context.c b/src/gallium/drivers/etnaviv/etnaviv_context.c
index d767cd1..88580c1 100644
--- a/src/gallium/drivers/etnaviv/etnaviv_context.c
+++ b/src/gallium/drivers/etnaviv/etnaviv_context.c
@@ -251,20 +251,20 @@
 {
    struct etna_context *ctx = CALLOC_STRUCT(etna_context);
    struct etna_screen *screen;
-   struct pipe_context *pctx = NULL;
+   struct pipe_context *pctx;
 
    if (ctx == NULL)
       return NULL;
 
+   pctx = &ctx->base;
+   pctx->priv = ctx;
+   pctx->screen = pscreen;
+
    screen = etna_screen(pscreen);
    ctx->stream = etna_cmd_stream_new(screen->pipe, 0x2000, &etna_cmd_stream_reset_notify, ctx);
    if (ctx->stream == NULL)
       goto fail;
 
-   pctx = &ctx->base;
-   pctx->priv = ctx;
-   pctx->screen = pscreen;
-
    /* context ctxate setup */
    ctx->specs = screen->specs;
    ctx->screen = screen;