From nobody Sun Sep 14 22:18:29 2025 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3C7FFC00A5A for ; Wed, 18 Jan 2023 01:29:20 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229580AbjARB3S (ORCPT ); Tue, 17 Jan 2023 20:29:18 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49366 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229483AbjARB3O (ORCPT ); Tue, 17 Jan 2023 20:29:14 -0500 Received: from ssh248.corpemail.net (ssh248.corpemail.net [210.51.61.248]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 428AC3D937; Tue, 17 Jan 2023 17:29:13 -0800 (PST) Received: from ([60.208.111.195]) by ssh248.corpemail.net ((D)) with ASMTP (SSL) id NSV00010; Wed, 18 Jan 2023 09:29:10 +0800 Received: from localhost.localdomain (10.200.104.97) by jtjnmail201612.home.langchao.com (10.100.2.12) with Microsoft SMTP Server id 15.1.2507.16; Wed, 18 Jan 2023 09:29:10 +0800 From: Bo Liu To: , , , , CC: , , Bo Liu Subject: [PATCH] EDAC/mc_sysfs: use strscpy() to instead of strncpy() Date: Tue, 17 Jan 2023 20:29:06 -0500 Message-ID: <20230118012906.2376-1-liubo03@inspur.com> X-Mailer: git-send-email 2.18.2 MIME-Version: 1.0 X-Originating-IP: [10.200.104.97] tUid: 2023118092910b4e327ae6b9546c5955db50b9c2564f4 X-Abuse-Reports-To: service@corp-email.com Abuse-Reports-To: service@corp-email.com X-Complaints-To: service@corp-email.com X-Report-Abuse-To: service@corp-email.com Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" The implementation of strscpy() is more robust and safer. That's now the recommended way to copy NUL terminated strings. Signed-off-by: Bo Liu --- drivers/edac/edac_mc_sysfs.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/edac/edac_mc_sysfs.c b/drivers/edac/edac_mc_sysfs.c index 15f63452a9be..b709cbe8dcf9 100644 --- a/drivers/edac/edac_mc_sysfs.c +++ b/drivers/edac/edac_mc_sysfs.c @@ -229,8 +229,7 @@ static ssize_t channel_dimm_label_store(struct device *= dev, if (copy_count =3D=3D 0 || copy_count >=3D sizeof(rank->dimm->label)) return -EINVAL; =20 - strncpy(rank->dimm->label, data, copy_count); - rank->dimm->label[copy_count] =3D '\0'; + strscpy(rank->dimm->label, data, copy_count + 1); =20 return count; } @@ -535,8 +534,7 @@ static ssize_t dimmdev_label_store(struct device *dev, if (copy_count =3D=3D 0 || copy_count >=3D sizeof(dimm->label)) return -EINVAL; =20 - strncpy(dimm->label, data, copy_count); - dimm->label[copy_count] =3D '\0'; + strscpy(dimm->label, data, copy_count + 1); =20 return count; } --=20 2.27.0