On Mon, Apr 01, 2024 at 12:13:19PM -0700, Charles Hardin wrote:

> There is code in the bootp parsing for NIS domain and add the
> same support for the dhcp options as well. This allows the same
> usage of the data when the dhcp command is used in the boot
> command.
> 
> Signed-off-by: Charles Hardin <[email protected]>
> ---
>  net/bootp.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/net/bootp.c b/net/bootp.c
> index 6800290963..046caf3685 100644
> --- a/net/bootp.c
> +++ b/net/bootp.c
> @@ -883,6 +883,14 @@ static void dhcp_process_options(uchar *popt, uchar *end)
>                       break;
>               case 28:        /* Ignore Broadcast Address Option */
>                       break;
> +             case 40:        /* NIS Domain name */
> +                     if (net_nis_domain[0] == 0) {
> +                             size = truncate_sz("NIS Domain Name",
> +                                     sizeof(net_nis_domain), size);
> +                             memcpy(&net_nis_domain, ext + 2, size);
> +                             net_nis_domain[size] = 0;
> +                     }
> +                     break;
>  #if defined(CONFIG_CMD_SNTP) && defined(CONFIG_BOOTP_NTPSERVER)
>               case 42:        /* NTP server IP */
>                       net_copy_ip(&net_ntp_server, (popt + 2));

This fails to build:
net/bootp.c: In function 'dhcp_process_options':
net/bootp.c:890:57: error: 'ext' undeclared (first use in this function)
  890 |                                 memcpy(&net_nis_domain, ext + 2, size);
      |                                                         ^~~
net/bootp.c:890:57: note: each undeclared identifier is reported only once for 
each function it appears in

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to