Skip to content

Topic/test#85

Open
yuvaramachandran-gurusamy wants to merge 7 commits intodevelopfrom
topic/Test
Open

Topic/test#85
yuvaramachandran-gurusamy wants to merge 7 commits intodevelopfrom
topic/Test

Conversation

@yuvaramachandran-gurusamy
Copy link
Copy Markdown
Contributor

No description provided.

apatel859 and others added 4 commits July 28, 2025 18:05
Signed-off-by: apatel859 <Amit_Patel5@comcast.com>
Signed-off-by: apatel859 [Amit_Patel5@comcast.com]
@yuvaramachandran-gurusamy yuvaramachandran-gurusamy requested a review from a team as a code owner July 28, 2025 20:06
@yuvaramachandran-gurusamy yuvaramachandran-gurusamy requested a review from a team July 28, 2025 20:06
@rdkcmf-jenkins
Copy link
Copy Markdown
Contributor

Coverity Issue - Unchecked return value

Calling "IARM_Bus_RegisterCall" without checking return value (as is done elsewhere 14 out of 14 times).

Medium Impact, CWE-252
CHECKED_RETURN

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
dsmgr/dsMgrPwrEventListener.c:141

@rdkcmf-jenkins
Copy link
Copy Markdown
Contributor

Coverity Issue - Unchecked return value

Calling "IARM_Bus_RegisterCall" without checking return value (as is done elsewhere 14 out of 14 times).

Medium Impact, CWE-252
CHECKED_RETURN

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
dsmgr/dsMgrPwrEventListener.c:139

@rdkcmf-jenkins
Copy link
Copy Markdown
Contributor

Coverity Issue - Unchecked return value

Calling "IARM_Bus_RegisterCall" without checking return value (as is done elsewhere 14 out of 14 times).

Medium Impact, CWE-252
CHECKED_RETURN

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
dsmgr/dsMgrPwrEventListener.c:140

@rdkcmf-jenkins
Copy link
Copy Markdown
Contributor

Coverity Issue - Unchecked return value

Calling "IARM_Bus_RegisterCall" without checking return value (as is done elsewhere 14 out of 14 times).

Medium Impact, CWE-252
CHECKED_RETURN

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
dsmgr/dsMgrPwrEventListener.c:142

@rdkcmf-jenkins
Copy link
Copy Markdown
Contributor

Coverity Issue - Unchecked return value

Calling "IARM_Bus_RegisterCall" without checking return value (as is done elsewhere 14 out of 14 times).

Medium Impact, CWE-252
CHECKED_RETURN

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
dsmgr/dsMgrPwrEventListener.c:138

Comment thread stubs/ds_stubs.cpp
}

AudioOutputPort::AudioOutputPort(const int type, const int index, const int id) {
}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Coverity Issue - Uninitialized scalar field

Non-static class member "_muted" is not initialized in this constructor nor in any functions that it calls.

Medium Impact, CWE-457
UNINIT_CTOR


memset(&setMode, 0, sizeof(setMode));
setMode.handle = getHandle.handle;
strncpy(setMode.portName, aPort.getName().c_str(), 32);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Coverity Issue - Buffer not null terminated

Calling "strncpy" with a maximum size argument of 32 bytes on destination array "setMode.portName" of size 32 bytes might leave the destination string unterminated.

High Impact, CWE-170
BUFFER_SIZE


memset(&setMode, 0, sizeof(setMode));
setMode.handle = getHandle.handle;
strncpy(setMode.portName, aPort.getName().c_str(), 32);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Coverity Issue - Buffer not null terminated

Calling "strncpy" with a maximum size argument of 32 bytes on destination array "setMode.portName" of size 32 bytes might leave the destination string unterminated.

High Impact, CWE-170
BUFFER_SIZE

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