diff options
| author |   Jacob Keller <jacob.e.keller@intel.com> | 2023-02-27 12:24:28 -0800 |
|---|---|---|
| committer |   Kees Cook <keescook@chromium.org> | 2024-02-29 13:38:01 -0800 |
| commit | e7549481255167dcdab355c539562c7ace17e111 (patch) | |
| tree | abc01d0800b8e65bc2bccad2be0aeda1981535ca /lib/string_helpers.c | |
| parent | coccinelle: Add rules to find str_plural() replacements (diff) | |
| download | wireguard-linux-e7549481255167dcdab355c539562c7ace17e111.tar.xz wireguard-linux-e7549481255167dcdab355c539562c7ace17e111.zip | |
coccinelle: semantic patch to check for potential struct_size calls
include/linux/overflow.h includes helper macros intended for calculating
sizes of allocations. These macros prevent accidental overflow by
saturating at SIZE_MAX.
In general when calculating such sizes use of the macros is preferred. Add
a semantic patch which can detect code patterns which can be replaced by
struct_size.
Note that I set the confidence to medium because this patch doesn't make an
attempt to ensure that the relevant array is actually a flexible array. The
struct_size macro does specifically require a flexible array. In many cases
the detected code could be refactored to a flexible array, but this is not
always possible (such as if there are multiple over-allocations).
Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>
Link: https://lore.kernel.org/r/20230227202428.3657443-1-jacob.e.keller@intel.com
Signed-off-by: Kees Cook <keescook@chromium.org>
Diffstat (limited to 'lib/string_helpers.c')
0 files changed, 0 insertions, 0 deletions