| Message ID | 1592192317-10566-1-git-send-email-zhangfei.gao@linaro.org |
|---|---|
| State | Accepted |
| Commit | 6ec5e8b5e7148c3ece1850dffd62a493dbfc2d22 |
| Headers | show |
| Series | [v2] crypto: hisilicon - fix strncpy warning with strscpy | expand |
On Mon, Jun 15, 2020 at 11:38:37AM +0800, Zhangfei Gao wrote: > Use strscpy to fix the warning > warning: 'strncpy' specified bound 64 equals destination size > > Reported-by: kernel test robot <lkp@intel.com> > Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org> > --- > v2: Use strscpy instead of strlcpy since better truncation handling > suggested by Herbert > Rebase to 5.8-rc1 > > drivers/crypto/hisilicon/qm.c | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) Patch applied. Thanks. -- Email: Herbert Xu <herbert@gondor.apana.org.au> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
diff --git a/drivers/crypto/hisilicon/qm.c b/drivers/crypto/hisilicon/qm.c index 9bb263cec6c3..8ac293afa8ab 100644 --- a/drivers/crypto/hisilicon/qm.c +++ b/drivers/crypto/hisilicon/qm.c @@ -2179,8 +2179,12 @@ static int qm_alloc_uacce(struct hisi_qm *qm) .flags = UACCE_DEV_SVA, .ops = &uacce_qm_ops, }; + int ret; - strncpy(interface.name, pdev->driver->name, sizeof(interface.name)); + ret = strscpy(interface.name, pdev->driver->name, + sizeof(interface.name)); + if (ret < 0) + return -ENAMETOOLONG; uacce = uacce_alloc(&pdev->dev, &interface); if (IS_ERR(uacce))
Use strscpy to fix the warning warning: 'strncpy' specified bound 64 equals destination size Reported-by: kernel test robot <lkp@intel.com> Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org> --- v2: Use strscpy instead of strlcpy since better truncation handling suggested by Herbert Rebase to 5.8-rc1 drivers/crypto/hisilicon/qm.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) -- 2.23.0