summaryrefslogtreecommitdiff
path: root/drivers/char
diff options
context:
space:
mode:
authoryang.yang29@zte.com.cn <yang.yang29@zte.com.cn>2022-12-05 14:36:40 +0300
committerCorey Minyard <cminyard@mvista.com>2022-12-05 15:50:09 +0300
commitc6f613e5f35b0e2154d5ca12f0e8e0be0c19be9a (patch)
tree35ca6ccdb94514fb2f87e37ffbf02b8a43ceb457 /drivers/char
parentb8fadb396414972c7e526f3f94c0b4d114d2b2a1 (diff)
downloadlinux-c6f613e5f35b0e2154d5ca12f0e8e0be0c19be9a.tar.xz
ipmi/watchdog: use strscpy() to instead of strncpy()
Xu Panda <xu.panda@zte.com.cn> The implementation of strscpy() is more robust and safer. That's now the recommended way to copy NUL terminated strings. Signed-off-by: Xu Panda <xu.panda@zte.com.cn> Signed-off-by: Yang Yang <yang.yang29@zte.com> Message-Id: <202212051936400309332@zte.com.cn> Signed-off-by: Corey Minyard <cminyard@mvista.com>
Diffstat (limited to 'drivers/char')
-rw-r--r--drivers/char/ipmi/ipmi_watchdog.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/char/ipmi/ipmi_watchdog.c b/drivers/char/ipmi/ipmi_watchdog.c
index 47365150e431..0d4a8dcacfd4 100644
--- a/drivers/char/ipmi/ipmi_watchdog.c
+++ b/drivers/char/ipmi/ipmi_watchdog.c
@@ -213,8 +213,7 @@ static int set_param_str(const char *val, const struct kernel_param *kp)
char valcp[16];
char *s;
- strncpy(valcp, val, 15);
- valcp[15] = '\0';
+ strscpy(valcp, val, 16);
s = strstrip(valcp);