commit 09e2bd70fd92c532a0bcb7eaebaacdaefdb2c0ce
author Stephanos Ioannidis <root@stephanos.io> Mon Jul 11 22:01:49 2022 +0900
committer Andrzej Puzdrowski <andrzej.puzdrowski@nordicsemi.no> Thu Aug 04 09:28:17 2022 +0200
tree fdcf4b7be0a58e95e3ca05e001b10e7bbfabe4e3
parent b4c04d3393dadfce36064326fd7f8baa849b80be

boot_serial: Fix incorrect format specifier for off_t

The `BOOT_LOG_INF` function, which uses the format specifiers defined
by the C standard, was incorrectly printing a variable with the type of
`off_t` using the `%x` format specifier, which is intended to be used
with the `int` type.

The `off_t` type, specified by the POSIX standard, is not guaranteed to
be `int`, and it may be defined as `long` or `long long` depending on
the toolchain and the target architecture.

This commit updates the print routine such that it casts the arguments
of the `off_t` type to `intmax_t` and prints them out using the
corresponding `%jx` format specifier.

Signed-off-by: Stephanos Ioannidis <root@stephanos.io>

boot/boot_serial/src/boot_serial.c

diff --git a/boot/boot_serial/src/boot_serial.c b/boot/boot_serial/src/boot_serial.c
index 17b798b..4ebff17 100644
--- a/boot/boot_serial/src/boot_serial.c
+++ b/boot/boot_serial/src/boot_serial.c
@@ -318,7 +318,8 @@
     }
 
     size = flash_sector_get_off(&sect) + flash_sector_get_size(&sect) - start;
-    BOOT_LOG_INF("Erasing range 0x%x:0x%x", start, start + size - 1);
+    BOOT_LOG_INF("Erasing range 0x%jx:0x%jx", (intmax_t)start,
+		 (intmax_t)(start + size - 1));
 
     rc = flash_area_erase(fap, start, size);
     if (rc != 0) {