diff options
| author | Miguel Ojeda <miguel.ojeda.sandonis@gmail.com> | 2018-08-14 21:38:26 +0200 |
|---|---|---|
| committer | Miguel Ojeda <miguel.ojeda.sandonis@gmail.com> | 2018-09-30 20:14:04 +0200 |
| commit | 98cade0a08ba339cd11d6e89b0df5d1d2fa21202 (patch) | |
| tree | 942f0971ff424f94ec0e12b4f1e2f50f3cedebe8 /drivers | |
| parent | Compiler Attributes: enable -Wstringop-truncation on W=1 (gcc >= 8) (diff) | |
| download | linux-98cade0a08ba339cd11d6e89b0df5d1d2fa21202.tar.xz linux-98cade0a08ba339cd11d6e89b0df5d1d2fa21202.zip | |
Compiler Attributes: auxdisplay: panel: use __nonstring
Let gcc know these arrays are not meant to be NUL-terminated
by annotating them with the new __nonstring variable attribute;
and remove the comment since it conveys the same information.
Tested-by: Sedat Dilek <sedat.dilek@gmail.com> # on top of v4.19-rc5, clang 7
Reviewed-by: Nick Desaulniers <ndesaulniers@google.com>
Reviewed-by: Luc Van Oostenryck <luc.vanoostenryck@gmail.com>
Signed-off-by: Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/auxdisplay/panel.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/drivers/auxdisplay/panel.c b/drivers/auxdisplay/panel.c index 3b25a643058c..21b9b2f2470a 100644 --- a/drivers/auxdisplay/panel.c +++ b/drivers/auxdisplay/panel.c @@ -155,10 +155,9 @@ struct logical_input { int release_data; } std; struct { /* valid when type == INPUT_TYPE_KBD */ - /* strings can be non null-terminated */ - char press_str[sizeof(void *) + sizeof(int)]; - char repeat_str[sizeof(void *) + sizeof(int)]; - char release_str[sizeof(void *) + sizeof(int)]; + char press_str[sizeof(void *) + sizeof(int)] __nonstring; + char repeat_str[sizeof(void *) + sizeof(int)] __nonstring; + char release_str[sizeof(void *) + sizeof(int)] __nonstring; } kbd; } u; }; |