From 7dcd597f476a97f8718a68df9d6d02a81e6dd524 Mon Sep 17 00:00:00 2001 From: Ivan Delalande Date: Wed, 21 Jun 2017 16:43:05 -0700 Subject: [PATCH] dmesg: print only 2 hex digits for each hex-escaped byte As buf is passed as a signed char buffer in fwrite_hex, fprintf will print every byte from 0x80 as a signed-extended int causing each of these bytes to be printed as "\xffffff80" and such, which can be pretty confusing. Force fprintf to use the argument as a char to make it print only 2 digits, e.g. "\x80". Signed-off-by: Ivan Delalande --- sys-utils/dmesg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sys-utils/dmesg.c b/sys-utils/dmesg.c index 3263c3a06d..6ddc48cbba 100644 --- a/sys-utils/dmesg.c +++ b/sys-utils/dmesg.c @@ -602,7 +602,7 @@ static int fwrite_hex(const char *buf, size_t size, FILE *out) size_t i; for (i = 0; i < size; i++) { - int rc = fprintf(out, "\\x%02x", buf[i]); + int rc = fprintf(out, "\\x%02hhx", buf[i]); if (rc < 0) return rc; } -- 2.47.2