[PATCH 4/4] lib/igt_debugfs: Only use valid values in igt_crc_to_str()

From: Liviu Dudau <liviu.dudau@arm.com>
To: Petri Latvala <petri.latvala@intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: [PATCH 4/4] lib/igt_debugfs: Only use valid values in igt_crc_to_str()
Date: Mon,  5 Jun 2017 14:28:40 +0100	[thread overview]
Message-ID: <20170605132840.9125-5-liviu.dudau@arm.com> (raw)
In-Reply-To: <20170605132840.9125-1-liviu.dudau@arm.com>

From: Brian Starkey <brian.starkey@arm.com>

Not all elements in the crc array may be valid, so only use the valid
ones to generate the string.

Signed-off-by: Brian Starkey <brian.starkey@arm.com>
---
 lib/igt_debugfs.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/lib/igt_debugfs.c b/lib/igt_debugfs.c
index f5ed3daf..80f25c61 100644
--- a/lib/igt_debugfs.c
+++ b/lib/igt_debugfs.c
@@ -312,10 +312,11 @@ void igt_assert_crc_equal(const igt_crc_t *a, const igt_crc_t *b)
  */
 char *igt_crc_to_string(igt_crc_t *crc)
 {
-	char buf[128];
+	int i;
+	char buf[128] = { 0 };
 
-	sprintf(buf, "%08x %08x %08x %08x %08x", crc->crc[0],
-		crc->crc[1], crc->crc[2], crc->crc[3], crc->crc[4]);
+	for (i = 0; i < crc->n_words; i++)
+		sprintf(buf + strlen(buf), "%08x ", crc->crc[i]);
 
 	return strdup(buf);
 }
-- 
2.13.0

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