dwfl: fix potential overflow when reporting on kernel modules
authorMatthias Maennich <maennich@google.com>
Thu, 18 Nov 2021 19:44:50 +0000 (19:44 +0000)
committerMark Wielaard <mark@klomp.org>
Sat, 20 Nov 2021 00:05:11 +0000 (01:05 +0100)
commit2e3bc18672f0cede0332ae3194eb2e33d4cc5fd7
tree5fc3768eedb8ffa01d33c5250bc44d9c860fa5e4
parent479d60d5a061b1b052d653c898b7b99f26cab3b8
dwfl: fix potential overflow when reporting on kernel modules

dwfl_linux_kernel_report_modules_ has an outstanding ancient bug when
reading kernel module information from a modules list file. The target
buffer for the module name was sized too small to hold potential values.
Fix that by increasing the value to account for the null termination.

In practice, this unlikely ever happened, but it now got diagnosed by
LLVM as part of a stricter -Wfortify-source implementation [1]:

libdwfl/linux-kernel-modules.c:1019:7: error: 'sscanf' may overflow; destination buffer in argument 3 has size 128, but the corresponding specifier may require size 129 [-Werror,-Wfortify-source]
                    modname, &modsz, &modaddr) == 3)

[1] https://github.com/llvm/llvm-project/commit/2db66f8d48beeea835cb9a6940e25bc04ab5d941

Suggested-by: Paul Pluzhnikov <ppluzhnikov@google.com>
Signed-off-by: Matthias Maennich <maennich@google.com>
libdwfl/ChangeLog
libdwfl/linux-kernel-modules.c