nir/print: fix coverity error

Not something actually hit in real life (now state is never non-null,
but only case state->syms is null is if nir_print_instr() path).  But it
was something I overlooked the first time, so might as well fix it.

    *** CID 1324642:  Null pointer dereferences  (REVERSE_INULL)
    /src/glsl/nir/nir_print.c: 299 in print_var_decl()
    293
    294           fprintf(fp, " (%s, %u)", loc, var->data.driver_location);
    295        }
    296
    297        fprintf(fp, "\n");
    298
    >>>     CID 1324642:  Null pointer dereferences  (REVERSE_INULL)
    >>>     Null-checking "state" suggests that it may be null, but it has already been dereferenced on all paths leading to the check.
    299        if (state) {
    300           _mesa_set_add(state->syms, name);
    301           _mesa_hash_table_insert(state->ht, var, name);
    302        }
    303     }
    304

Signed-off-by: Rob Clark <robclark@freedesktop.org>

diff --git a/src/glsl/nir/nir_print.c b/src/glsl/nir/nir_print.c
index 6e86140ed9d6c7622da286797f8bae1d63f330b0..a19aa8b91326100f3a89813fb9295c4dcd26b5c3 100644 (file)
--- a/src/glsl/nir/nir_print.c
+++ b/src/glsl/nir/nir_print.c
@@ -296,7 +296,7 @@ print_var_decl(nir_variable *var, print_state *state)
 
    fprintf(fp, "\n");
 
-   if (state) {
+   if (state->syms) {
       _mesa_set_add(state->syms, name);
       _mesa_hash_table_insert(state->ht, var, name);
    }