* [PATCH] [net] ISDN: eicon: silence misleading array-bounds warning
@ 2017-01-27 12:32 Arnd Bergmann
2017-01-27 16:28 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: Arnd Bergmann @ 2017-01-27 12:32 UTC (permalink / raw)
To: Armin Schindler, Karsten Keil
Cc: Nicholas Mc Guire, kernel-build-reports, Arnd Bergmann, stable,
David S. Miller, netdev, linux-kernel
With some gcc versions, we get a warning about the eicon driver,
and that currently shows up as the only remaining warning in one
of the build bots:
In file included from ../drivers/isdn/hardware/eicon/message.c:30:0:
eicon/message.c: In function 'mixer_notify_update':
eicon/platform.h:333:18: warning: array subscript is above array bounds [-Warray-bounds]
The code is easily changed to open-code the unusual PUT_WORD() line
causing this to avoid the warning.
Cc: stable@vger.kernel.org
Link: http://arm-soc.lixom.net/buildlogs/stable-rc/v4.4.45/
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
drivers/isdn/hardware/eicon/message.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/isdn/hardware/eicon/message.c b/drivers/isdn/hardware/eicon/message.c
index 1a1d99704fe6..296f1411fe84 100644
--- a/drivers/isdn/hardware/eicon/message.c
+++ b/drivers/isdn/hardware/eicon/message.c
@@ -11297,7 +11297,8 @@ static void mixer_notify_update(PLCI *plci, byte others)
((CAPI_MSG *) msg)->header.ncci = 0;
((CAPI_MSG *) msg)->info.facility_req.Selector = SELECTOR_LINE_INTERCONNECT;
((CAPI_MSG *) msg)->info.facility_req.structs[0] = 3;
- PUT_WORD(&(((CAPI_MSG *) msg)->info.facility_req.structs[1]), LI_REQ_SILENT_UPDATE);
+ ((CAPI_MSG *) msg)->info.facility_req.structs[1] = LI_REQ_SILENT_UPDATE & 0xff;
+ ((CAPI_MSG *) msg)->info.facility_req.structs[2] = LI_REQ_SILENT_UPDATE >> 8;
((CAPI_MSG *) msg)->info.facility_req.structs[3] = 0;
w = api_put(notify_plci->appl, (CAPI_MSG *) msg);
if (w != _QUEUE_FULL)
--
2.9.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] [net] ISDN: eicon: silence misleading array-bounds warning
2017-01-27 12:32 [PATCH] [net] ISDN: eicon: silence misleading array-bounds warning Arnd Bergmann
@ 2017-01-27 16:28 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2017-01-27 16:28 UTC (permalink / raw)
To: arnd
Cc: mac, isdn, der.herr, kernel-build-reports, stable, netdev, linux-kernel
From: Arnd Bergmann <arnd@arndb.de>
Date: Fri, 27 Jan 2017 13:32:14 +0100
> With some gcc versions, we get a warning about the eicon driver,
> and that currently shows up as the only remaining warning in one
> of the build bots:
>
> In file included from ../drivers/isdn/hardware/eicon/message.c:30:0:
> eicon/message.c: In function 'mixer_notify_update':
> eicon/platform.h:333:18: warning: array subscript is above array bounds [-Warray-bounds]
>
> The code is easily changed to open-code the unusual PUT_WORD() line
> causing this to avoid the warning.
>
> Cc: stable@vger.kernel.org
> Link: http://arm-soc.lixom.net/buildlogs/stable-rc/v4.4.45/
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Ok, applied, thanks Arnd.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-01-27 16:28 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-27 12:32 [PATCH] [net] ISDN: eicon: silence misleading array-bounds warning Arnd Bergmann
2017-01-27 16:28 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).