michaelh/zephyr
1
0
Fork 0
Code Releases Activity

logging: use unsigned print format specifier

Browse source 
Even though it's highly unlikely that a component of time
would ever approach INT_MAX, use the unsigned specifier to mitigate
any unexpected behaviour.

Fixes #36814

Signed-off-by: Christopher Friedt <chrisfriedt@gmail.com>
This commit is contained in:
Christopher Friedt 2021-07-12 10:52:42 -04:00
commit 8e9d055d49
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
subsys/logging/log_output.c
View file

@ -206,16 +206,16 @@ static int timestamp_print(const struct log_output *output,
strftime(time_str, sizeof(time_str), "%FT%T", tm);
length = print_formatted(output, "%s.%06dZ ",
length = print_formatted(output, "%s.%06uZ ",
time_str, ms * 1000U + us);
#else
length = print_formatted(output,
"1970-01-01T%02d:%02d:%02d.%06dZ ",
"1970-01-01T%02u:%02u:%02u.%06uZ ",
hours, mins, seconds, ms * 1000U + us);
#endif
} else {
length = print_formatted(output,
"[%02d:%02d:%02d.%03d,%03d] ",
"[%02u:%02u:%02u.%03u,%03u] ",
hours, mins, seconds, ms, us);
}
} else {