From: Caio Marcelo de Oliveira Filho <caio.oliveira@intel.com>
Date: Fri, 23 Aug 2019 14:59:21 +0000 (-0700)
Subject: intel/decoders: Avoid uninitialized variable warnings
X-Git-Tag: upstream/19.3.0~2539
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=412ed1338fc2b745e7b171282d51f88c7b7801c0;p=platform%2Fupstream%2Fmesa.git

intel/decoders: Avoid uninitialized variable warnings

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>
---

diff --git a/src/intel/tools/aubinator_viewer_decoder.cpp b/src/intel/tools/aubinator_viewer_decoder.cpp
index f958659..44038a9 100644
--- a/src/intel/tools/aubinator_viewer_decoder.cpp
+++ b/src/intel/tools/aubinator_viewer_decoder.cpp
@@ -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)) {