intel/decoders: Avoid uninitialized variable warnings
authorCaio Marcelo de Oliveira Filho <caio.oliveira@intel.com>
Fri, 23 Aug 2019 14:59:21 +0000 (07:59 -0700)
committerCaio Marcelo de Oliveira Filho <caio.oliveira@intel.com>
Fri, 23 Aug 2019 20:25:27 +0000 (13:25 -0700)
Initialize `next_batch_addr` and `second_level`.  If the batch is well
formed, those values will be overriden, if not, they are as good as
uninitialized garbage.

Acked-by: Eric Engestrom <eric.engestrom@intel.com>
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
src/intel/tools/aubinator_viewer_decoder.cpp

index f9586590221cc19d1b16ff9e0033b6d55aabf9b4..44038a9b226957d130c53f16d9b4136abf46387e 100644 (file)
@@ -955,9 +955,9 @@ aub_viewer_render_batch(struct aub_viewer_decode_ctx *ctx,
       }
 
       if (strcmp(inst_name, "MI_BATCH_BUFFER_START") == 0) {
-         uint64_t next_batch_addr;
+         uint64_t next_batch_addr = 0xd0d0d0d0;
          bool ppgtt = false;
-         bool second_level;
+         bool second_level = false;
          struct gen_field_iterator iter;
          gen_field_iterator_init(&iter, inst, p, 0, false);
          while (gen_field_iterator_next(&iter)) {