Re: [PATCH] Send hostname as part of DHCP request.

[Denis Kenzior <denkenz@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 2004 bytes --]

Hi,

On 6/18/21 9:00 AM, mjohnson459(a)gmail.com wrote:

Can you include a proper name in the next version so it can be applied?  We 
won't apply patches with just an email address and no proper author name.

> This is based on a previous patch by Roberto Santalla Fernández.
> 
> A new config is introduced into the network config file under IPv4
> called SendHostname. If this is set to true then we add the hostname
> into all DHCP requests. The default is false.

The patch looks fine.  Just a small nitpick below...

> ---
>   src/iwd.network.rst |  6 ++++++
>   src/netconfig.c     | 15 +++++++++++++++
>   2 files changed, 21 insertions(+)
> 

<snip>

> diff --git a/src/netconfig.c b/src/netconfig.c
> index 316431ee..eb6d80ff 100644
> --- a/src/netconfig.c
> +++ b/src/netconfig.c
> @@ -1004,6 +1004,8 @@ bool netconfig_configure(struct netconfig *netconfig,
>   				netconfig_notify_func_t notify, void *user_data)
>   {
>   	char *mdns;
> +	char hostname[HOST_NAME_MAX + 1];
> +	bool send_hostname;
>   
>   	netconfig->dns4_overrides = l_settings_get_string_list(active_settings,
>   							"IPv4", "DNS", ' ');
> @@ -1044,6 +1046,19 @@ bool netconfig_configure(struct netconfig *netconfig,
>   	l_dhcp6_client_set_address(netconfig->dhcp6_client, ARPHRD_ETHER,
>   							mac_address, ETH_ALEN);
>   
> +	if (!l_settings_get_bool(active_settings,
> +					"IPv4", "SendHostname", &send_hostname))
> +		send_hostname = false;
> +
> +	if (send_hostname) {
> +		if (gethostname(hostname, sizeof(hostname)) == 0) {
> +			l_dhcp_client_set_hostname(
> +				netconfig->dhcp_client, hostname);
> +		} else {
> +			l_warn("netconfig: Unable to get hostname");
> +		}

Could you also print the errno / strerror() on the failure path so that we can 
easily see the reason this goes wrong in the logs?

> +	}
> +
>   	netconfig_ipv4_select_and_install(netconfig);
>   
>   	netconfig_ipv6_select_and_install(netconfig);
> 

Regards,
-Denis