RDKCOM-5595: RDKBDEV-3450 The CM is not updated with DNS Name (Option15), instead utopia.net is displayed in LAN side provisioning.

[aprasad-97]

Reason for change: Added provisioning to fetch the DHCPv4 Option 15 (Domain Name) value via udhcpc and forward it to the WAN Manager. The WAN Manager then sets the corresponding sysevent, enabling dnsmasq to propagate the domain name to LAN clients.

Test Procedure:

1. Configure in DHCP Policy, DHCP option-15 with dns name in CNR. (eg: qacafe)
2. Reboot CM.
3. Wait a few minutes until eRouter0 provisioning will finish.
4. Check that /var/dnsmasq.conf is updated with the DNS name
5. Check the DHCP provisioning packets in the LAN. The DHCP offer packet in the LAN should display correct domain name in option-15. Instead, it currently displays utopia.net .

Risks: Low
Priority: P1

[Copilot]

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds DHCP domain-name propagation into the udhcpc service’s data path so it can be stored and logged alongside existing DHCP lease fields.

Changes:

• Introduces a DHCP_DOMAIN_NAME environment key.
• Logs the domain value in handle_wan.
• Reads the DHCP domain from the environment into ipc_dhcpv4_data_t.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

[tinaelizabeth84]

@aprasad-97 Please address the coPilot review comments. If they are not valid please provide an explanation and resolve them. Thanks

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

[aprasad-97]

@aprasad-97 Please address the coPilot review comments. If they are not valid please provide an explanation and resolve them. Thanks

Adressed. @tinaelizabeth84

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (1)

source/service_udhcpc/service_udhcpc.c:1345

• The newly added Option 15 (domain name) extraction/forwarding path isn’t covered by the existing service_udhcpc unit tests. Given this logic directly affects what WAN Manager (and ultimately dnsmasq/LAN DHCP offers) will advertise, it should be verified with at least one test that sets the domain env var and checks that get_and_fill_env_data() populates dhcpv4_data->domain for bound/renew states (and leaves it empty when unset).

        /** Domain name */
        if ((env = getenv(DHCP_DOMAIN_NAME)) != NULL)
        {
            safec_rc = strcpy_s(dhcpv4_data->domain, sizeof(dhcpv4_data->domain), env); // CID 187457: Buffer not null terminated (BUFFER_SIZE)
            ERR_CHK(safec_rc);