Re: [PATCH, net] isdn: kcapi: avoid uninitialized data

From: David Miller
Date: Tue Mar 28 2017 - 21:00:08 EST


From: Arnd Bergmann <arnd@xxxxxxxx>
Date: Tue, 28 Mar 2017 12:11:07 +0200

> gcc-7 points out that the AVMB1_ADDCARD ioctl results in an unintialized
> value ending up in the cardnr parameter:
>
> drivers/isdn/capi/kcapi.c: In function 'old_capi_manufacturer':
> drivers/isdn/capi/kcapi.c:1042:24: error: 'cdef.cardnr' may be used uninitialized in this function [-Werror=maybe-uninitialized]
> cparams.cardnr = cdef.cardnr;
>
> This has been broken since before the start of the git history, so
> either the value is not used for anything important, or the ioctl
> command doesn't get called in practice.
>
> Setting the cardnr to zero avoids the warning and makes sure
> we have consistent behavior.
>
> Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>

Applied.