Hi Daniel,

On Thu, Jul 07, 2011 at 08:35:44AM +0200, Daniel Wagner wrote:
> >> @@ -1116,8 +1116,6 @@ int connman_device_remove_network(struct 
> >> connman_device *device,
> >>    if (network == NULL)
> >>            return 0;
> >>  
> >> -  __connman_network_set_device(network, NULL);
> >> -
> > I may be missing some logic, but then who is calling network_remove() for 
> > the
> > removed network ?
> 
> On the next line we remove the network from the device->networks hash
> table. When it's removed from the hash table, unregister_network() is
> called which does a __connman_network_set_device(network, NULL) again.
> Therefore I though the one above it is not necessary. Do I miss something?
No you're not, that makes sense. Thanks for the explanation.

Cheers,
Samuel.

-- 
Intel Open Source Technology Centre
http://oss.intel.com/
_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman

Reply via email to