Hi Jaakko,

On ke, 2015-07-15 at 11:35 +0300, Jaakko Hannikainen wrote:
> Same as previous commit, but for Proxy.Configuration.

Proper commit message please.

> ---
>  src/service.c | 14 ++++++++++----
>  1 file changed, 10 insertions(+), 4 deletions(-)
> 
> diff --git a/src/service.c b/src/service.c
> index 37f8fb9..2bf64e8 100644
> --- a/src/service.c
> +++ b/src/service.c
> @@ -3071,9 +3071,12 @@ static int update_proxy_configuration(struct 
> connman_service *service,
>               if (servers_str) {
>                       g_strfreev(service->proxies);
>  
> -                     if (servers_str->len > 0)
> -                             service->proxies = g_strsplit_set(
> +                     if (servers_str->len > 0) {
> +                             char **proxies = g_strsplit_set(
>                                       servers_str->str, " ", 0);
> +                             proxies = remove_empty_strings(proxies);
> +                             service->proxies = proxies;
> +                     }
>                       else
>                               service->proxies = NULL;
>               }
> @@ -3081,9 +3084,12 @@ static int update_proxy_configuration(struct 
> connman_service *service,
>               if (excludes_str) {
>                       g_strfreev(service->excludes);
>  
> -                     if (excludes_str->len > 0)
> -                             service->excludes = g_strsplit_set(
> +                     if (excludes_str->len > 0) {
> +                             char **excludes = g_strsplit_set(
>                                       excludes_str->str, " ", 0);
> +                             excludes = remove_empty_strings(excludes);
> +                             service->excludes = excludes;
> +                     }
>                       else
>                               service->excludes = NULL;
>               }


Cheers,
Jukka


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

Reply via email to