From: Justin Stitt <justinstitt@google.com> To: Arnd Bergmann <arnd@kernel.org> Cc: linux-kernel@vger.kernel.org, Florian Fainelli <florian.fainelli@broadcom.com>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Arnd Bergmann <arnd@arndb.de>, Broadcom internal kernel review list <bcm-kernel-feedback-list@broadcom.com>, linux-rpi-kernel@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-staging@lists.linux.dev Subject: Re: [PATCH 01/11] staging: vc04_services: changen strncpy() to strscpy_pad() Date: Thu, 28 Mar 2024 23:10:44 +0000 [thread overview] Message-ID: <hbwxiid52lv2dq7rdtvard3up46khu5uv335tzavwag3v6s6jl@dzjrnooz553m> (raw) In-Reply-To: <20240328140512.4148825-2-arnd@kernel.org> Hi, On Thu, Mar 28, 2024 at 03:04:45PM +0100, Arnd Bergmann wrote: > From: Arnd Bergmann <arnd@arndb.de> > > gcc-14 warns about this strncpy() that results in a non-terminated > string for an overflow: > > In file included from include/linux/string.h:369, > from drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c:20: > In function 'strncpy', > inlined from 'create_component' at drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c:940:2: > include/linux/fortify-string.h:108:33: error: '__builtin_strncpy' specified bound 128 equals destination size [-Werror=stringop-truncation] > > Change it to strscpy_pad(), which produces a properly terminated and > zero-padded string. > > Signed-off-by: Arnd Bergmann <arnd@arndb.de> If there is other review that warrants a change, we might as well switch over to the new 2-argument version of strscpy{_pad}() introduced in Commit e6584c3964f2f ("string: Allow 2-argument strscpy()"). No need to change for only this reason, though. Reviewed-by: Justin Stitt <justinstitt@google.com> > --- > drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c b/drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c > index 258aa0e37f55..6ca5797aeae5 100644 > --- a/drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c > +++ b/drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c > @@ -937,8 +937,8 @@ static int create_component(struct vchiq_mmal_instance *instance, > /* build component create message */ > m.h.type = MMAL_MSG_TYPE_COMPONENT_CREATE; > m.u.component_create.client_component = component->client_component; > - strncpy(m.u.component_create.name, name, > - sizeof(m.u.component_create.name)); > + strscpy_pad(m.u.component_create.name, name, > + sizeof(m.u.component_create.name)); > > ret = send_synchronous_mmal_msg(instance, &m, > sizeof(m.u.component_create), > -- > 2.39.2 > Thanks Justin
WARNING: multiple messages have this Message-ID (diff)
From: Justin Stitt <justinstitt@google.com> To: Arnd Bergmann <arnd@kernel.org> Cc: linux-kernel@vger.kernel.org, Florian Fainelli <florian.fainelli@broadcom.com>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Arnd Bergmann <arnd@arndb.de>, Broadcom internal kernel review list <bcm-kernel-feedback-list@broadcom.com>, linux-rpi-kernel@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-staging@lists.linux.dev Subject: Re: [PATCH 01/11] staging: vc04_services: changen strncpy() to strscpy_pad() Date: Thu, 28 Mar 2024 23:10:44 +0000 [thread overview] Message-ID: <hbwxiid52lv2dq7rdtvard3up46khu5uv335tzavwag3v6s6jl@dzjrnooz553m> (raw) In-Reply-To: <20240328140512.4148825-2-arnd@kernel.org> Hi, On Thu, Mar 28, 2024 at 03:04:45PM +0100, Arnd Bergmann wrote: > From: Arnd Bergmann <arnd@arndb.de> > > gcc-14 warns about this strncpy() that results in a non-terminated > string for an overflow: > > In file included from include/linux/string.h:369, > from drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c:20: > In function 'strncpy', > inlined from 'create_component' at drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c:940:2: > include/linux/fortify-string.h:108:33: error: '__builtin_strncpy' specified bound 128 equals destination size [-Werror=stringop-truncation] > > Change it to strscpy_pad(), which produces a properly terminated and > zero-padded string. > > Signed-off-by: Arnd Bergmann <arnd@arndb.de> If there is other review that warrants a change, we might as well switch over to the new 2-argument version of strscpy{_pad}() introduced in Commit e6584c3964f2f ("string: Allow 2-argument strscpy()"). No need to change for only this reason, though. Reviewed-by: Justin Stitt <justinstitt@google.com> > --- > drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c b/drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c > index 258aa0e37f55..6ca5797aeae5 100644 > --- a/drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c > +++ b/drivers/staging/vc04_services/vchiq-mmal/mmal-vchiq.c > @@ -937,8 +937,8 @@ static int create_component(struct vchiq_mmal_instance *instance, > /* build component create message */ > m.h.type = MMAL_MSG_TYPE_COMPONENT_CREATE; > m.u.component_create.client_component = component->client_component; > - strncpy(m.u.component_create.name, name, > - sizeof(m.u.component_create.name)); > + strscpy_pad(m.u.component_create.name, name, > + sizeof(m.u.component_create.name)); > > ret = send_synchronous_mmal_msg(instance, &m, > sizeof(m.u.component_create), > -- > 2.39.2 > Thanks Justin _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2024-03-28 23:10 UTC|newest] Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top 2024-03-28 14:04 [PATCH 00/11] address remaining stringop-truncation warnings Arnd Bergmann 2024-03-28 14:04 ` Arnd Bergmann 2024-03-28 14:04 ` [PATCH 01/11] staging: vc04_services: changen strncpy() to strscpy_pad() Arnd Bergmann 2024-03-28 14:04 ` Arnd Bergmann 2024-03-28 14:42 ` Dan Carpenter 2024-03-28 14:42 ` Dan Carpenter 2024-03-28 16:15 ` Arnd Bergmann 2024-03-28 16:15 ` Arnd Bergmann 2024-03-28 23:10 ` Justin Stitt [this message] 2024-03-28 23:10 ` Justin Stitt 2024-03-28 14:04 ` [PATCH 02/11] scsi: devinfo: rework scsi_strcpy_devinfo() Arnd Bergmann 2024-03-28 16:46 ` Bart Van Assche 2024-03-28 23:14 ` Justin Stitt 2024-03-28 23:18 ` Arnd Bergmann 2024-03-28 14:04 ` [PATCH 03/11] staging: replace weird strncpy() with memcpy() Arnd Bergmann 2024-03-28 16:35 ` Dan Carpenter 2024-04-08 14:45 ` Arnd Bergmann 2024-04-08 15:59 ` Dan Carpenter 2024-04-08 19:20 ` Arnd Bergmann 2024-03-28 14:04 ` [PATCH 04/11] orangefs: convert strncpy() to strscpy() Arnd Bergmann 2024-03-28 23:17 ` Justin Stitt 2024-03-28 14:04 ` [PATCH 05/11] test_hexdump: avoid string truncation warning Arnd Bergmann 2024-03-28 23:54 ` Justin Stitt 2024-04-08 15:38 ` Arnd Bergmann 2024-04-08 19:53 ` Justin Stitt 2024-03-28 14:04 ` [PATCH 06/11] acpi: avoid warning for truncated string copy Arnd Bergmann 2024-03-28 23:20 ` Justin Stitt 2024-04-08 14:41 ` Rafael J. Wysocki 2024-03-28 14:04 ` [PATCH 07/11] block/partitions/ldm: convert strncpy() to strscpy() Arnd Bergmann 2024-03-28 23:24 ` Justin Stitt 2024-03-28 14:04 ` [PATCH 08/11] blktrace: convert strncpy() to strscpy_pad() Arnd Bergmann 2024-03-28 14:14 ` Steven Rostedt 2024-04-08 18:05 ` Arnd Bergmann 2024-03-28 14:04 ` [PATCH 09/11] staging: rtl8723bs: convert strncpy to strscpy Arnd Bergmann 2024-03-28 23:01 ` Justin Stitt 2024-04-08 18:15 ` Arnd Bergmann 2024-03-28 14:04 ` [PATCH 10/11] staging: greybus: change strncpy() to strscpy() Arnd Bergmann 2024-03-28 15:00 ` Dan Carpenter 2024-04-08 18:26 ` Arnd Bergmann 2024-04-09 7:09 ` Dan Carpenter 2024-03-28 23:28 ` Justin Stitt 2024-04-08 18:30 ` Arnd Bergmann 2024-03-28 14:04 ` [PATCH 11/11] kbuild: enable -Wstringop-truncation globally Arnd Bergmann
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=hbwxiid52lv2dq7rdtvard3up46khu5uv335tzavwag3v6s6jl@dzjrnooz553m \ --to=justinstitt@google.com \ --cc=arnd@arndb.de \ --cc=arnd@kernel.org \ --cc=bcm-kernel-feedback-list@broadcom.com \ --cc=florian.fainelli@broadcom.com \ --cc=gregkh@linuxfoundation.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-rpi-kernel@lists.infradead.org \ --cc=linux-staging@lists.linux.dev \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.