diff options
| author | Daniel Yang <danielyangkang@gmail.com> | 2024-11-13 10:20:58 +0100 |
|---|---|---|
| committer | Steffen Klassert <steffen.klassert@secunet.com> | 2024-11-14 11:38:37 +0100 |
| commit | 9e1a6db68e3ccc5c20fd2d6243285d1cc7215fe4 (patch) | |
| tree | e9ba27d7659b37d90ed3a1019a4f8e20850f9342 /net/xfrm | |
| parent | xfrm: Add error handling when nla_put_u32() returns an error (diff) | |
| download | linux-9e1a6db68e3ccc5c20fd2d6243285d1cc7215fe4.tar.xz linux-9e1a6db68e3ccc5c20fd2d6243285d1cc7215fe4.zip | |
xfrm: replace deprecated strncpy with strscpy_pad
The function strncpy is deprecated since it does not guarantee the
destination buffer is NULL terminated. Recommended replacement is
strscpy. The padded version was used to remain consistent with the other
strscpy_pad usage in the modified function.
Signed-off-by: Daniel Yang <danielyangkang@gmail.com>
Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>
Diffstat (limited to 'net/xfrm')
| -rw-r--r-- | net/xfrm/xfrm_user.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/xfrm/xfrm_user.c b/net/xfrm/xfrm_user.c index fab18b85af53..6b0800c7c75e 100644 --- a/net/xfrm/xfrm_user.c +++ b/net/xfrm/xfrm_user.c @@ -1101,7 +1101,7 @@ static int copy_to_user_auth(struct xfrm_algo_auth *auth, struct sk_buff *skb) if (!nla) return -EMSGSIZE; algo = nla_data(nla); - strncpy(algo->alg_name, auth->alg_name, sizeof(algo->alg_name)); + strscpy_pad(algo->alg_name, auth->alg_name, sizeof(algo->alg_name)); if (redact_secret && auth->alg_key_len) memset(algo->alg_key, 0, (auth->alg_key_len + 7) / 8); |