Skip to content

Conversation

@Vismalskumar0
Copy link
Contributor

No description provided.

@Vismalskumar0 Vismalskumar0 requested a review from a team as a code owner October 23, 2025 14:48
@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Resource leak

Freeing pointer-to-pointer "pParam_name_list" without freeing the pointer it points to leaks the storage that inner pointer points to.

High Impact, CWE-404
RESOURCE_LEAK

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/parodusClient/waldb/waldb.cpp:1088

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Array compared against 0

Comparing an array to null is not useful: ""NumberOfEntries" != NULL", since the test will always evaluate as true.

Medium Impact, CWE-398
NO_EFFECT

How to fix

Was ""NumberOfEntries"" formerly declared as a pointer?

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/parodusClient/waldb/waldb.cpp:209

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Resource leak

Variable "dmParam" going out of scope leaks the storage "dmParam.objectName" points to.

High Impact, CWE-404
RESOURCE_LEAK

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/parodusClient/waldb/waldb.cpp:1105

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Resource leak

Variable "dmParam" going out of scope leaks the storage "dmParam.bsUpdate" points to.

High Impact, CWE-404
RESOURCE_LEAK

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/parodusClient/waldb/waldb.cpp:1105

@rdkcmf-jenkins
Copy link
Contributor

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

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

High Impact, CWE-543
LOCK_EVASION

How to fix

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

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/parodusClient/waldb/waldb.cpp:93

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Resource leak

Variable "dmParam" going out of scope leaks the storage "dmParam.access" points to.

High Impact, CWE-404
RESOURCE_LEAK

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/parodusClient/waldb/waldb.cpp:1105

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Resource leak

Variable "dmParam" going out of scope leaks the storage "dmParam.paramName" points to.

High Impact, CWE-404
RESOURCE_LEAK

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/parodusClient/waldb/waldb.cpp:1105

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Copy into fixed size buffer

You might overrun the 512-character fixed-size string "currentParam" by copying the return value of "Value" without checking the length.

Low Impact, CWE-120
STRING_OVERFLOW

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/parodusClient/waldb/waldb.cpp:963

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