Skip to content

Rdkb 63101 ml coverity issues#842

Merged
gsathish86 merged 6 commits intordkcentral:developfrom
bharathivelp:RDKB-63101_ML_coverity_issues
Feb 6, 2026
Merged

Rdkb 63101 ml coverity issues#842
gsathish86 merged 6 commits intordkcentral:developfrom
bharathivelp:RDKB-63101_ML_coverity_issues

Conversation

@bharathivelp
Copy link
Contributor

No description provided.

@bharathivelp bharathivelp requested a review from a team as a code owner January 21, 2026 12:15
@bharathivelp bharathivelp force-pushed the RDKB-63101_ML_coverity_issues branch 4 times, most recently from c19f6b3 to 4d5dedf Compare January 22, 2026 11:06
Copy link
Contributor

@pramod7456 pramod7456 left a comment

Choose a reason for hiding this comment

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

(psm_notify_get != NULL) what is Coverity error seen for this and other places also where NULL check is removed?

@bharathivelp
Copy link
Contributor Author

(psm_notify_get != NULL) what is Coverity error seen for this and other places also where NULL check is removed?

The error is NO_EFFECT,
psm_notify_get is an array so no need to check NULL

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

Risks: Low
Priority: P1
Signed-off-by: Velpula_Bharathi@comcast.com
Reason for change: Fixing high medium priority coverity issues.
Test Procedure: Build should be successful and the regression test should also succeed.

Risks: Low
Priority: P1
Signed-off-by: Velpula_Bharathi@comcast.com
@bharathivelp bharathivelp force-pushed the RDKB-63101_ML_coverity_issues branch 2 times, most recently from dae1d53 to 2985c41 Compare January 28, 2026 11:26
@gsathish86 gsathish86 merged commit 6db89a3 into rdkcentral:develop Feb 6, 2026
7 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Feb 6, 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