diff options
| author | yang.yang29@zte.com.cn <yang.yang29@zte.com.cn> | 2022-12-05 12:36:40 +0100 |
|---|---|---|
| committer | Corey Minyard <cminyard@mvista.com> | 2022-12-05 13:50:09 +0100 |
| commit | c6f613e5f35b0e2154d5ca12f0e8e0be0c19be9a (patch) | |
| tree | 35ca6ccdb94514fb2f87e37ffbf02b8a43ceb457 /drivers/char/ipmi/ipmi_watchdog.c | |
| parent | ipmi: ssif_bmc: Convert to i2c's .probe_new() (diff) | |
| download | linux-c6f613e5f35b0e2154d5ca12f0e8e0be0c19be9a.tar.xz linux-c6f613e5f35b0e2154d5ca12f0e8e0be0c19be9a.zip | |
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/ipmi/ipmi_watchdog.c')
| -rw-r--r-- | drivers/char/ipmi/ipmi_watchdog.c | 3 |
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); |