Skip to content

RDKB-65720: CCI crash during beacon data parsing#9

Open
rranga540 wants to merge 1 commit into
rdkcentral:developfrom
rranga540:beacon_parse_crash
Open

RDKB-65720: CCI crash during beacon data parsing#9
rranga540 wants to merge 1 commit into
rdkcentral:developfrom
rranga540:beacon_parse_crash

Conversation

@rranga540

Copy link
Copy Markdown
Contributor

Impacted Platforms: TXB7, TXB8, VXB10, SXB10
Reason for change: Failed to parse the start_ap_data
Test Procedure: Trigger any vapconfiguratio test case from cci.
Risks: Low
Priority: P2
Signed-off-by:RaviShankar.R_Rangaraj@comcast.com

Copilot AI review requested due to automatic review settings June 24, 2026 10:24
@rdkcmf-jenkins

Copy link
Copy Markdown
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 1 file pending approval, 0 files pending identification.

  • Protex Server Path: /home/blackduck/github/OneWifiTestSuite-rdkfmac/9/rdkcentral/OneWifiTestSuite-rdkfmac

  • Commit: 73bad99

Report detail: gist'

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses a CCI crash when parsing start_ap_data by removing SSID handling from the START_AP message path, so beacon head/tail data parsing no longer depends on SSID fields/offsets.

Changes:

  • Removed ssid/ssid_len from the wlan_emu_cfg80211_start_ap_t message struct (non-5.15 mod_src).
  • Stopped extracting/copying NL80211_ATTR_SSID into the START_AP message in mod_src/rdkfmac_netdev.c.
  • Stopped serializing SSID length/data in the START_AP char-device payload in linux_5.15/mod_src/rdkfmac_cdev.c.

Reviewed changes

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

File Description
mod_src/wlan_emu_msg_data.h Removes SSID fields from the START_AP message struct to align message layout with beacon parsing expectations.
mod_src/rdkfmac_netdev.c Drops SSID attribute extraction when building START_AP message data.
linux_5.15/mod_src/rdkfmac_cdev.c Updates START_AP message serialization to omit SSID fields, preventing downstream parsing offset issues.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants