RDKB-64347: Fixing coverity issues

[vanshika-singhal24]

Reason for change: Fixing low priority coverity issues.
Test Procedure: Build should be successful and the regression test should also succeed.

Risks: Low
Priority: P1
Signed-off-by: vanshika_lnu@comcast.com

[Copilot]

Pull request overview

This PR aims to address low-priority Coverity findings across the WiFi utilities, monitoring, and control-path code by tightening buffer handling and adjusting some data-structure usage.

Changes:

• Adds buffer-length-aware ping frame construction for ICMPv4/ICMPv6 (replacing unsafe string copies).
• Refactors hotspot/LnF VAP handling to use dynamically allocated wifi_vap_info_t.
• Refactors host-sync logic to use dynamically allocated LM_wifi_hosts_t and introduces a mutex field in that struct.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File	Description
source/utils/wifi_util.h	Adds hosts_lock to LM_wifi_hosts_t and removes packing attributes.
source/stats/wifi_monitor.c	Makes ping framing functions static and adds buffer_len + safer payload formatting.
source/core/wifi_ctrl_queue_handlers.c	Introduces heap allocation/cleanup paths for hotspot/LnF handling and host-sync data structures; adds mutex init/destroy.

---

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