diff options
author | Kees Cook <keescook@chromium.org> | 2023-09-22 10:51:41 -0700 |
---|---|---|
committer | Helge Deller <deller@gmx.de> | 2023-09-23 20:35:44 +0200 |
commit | e34872523ca56b6a3ed7a5b06febdc66b4f16e3c (patch) | |
tree | a020c037e2a99e9ab1e3adf915b6671035c0fbdf /include/video | |
parent | 053d7dcd3440fa953ef4ca08de8e0a33d23d3b29 (diff) | |
download | lwn-e34872523ca56b6a3ed7a5b06febdc66b4f16e3c.tar.gz lwn-e34872523ca56b6a3ed7a5b06febdc66b4f16e3c.zip |
fbdev: mmp: Annotate struct mmp_path with __counted_by
Prepare for the coming implementation by GCC and Clang of the __counted_by
attribute. Flexible array members annotated with __counted_by can have
their accesses bounds-checked at run-time checking via CONFIG_UBSAN_BOUNDS
(for array indexing) and CONFIG_FORTIFY_SOURCE (for strcpy/memcpy-family
functions).
As found with Coccinelle[1], add __counted_by for struct mmp_path.
[1] https://github.com/kees/kernel-tools/blob/trunk/coccinelle/examples/counted_by.cocci
Cc: Helge Deller <deller@gmx.de>
Cc: linux-fbdev@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Signed-off-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Helge Deller <deller@gmx.de>
Diffstat (limited to 'include/video')
-rw-r--r-- | include/video/mmp_disp.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/video/mmp_disp.h b/include/video/mmp_disp.h index 77252cb46361..a722dcbf5073 100644 --- a/include/video/mmp_disp.h +++ b/include/video/mmp_disp.h @@ -231,7 +231,7 @@ struct mmp_path { /* layers */ int overlay_num; - struct mmp_overlay overlays[]; + struct mmp_overlay overlays[] __counted_by(overlay_num); }; extern struct mmp_path *mmp_get_path(const char *name); |