We need to zero out the builtin reg spec we are parsing into.
Otherwise engine will be uninitialized and we segfault when trying
to find engine and accessing reg->engine in later stage.

Fixes: 7f0be0e7d9be ("tools/intel_reg: Add reading and writing registers 
through engine")
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=104895
Cc: Chris Wilson <ch...@chris-wilson.co.uk>
Cc: Marta Löfstedt <marta.lofst...@intel.com>
Cc: Jani Nikula <jani.nik...@intel.com>
Signed-off-by: Mika Kuoppala <mika.kuopp...@linux.intel.com>
---
 tools/intel_reg_decode.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tools/intel_reg_decode.c b/tools/intel_reg_decode.c
index 46b19676..b351dcb5 100644
--- a/tools/intel_reg_decode.c
+++ b/tools/intel_reg_decode.c
@@ -2748,6 +2748,8 @@ static ssize_t get_regs(struct reg **regs, size_t *nregs, 
ssize_t index,
                                &known_registers[i].regs[j];
                        struct reg reg;
 
+                       memset(&reg, 0, sizeof(reg));
+
                        /* XXX: Could be optimized. */
                        parse_port_desc(&reg, NULL);
 
-- 
2.14.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to