On Tue, Jun 16, 2015 at 04:11:25PM +0900, Dean Lee wrote: > initialize it. > > Signed-off-by: Dean Lee <dean.lee@xxxxxxxxx> > --- > 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 -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html