Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RDKTV-35073: Telemetry2_0 crash with signature ProfileXConf_isNameEqual #42

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

yogeswaransky
Copy link
Contributor

No description provided.

@yogeswaransky yogeswaransky requested a review from a team as a code owner March 17, 2025 05:49
@rdkcmf-jenkins
Copy link

Coverity Issue - Check of thread-shared field evades lock acquisition

Thread1 sets "initialized" to a new value. Now the two threads have an inconsistent view of "initialized" and updates to fields correlated with "initialized" may be lost.

High Impact, CWE-543
LOCK_EVASION

How to fix

Guard the modification of "initialized" and the read used to decide whether to modify "initialized" with the same set of locks.

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
source/bulkdata/profilexconf.c:569

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.

2 participants