Messages in this thread Patch in this message | | | From | Xiongfeng Wang <> | Subject | [PATCH] auxdisplay: use correct string length | Date | Mon, 8 Jan 2018 20:08:05 +0800 |
| |
From: Xiongfeng Wang <xiongfeng.wang@linaro.org>
gcc-8 reports
drivers/auxdisplay/panel.c: In function 'panel_attach': ./include/linux/string.h:245:9: warning: '__builtin_strncpy' specified bound 12 equals destination size [-Wstringop-truncation]
We need one less byte or call strlcpy() to make it a nul-terminated string.
Signed-off-by: Xiongfeng Wang <xiongfeng.wang@linaro.org> --- drivers/auxdisplay/panel.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/auxdisplay/panel.c b/drivers/auxdisplay/panel.c index ea7869c..f8344e4 100644 --- a/drivers/auxdisplay/panel.c +++ b/drivers/auxdisplay/panel.c @@ -1506,10 +1506,10 @@ static struct logical_input *panel_bind_key(const char *name, const char *press, key->rise_time = 1; key->fall_time = 1; - strncpy(key->u.kbd.press_str, press, sizeof(key->u.kbd.press_str)); - strncpy(key->u.kbd.repeat_str, repeat, sizeof(key->u.kbd.repeat_str)); + strncpy(key->u.kbd.press_str, press, sizeof(key->u.kbd.press_str) - 1); + strncpy(key->u.kbd.repeat_str, repeat, sizeof(key->u.kbd.repeat_str) - 1); strncpy(key->u.kbd.release_str, release, - sizeof(key->u.kbd.release_str)); + sizeof(key->u.kbd.release_str - 1)); list_add(&key->list, &logical_inputs); return key; } -- 1.8.3.1
| |