Skip to content

RDKB-62620: Coverity Cleanup for CcspCommonLibrary#45

Open
rirfha948 wants to merge 2 commits intodevelopfrom
Coverity-cleanup-missinglock
Open

RDKB-62620: Coverity Cleanup for CcspCommonLibrary#45
rirfha948 wants to merge 2 commits intodevelopfrom
Coverity-cleanup-missinglock

Conversation

@rirfha948
Copy link
Contributor

@rirfha948 rirfha948 commented Nov 27, 2025

UserStory: RDKB-61923
Subtask: RDKB-62620
Reason for change: Fixes for CcspCommonLibrary Coverity Fixes
Category: Missing lock
Severity: MEDIUM
Test Procedure:

  • TBD Risks: None
    Priority: P3
    1.Is this a Bug or a User Story (US)?

It is a user story - RDKB-61923

If it is a User Story:
Have all dependent PRs from other components been listed (if any)?

Independent Subtask only, RDKB-62620
Does the commit message include both the User Story ticket and the Subtask ticket?
YES
Will be all changes related to the User Story squashed and merged in a single commit?
Seperate commits
Has the PR been raised only after completing all changes for the User Story (no partial changes)?
After completion of Sub-task
Has code development for the User Story been completed?
Yes
If yes, has the Gerrit topic or list of all dependent PRs across components (including meta-layer changes) been shared?
NOT APPLICABLE

Is there a validation log available in the Jira ticket for verifying builds with the updated generic-srcrev.inc across all platforms?

Bringup tested.
dmclilog.txt

If yes, have the links to validation comments been shared?

Verification Build

image

Reason for change: Fixes for CcspCommonLibrary Coverity Fixes
                   Category: Missing lock
                   Severity: MEDIUM
Test Procedure:
  - TBD
Risks: None
Priority: P3

Signed-off-by: rirfha948 <rasina_irfhan@comcast.com>
@aadhithan01
Copy link

aadhithan01 commented Dec 1, 2025

As per the request Logs has been attached and It looks good to me, But I can see Unit test got failed.

Unit tests / Execute unit tests in gtest test suite (pull_request)Failing after 41s

could you please review this ?

Copy link

@aadhithan01 aadhithan01 left a comment

Choose a reason for hiding this comment

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

Unit test got failed. .
Hi Aadhi,
It says the script itself is not found.

Think this can be skipped since gtest cases were not written for common library

Thanks
Rasina

@aadhithan01 aadhithan01 self-requested a review December 1, 2025 11:07
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