Blame SOURCES/0017-dmidecode-Use-the-print-helpers-in-dump-mode-too.patch

f877e3
From da06888d08b9f1108fe89560c06d39675c10cd95 Mon Sep 17 00:00:00 2001
f877e3
From: Jean Delvare <jdelvare@suse.de>
f877e3
Date: Wed, 1 Apr 2020 10:00:30 +0200
f877e3
Subject: [PATCH 17/23] dmidecode: Use the print helpers in dump mode too
f877e3
f877e3
Signed-off-by: Jean Delvare <jdelvare@suse.de>
f877e3
---
f877e3
 dmidecode.c | 30 ++++++++++++++++++------------
f877e3
 1 file changed, 18 insertions(+), 12 deletions(-)
f877e3
f877e3
diff --git a/dmidecode.c b/dmidecode.c
f877e3
index 3d1da955bab8..5e9b9899ec1f 100644
f877e3
--- a/dmidecode.c
f877e3
+++ b/dmidecode.c
f877e3
@@ -220,43 +220,49 @@ static int dmi_bcd_range(u8 value, u8 low, u8 high)
f877e3
 	return 1;
f877e3
 }
f877e3
 
f877e3
-static void dmi_dump(const struct dmi_header *h, const char *prefix)
f877e3
+static void dmi_dump(const struct dmi_header *h)
f877e3
 {
f877e3
+	static char raw_data[48];
f877e3
 	int row, i;
f877e3
+	unsigned int off;
f877e3
 	char *s;
f877e3
 
f877e3
-	printf("%sHeader and Data:\n", prefix);
f877e3
+	pr_list_start("Header and Data", NULL);
f877e3
 	for (row = 0; row < ((h->length - 1) >> 4) + 1; row++)
f877e3
 	{
f877e3
-		printf("%s\t", prefix);
f877e3
+		off = 0;
f877e3
 		for (i = 0; i < 16 && i < h->length - (row << 4); i++)
f877e3
-			printf("%s%02X", i ? " " : "",
f877e3
+			off += sprintf(raw_data + off, i ? " %02X" : "%02X",
f877e3
 			       (h->data)[(row << 4) + i]);
f877e3
-		printf("\n");
f877e3
+		pr_list_item(raw_data);
f877e3
 	}
f877e3
+	pr_list_end();
f877e3
 
f877e3
 	if ((h->data)[h->length] || (h->data)[h->length + 1])
f877e3
 	{
f877e3
-		printf("%sStrings:\n", prefix);
f877e3
+		pr_list_start("Strings", NULL);
f877e3
 		i = 1;
f877e3
 		while ((s = _dmi_string(h, i++, !(opt.flags & FLAG_DUMP))))
f877e3
 		{
f877e3
 			if (opt.flags & FLAG_DUMP)
f877e3
 			{
f877e3
 				int j, l = strlen(s) + 1;
f877e3
+
f877e3
+				off = 0;
f877e3
 				for (row = 0; row < ((l - 1) >> 4) + 1; row++)
f877e3
 				{
f877e3
-					printf("%s\t", prefix);
f877e3
 					for (j = 0; j < 16 && j < l - (row << 4); j++)
f877e3
-						printf("%s%02X", j ? " " : "",
f877e3
+						off += sprintf(raw_data + off,
f877e3
+						       j ? " %02X" : "%02X",
f877e3
 						       (unsigned char)s[(row << 4) + j]);
f877e3
-					printf("\n");
f877e3
+					pr_list_item(raw_data);
f877e3
 				}
f877e3
 				/* String isn't filtered yet so do it now */
f877e3
 				ascii_filter(s, l - 1);
f877e3
 			}
f877e3
-			printf("%s\t%s\n", prefix, s);
f877e3
+			pr_list_item("%s", s);
f877e3
 		}
f877e3
+		pr_list_end();
f877e3
 	}
f877e3
 }
f877e3
 
f877e3
@@ -5060,7 +5066,7 @@ static void dmi_decode(const struct dmi_header *h, u16 ver)
f877e3
 				return;
f877e3
 			pr_handle_name("%s Type",
f877e3
 				h->type >= 128 ? "OEM-specific" : "Unknown");
f877e3
-			dmi_dump(h, "\t");
f877e3
+			dmi_dump(h);
f877e3
 	}
f877e3
 	printf("\n");
f877e3
 }
f877e3
@@ -5206,7 +5212,7 @@ static void dmi_table_decode(u8 *buf, u32 len, u16 num, u16 ver, u32 flags)
f877e3
 		{
f877e3
 			if (opt.flags & FLAG_DUMP)
f877e3
 			{
f877e3
-				dmi_dump(&h, "\t");
f877e3
+				dmi_dump(&h);
f877e3
 				printf("\n");
f877e3
 			}
f877e3
 			else
f877e3
-- 
f877e3
2.17.1
f877e3