Fuzzing

[Atchaya-Sivagnanam]

RDKB-64403 Fix memory issue found by libFuzzer

Reason for change: Fix memory issue identified through fuzz testing.
Test Procedure: Debug
Priority: P1
Risks: Low
Signed-off-by: Atchaya Sivagnanam Atchaya_Sivagnanam@comcast.com

[Copilot]

Pull request overview

This PR addresses multiple memory-safety issues surfaced by libFuzzer across the webconfig encoder/decoder and WiFi monitor/stats code paths, primarily by bounding string handling and fixing off-by-one / bounds checks.

Changes:

• Add bounded helpers in wifi_encoder.c to avoid unbounded strlen()/string reads on fixed-size char arrays and harden ChannelAvailability string building.
• Harden decoders/monitors with safer copies (snprintf), index bounds checks, and corrected comparisons (>= vs >).
• Adjust WiFi API radio subdoc decode initialization behavior.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
source/webconfig/wifi_webconfig_wifiapi_radio.c	Alters decoded-params initialization during wifiapi radio subdoc decode.
source/webconfig/wifi_encoder.c	Introduces bounded string helpers; tightens buffer building; adds NULL checks; updates multiple JSON string encodes.
source/webconfig/wifi_decoder.c	Avoids writing into cJSON-owned buffers, adds bounds checks, and replaces unsafe strcpy with snprintf in several decodes.
source/stats/wifi_stats_radio_temperature.c	Fixes radio index upper-bound check (>=).
source/stats/wifi_stats_radio_diagnostics.c	Fixes radio index upper-bound check (>=).
source/stats/wifi_stats_radio_channel.c	Fixes radio index upper-bound check (>=).
source/stats/wifi_stats_neighbor_report.c	Fixes radio index upper-bound check (>=).
source/stats/wifi_monitor.c	Adds index bounds check for stats enable array, fixes formatting/copy bounds, and corrects loop upper bound.

---

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

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.