Hi Jukka,

On Wed, Sep 14, 2011 at 03:32:53PM +0300, Jukka Rissanen wrote:
> Provider name is already allocated in connman_provider_get() so we
> must deallocate it before setting it again.
> ---
>  src/provider.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/src/provider.c b/src/provider.c
> index 8f1862e..d6d1b7e 100644
> --- a/src/provider.c
> +++ b/src/provider.c
> @@ -533,6 +533,7 @@ int __connman_provider_create_and_connect(DBusMessage 
> *msg)
>  
>               provider->host = g_strdup(host);
>               provider->domain = g_strdup(domain);
> +             g_free(provider->name);
>               provider->name = g_strdup(name);
>               provider->type = g_strdup(type);
Patch applied, thanks.

Cheers,
Samuel.

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

Reply via email to