Skip to content

RDKB-65078 : [Sprint]Observing CCI with fingerprint 73073501 (#27) - #37

Merged
gsathish86 merged 2 commits into
rdkcentral:mainfrom
rranga540:RDKB-65078_main
Jul 21, 2026
Merged

RDKB-65078 : [Sprint]Observing CCI with fingerprint 73073501 (#27)#37
gsathish86 merged 2 commits into
rdkcentral:mainfrom
rranga540:RDKB-65078_main

Conversation

@rranga540

Copy link
Copy Markdown
Contributor

RDKB-65078: CCI crashing while executing PR0193

Impacted Platforms: TXB7, XB8, VXB10, SXB10
Reason for change: Connectivity_q is null due to parse failure. Test Procedure: Trigger PR0193 and check the results. Risks: Low
Priority: P2
Signed-off-by:RaviShankar.R_Rangaraj@comcast.com

RDKB-65078: CCI crashing while executing PR0193

Impacted Platforms: TXB7, XB8, VXB10, SXB10
Reason for change: Connectivity_q is null due to parse failure. Test Procedure: Trigger PR0193 and check the results. Risks: Low
Priority: P2
Signed-off-by:RaviShankar.R_Rangaraj@comcast.com

…ral#27)

RDKB-65078: CCI crashing while executing PR0193

Impacted Platforms: TXB7, XB8, VXB10, SXB10
Reason for change: Connectivity_q is null due to parse failure.
Test Procedure: Trigger PR0193 and check the results.
Risks: Low
Priority: P2
Signed-off-by:RaviShankar.R_Rangaraj@comcast.com

RDKB-65078: CCI crashing while executing PR0193

Impacted Platforms: TXB7, XB8, VXB10, SXB10
Reason for change: Connectivity_q is null due to parse failure.
Test Procedure: Trigger PR0193 and check the results.
Risks: Low
Priority: P2
Signed-off-by:RaviShankar.R_Rangaraj@comcast.com
Copilot AI review requested due to automatic review settings July 21, 2026 05:57

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 hardens STA management test parameter decoding and timeout handling to prevent CCI crashes during PR0193 when JSON parsing/config data is incomplete (e.g., connectivity_q not created due to parse/config issues).

Changes:

  • Treat missing/invalid TestDuration as a decode error (log + return error) instead of silently defaulting to 0.
  • Prevent NULL dereference in step_timeout() by checking connectivity_q for NULL before calling queue_count() / queue_peek().

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

Comment thread src/tests/wlan_emu_test_param_sta_management.cpp
@gsathish86
gsathish86 merged commit ad6ae3e into rdkcentral:main Jul 21, 2026
3 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 21, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants