Skip to content

Conversation

@Vismalskumar0
Copy link
Contributor

No description provided.

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Parse recovery warning

variable "callbacks" not emitted, consider modeling it or review parse diagnostics to improve fidelity

Low Impact, CWE-none
RW.ENTITY_NOT_EMITTED

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/handlers/src/hostIf_jsonReqHandlerThread.cpp:148

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Array compared against 0

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

Medium Impact, CWE-398
NO_EFFECT

How to fix

Was "parentPrefix" 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/profiles/IP/Device_IP_Interface_IPv6Address.cpp:1680

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Array compared against 0

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

Medium Impact, CWE-398
NO_EFFECT

How to fix

Was ""Disabled"" 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/profiles/IP/Device_IP_Interface_IPv6Address.cpp:98

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Parse recovery warning

a value of type "int (*)(void *, const unsigned char , size_t)" cannot be used to initialize an entity of type "int ()(void *, const unsigned char *, unsigned int)"

Low Impact, CWE-none
RW.BAD_INITIALIZER_TYPE

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/handlers/src/hostIf_jsonReqHandlerThread.cpp:156

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Uninitialized scalar variable

Using uninitialized element of array "origin" when calling "strncmp".

High Impact, CWE-457
UNINIT

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/profiles/IP/Device_IP_Interface_IPv6Address.cpp:979

@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:1103

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Out-of-bounds access

Overrunning array "validLifetime" of 32 bytes by passing it to a function which accesses it at byte offset 127.

High Impact, CWE-119
OVERRUN

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/profiles/IP/Device_IP_Interface_IPv6Address.cpp:1077

@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:1103

@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 - Out-of-bounds access

Overrunning array "preferredLifetime" of 32 bytes by passing it to a function which accesses it at byte offset 127.

High Impact, CWE-119
OVERRUN

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/profiles/IP/Device_IP_Interface_IPv6Address.cpp:1058

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Array compared against 0

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

Medium Impact, CWE-398
NO_EFFECT

How to fix

Was ""Enabled"" 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/profiles/IP/Device_IP_Interface_IPv6Address.cpp:714

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Buffer not null terminated

Calling "strncpy" with a maximum size argument of 4096 bytes on destination array "param->paramName" of size 4096 bytes might leave the destination string unterminated.

High Impact, CWE-170
BUFFER_SIZE

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/handlers/src/hostIf_jsonReqHandlerThread.cpp:271

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Parse recovery warning

a value of type "int (*)(void *, const char , size_t)" cannot be used to initialize an entity of type "int ()(void *, const char *, unsigned int)"

Low Impact, CWE-none
RW.BAD_INITIALIZER_TYPE

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/handlers/src/hostIf_jsonReqHandlerThread.cpp:153

@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.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:1103

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Array compared against 0

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

Medium Impact, CWE-398
NO_EFFECT

How to fix

Was "stMsgData->paramName" 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/profiles/IP/Device_IP_Interface_IPv6Address.cpp:1025

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Array compared against 0

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

Medium Impact, CWE-398
NO_EFFECT

How to fix

Was ""Disabled"" 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/profiles/IP/Device_IP_Interface_IPv6Address.cpp:718

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Array compared against 0

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

Medium Impact, CWE-398
NO_EFFECT

How to fix

Was ""Disabled"" 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/profiles/IP/Device_IP_Interface_IPv6Address.cpp:109

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Uninitialized scalar variable

Using uninitialized element of array "ipaddressStatus" when calling "strncpy".

High Impact, CWE-457
UNINIT

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/profiles/IP/Device_IP_Interface_IPv6Address.cpp:816

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Array compared against 0

Comparing an array to null is not useful: ""Device.DeviceInfo.X_RDKCENTRAL-COM_RFC.Feature.Power.DeepSleepNotification.Enable" != NULL", since the test will always evaluate as true.

Medium Impact, CWE-398
NO_EFFECT

How to fix

Was ""Device.DeviceInfo.X_RDKCENTRAL-COM_RFC.Feature.Power.DeepSleepNotification.Enable"" 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/handlers/src/hostIf_IARM_ReqHandler.cpp:530

@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:1086

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Uninitialized scalar variable

Using uninitialized element of array "origin" when calling "strncpy".

High Impact, CWE-457
UNINIT

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/profiles/IP/Device_IP_Interface_IPv6Address.cpp:984

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Array compared against 0

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

Medium Impact, CWE-398
NO_EFFECT

How to fix

Was "childPrefixBits" 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/profiles/IP/Device_IP_Interface_IPv6Address.cpp:1737

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Parse recovery warning

a value of type "int (*)(void *, const unsigned char , size_t)" cannot be used to initialize an entity of type "int ()(void *, const unsigned char *, unsigned int)"

Low Impact, CWE-none
RW.BAD_INITIALIZER_TYPE

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/handlers/src/hostIf_jsonReqHandlerThread.cpp:154

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Parse recovery warning

a value of type "int (*)(void , long long)" cannot be used to initialize an entity of type "int ()(void *, long)"

Low Impact, CWE-none
RW.BAD_INITIALIZER_TYPE

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/handlers/src/hostIf_jsonReqHandlerThread.cpp:151

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Array compared against 0

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

Medium Impact, CWE-398
NO_EFFECT

How to fix

Was ""Enabled"" 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/profiles/IP/Device_IP_Interface_IPv6Address.cpp:1282

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Array compared against 0

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

Medium Impact, CWE-398
NO_EFFECT

How to fix

Was ""Disabled"" 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/profiles/IP/Device_IP_Interface_IPv6Address.cpp:1290

@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:961

@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:1103

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Uninitialized scalar variable

Using uninitialized element of array "prefixStatus" when calling "strncpy".

High Impact, CWE-457
UNINIT

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
src/hostif/profiles/IP/Device_IP_Interface_IPv6Address.cpp:1387

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