Hi Thierry,

> when using ConnectService of Manager interface a segfault arises due to a 
> null pointer. 
> 
> in __connman_service_create_and_connect:
> 
>       [...]
>       service = lookup_by_identifier(name);
> 
>       if (service != NULL)
>               goto done;
> 
>       network = create_hidden_wifi(device, ssid, mode, security);
>       if (network != NULL) {
>               connman_network_set_group(network, group);
>               service->network_created = TRUE;
>       }
>       [...]
> in this part "service" is null
> 
> please find a fix below
> regards
> 
> ---
>  src/service.c |    6 +++---
>  1 files changed, 3 insertions(+), 3 deletions(-)

indeed a nice catch and I applied this patch, but I had to fix up the
commit message manually.

Things like hi, regards or lines over 72 characters are bad for commit
messages.

Regards

Marcel


_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman

Reply via email to