[PATCH 3/9] config: Call utsname_set_domainname() for domainname provisioned in service

Patrik Flykt patrik.flykt at linux.intel.com
Thu Sep 12 06:20:00 PDT 2013


	Hi,

On Tue, 2013-09-10 at 15:46 +0200, Peter Meerwald wrote:
> ---
>  src/config.c |    4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/src/config.c b/src/config.c
> index d6a3aa4..5f0f9f8 100644
> --- a/src/config.c
> +++ b/src/config.c
> @@ -1212,8 +1212,10 @@ static void provision_service(gpointer key, gpointer value,
>  	__connman_service_set_config(service, config->config_ident,
>  						config->config_entry);
>  
> -	if (config->domain_name)
> +	if (config->domain_name) {
>  		__connman_service_set_domainname(service, config->domain_name);
> +		__connman_utsname_set_domainname(config->domain_name);
> +    }
>  
>  	if (config->nameservers) {
>  		int i;

At this point in time the service is merely created, it might never be
connected. So utsname cannot be set here.

It seems there is a bug with ConnMan, __connman_utsname_set_domainname()
should be called when the default service is changed. Now the domainname
is set from the last DHCPACK received.


Cheers,

	Patrik




More information about the connman mailing list