On Wed, 2013-11-13 at 13:12 +0200, Patrik Flykt wrote:
>       Hi,
> 
> On Tue, 2013-11-12 at 10:06 +0000, Michael Ikey Doherty wrote:
> > diff --git a/plugins/wifi.c b/plugins/wifi.c
> > index bac1fc6..f676218 100644
> > --- a/plugins/wifi.c
> > +++ b/plugins/wifi.c
> > @@ -181,6 +181,7 @@ static int wifi_probe(struct connman_device *device)
> >     wifi->tethering = false;
> >     wifi->bridged = false;
> >     wifi->bridge = NULL;
> > +   wifi->scan_params = NULL;
> >     wifi->state = G_SUPPLICANT_STATE_INACTIVE;
> 
> Just above these lines the whole wifi_data struct is allocated and
> initialized to zero so you could instead remove all the lines setting
> something to false or NULL. The value of G_SUPPLICANT_STATE_INACTIVE is
> not zero so that one needs to be kept.
Indeed, doing so now.

Kind Regards,
Michael Ikey Doherty
> 
> Cheers,
> 
>       Patrik
> 
> _______________________________________________
> connman mailing list
> [email protected]
> https://lists.connman.net/mailman/listinfo/connman

Attachment: signature.asc
Description: This is a digitally signed message part

---------------------------------------------------------------------
Intel Corporation (UK) Limited
Registered No. 1134945 (England)
Registered Office: Pipers Way, Swindon SN3 1RJ
VAT No: 860 2173 47

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.
_______________________________________________
connman mailing list
[email protected]
https://lists.connman.net/mailman/listinfo/connman

Reply via email to