locking/lockdep: Fix string sizing bug that triggers a format-truncation compiler...
authorLucy Mielke <lucymielke@icloud.com>
Thu, 12 Oct 2023 10:44:32 +0000 (12:44 +0200)
committerIngo Molnar <mingo@kernel.org>
Thu, 12 Oct 2023 18:37:59 +0000 (20:37 +0200)
commitac8b60be078abebc3ab8836f3f0ecac6980e0b4f
tree5813336a949675ed3db3ddebc10e5c137bd1034f
parente6115c6f7a0ce3388cc60b69a284facf78b5dbfd
locking/lockdep: Fix string sizing bug that triggers a format-truncation compiler-warning

On an allyesconfig, with "treat warnings as errors" unset, GCC emits
these warnings:

kernel/locking/lockdep_proc.c:438:32: Warning: Format specifier '%lld' may
be truncated when writing 1 to 17 bytes into a region
of size 15 [-Wformat-truncation=]

kernel/locking/lockdep_proc.c:438:31: Note: Format directive argument is
in the range [-92233720368547759223372036854775]

kernel/locking/lockdep_proc.c:438:9: Note: 'snprintf' has output
between 5 and 22 bytes into a target of size 15

In seq_time(), the longest s64 is "-9223372036854775808"-ish, which
converted to the fixed-point float format is "-9223372036854775.80": 21 bytes,
plus termination is another byte: 22. Therefore, a larger buffer size
of 22 is needed here - not 15. The code was safe due to the snprintf().

Fix it.

Signed-off-by: Lucy Mielke <lucymielke@icloud.com>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Link: https://lore.kernel.org/r/ZSfOEHRkZAWaQr3U@fedora.fritz.box
kernel/locking/lockdep_proc.c