[Intel-gfx] [PATCH i-g-t] lib/igt_debugfs: Prevent compiler warning on unchecked printf format

Gabriel Krisman Bertazi krisman at collabora.co.uk
Thu Jul 27 04:51:28 UTC 2017


Commit 34a54192e1fb ("lib/igt_debugfs: Add extended helper to format
crc to string") introduced the following compiler warning:

  igt_debugfs.c: In function ‘igt_crc_to_string_extended’:
  igt_debugfs.c:373:4: warning: format not a string literal, argument
  types not checked [-Wformat-nonliteral]
      i == (crc->n_words - 1) ? '\0' : delimiter);

This patch addresses the warning while also preventing a possible bad
memory access access if n_words is > 64.  I have no clue why we
care about the padding size (crc_size), but since this was added
recently, I'd rather leave it alone.

Signed-off-by: Gabriel Krisman Bertazi <krisman at collabora.co.uk>
---
 lib/igt_debugfs.c | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/lib/igt_debugfs.c b/lib/igt_debugfs.c
index 2aa335866066..ee1f0f544824 100644
--- a/lib/igt_debugfs.c
+++ b/lib/igt_debugfs.c
@@ -351,7 +351,7 @@ bool igt_check_crc_equal(const igt_crc_t *a, const igt_crc_t *b)
  * igt_crc_to_string_extended:
  * @crc: pipe CRC value to print
  * @delimiter: The delimiter to use between crc words
- * @crc_size: the number of bytes to print per crc word (either 4 or 2)
+ * @crc_size: the number of bytes to print per crc word (between 1 and 4)
  *
  * This function allocates a string and formats @crc into it, depending on
  * @delimiter and @crc_size.
@@ -362,16 +362,19 @@ bool igt_check_crc_equal(const igt_crc_t *a, const igt_crc_t *b)
 char *igt_crc_to_string_extended(igt_crc_t *crc, char delimiter, int crc_size)
 {
 	int i;
-	char *buf = calloc(128, sizeof(char));
-	const char *format[2] = { "%08x%c", "%04x%c" };
+	int len = 0;
+	int field_width = 2 * crc_size; /* Two chars per byte. */
+	char *buf = malloc((field_width+1) * crc->n_words * sizeof(char));
 
 	if (!buf)
 		return NULL;
 
 	for (i = 0; i < crc->n_words; i++)
-		sprintf(buf + strlen(buf), format[crc_size == 2], crc->crc[i],
-			i == (crc->n_words - 1) ? '\0' : delimiter);
+		len += sprintf(buf + len, "%0*x%c", field_width,
+			       crc->crc[i], delimiter);
 
+	/* Eat the last delimiter */
+	buf[len - 1] = '\0';
 	return buf;
 }
 
-- 
2.11.0



More information about the Intel-gfx mailing list