[dmidecode] [PATCH] dmidecode: Fix crash with option -u

[Jean Delvare]

A segmentation fault was reported with option -u. Turns out to be a
stupid thinko where the buffer offset was reset at the wrong loop
depth.

Reported-by: Jerry Hoemann <jerry.hoemann@hpe.com>
Fixes: da06888d08b9 ("dmidecode: Use the print helpers in dump mode too")
Signed-off-by: Jean Delvare <jdelvare@suse.de>
---
Jerry, does that work for you?

 dmidecode.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- dmidecode.orig/dmidecode.c  2021-01-18 13:56:38.278222177 +0100
+++ dmidecode/dmidecode.c       2021-01-18 13:57:01.850502525 +0100
@@ -248,9 +248,9 @@ static void dmi_dump(const struct dmi_he
                        {
                                int j, l = strlen(s) + 1;
 
-                               off = 0;
                                for (row = 0; row < ((l - 1) >> 4) + 1; row++)
                                {
+                                       off = 0;
                                        for (j = 0; j < 16 && j < l - (row << 
4); j++)
                                                off += sprintf(raw_data + off,
                                                       j ? " %02X" : "%02X",


-- 
Jean Delvare
SUSE L3 Support