Re: [PATCH 2/2] staging: wilc1000: modify uninitialized warning
From: Greg KH <gregkh@linuxfoundation.org>
Date: 2015-06-17 02:21:35
On Tue, Jun 16, 2015 at 04:11:25PM +0900, Dean Lee wrote:
quoted hunk ↗ jump to hunk
initialize it. Signed-off-by: Dean Lee <redacted> --- drivers/staging/wilc1000/host_interface.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c index 358283f..03e6bc5 100644 --- a/drivers/staging/wilc1000/host_interface.c +++ b/drivers/staging/wilc1000/host_interface.c@@ -7241,7 +7241,7 @@ s32 host_int_add_beacon(WILC_WFIDrvHandle hWFIDrv, u32 u32Interval, s32 s32Error = WILC_SUCCESS; tstrWILC_WFIDrv *pstrWFIDrv = (tstrWILC_WFIDrv *)hWFIDrv; tstrHostIFmsg strHostIFmsg; - tstrHostIFSetBeacon *pstrSetBeaconParam = &strHostIFmsg.uniHostIFmsgBody.strHostIFSetBeacon; + tstrHostIFSetBeacon *pstrSetBeaconParam = NULL;
This isn't fixing an "unitialized pointer" warning, the code is just fine. Well, it's really odd, but it is correct, there's nothing really wrong here that I can see. thanks, greg k-h