Skip to content

Coverity- Medium and low priority issues#524

Merged
DouglasAdler merged 2 commits into
masterfrom
feature/RDKEMW-17769
Jul 15, 2026
Merged

Coverity- Medium and low priority issues#524
DouglasAdler merged 2 commits into
masterfrom
feature/RDKEMW-17769

Conversation

@trupthi1403

@trupthi1403 trupthi1403 commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Summary: Fix for issues-AUTO_CAUSES_COPY, BAD_CHECK_OF_WAIT_COND, CHECKED_RETURN, COPY_INSTEAD_OF_MOVE, DEADCODE, PW.INCLUDE_RECURSION, PW.USELESS_TYPE_QUALIFIER_ON_RETURN_TYPE, UNCAUGHT_EXCEPT, UNINIT_CTOR
Type: Fix
Test Plan: UT/CT, Fullstack
Jira: RDKEMW-17769

Copilot AI review requested due to automatic review settings June 9, 2026 11:56
Comment thread media/server/gstplayer/source/GstWebAudioPlayer.cpp Fixed
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

Pull request must be merged with a description containing the required fields,

Summary:
Type: Feature/Fix/Cleanup
Test Plan:
Jira:

If there is no jira releated to this change, please put 'Jira: NO-JIRA'.

Description can be changed by editing the top comment on your pull request and making a new commit.

Copilot AI left a comment

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

Coverage statistics of your commit:
Lines coverage stays unchanged and is: 84.2%
Functions coverage stays unchanged and is: 92.6%

@trupthi1403
trupthi1403 force-pushed the feature/RDKEMW-17769 branch from 0584fa7 to 14b0820 Compare June 11, 2026 09:54
@github-actions

Copy link
Copy Markdown

Coverage statistics of your commit:
Lines coverage stays unchanged and is: 84.2%
Functions coverage stays unchanged and is: 92.6%

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 14 out of 14 changed files in this pull request and generated 5 comments.

Comment thread media/server/main/source/MainThread.cpp Outdated
Comment thread ipc/common/source/NamedSocket.cpp Outdated
Comment thread ipc/common/source/NamedSocket.cpp Outdated
Comment thread common/source/LinuxUtils.cpp Outdated
Comment thread common/source/LinuxUtils.cpp Outdated
Copilot AI review requested due to automatic review settings June 12, 2026 09:20
@trupthi1403
trupthi1403 force-pushed the feature/RDKEMW-17769 branch from 291502e to ef79d08 Compare June 12, 2026 09:20

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 6 comments.

Comment thread serverManager/service/source/ConfigReader.cpp Outdated
Comment thread media/server/gstplayer/source/GstWebAudioPlayer.cpp Outdated
Comment thread ipc/common/source/NamedSocket.cpp
Comment thread ipc/common/source/NamedSocket.cpp
Comment thread common/source/LinuxUtils.cpp
Comment thread common/source/LinuxUtils.cpp
@trupthi1403
trupthi1403 force-pushed the feature/RDKEMW-17769 branch from ef79d08 to 3aa2b72 Compare June 12, 2026 09:36
Copilot AI review requested due to automatic review settings June 12, 2026 11:09

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 6 comments.

Comment thread serverManager/service/source/ConfigReader.cpp Outdated
Comment thread media/server/gstplayer/source/GstWebAudioPlayer.cpp
Comment thread ipc/common/source/NamedSocket.cpp
Comment thread ipc/common/source/NamedSocket.cpp
Comment thread common/source/LinuxUtils.cpp
Comment thread common/source/LinuxUtils.cpp
@trupthi1403
trupthi1403 force-pushed the feature/RDKEMW-17769 branch from 7ae4d87 to b4d90a6 Compare June 12, 2026 12:08
Copilot AI review requested due to automatic review settings June 15, 2026 09:43
@trupthi1403
trupthi1403 force-pushed the feature/RDKEMW-17769 branch from b4d90a6 to 03c9c58 Compare June 15, 2026 09:43

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 17 out of 17 changed files in this pull request and generated 1 comment.

Comment thread serverManager/service/source/ConfigReader.cpp Outdated
@github-actions

Copy link
Copy Markdown

Coverage statistics of your commit:
Lines coverage stays unchanged and is: 84.2%
Functions coverage stays unchanged and is: 92.6%

@trupthi1403
trupthi1403 force-pushed the feature/RDKEMW-17769 branch from 03c9c58 to cccf5ed Compare June 17, 2026 06:25
@github-actions

Copy link
Copy Markdown

Coverage statistics of your commit:
Congratulations, your commit improved lines coverage from: 84.2% to 84.3%
Functions coverage stays unchanged and is: 92.6%

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 17 out of 17 changed files in this pull request and generated 2 comments.

Comment thread serverManager/service/source/ConfigReader.cpp Outdated
Comment thread media/server/main/source/MainThread.cpp Outdated
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

Coverage statistics of your commit:
Lines coverage stays unchanged and is: 84.3%
Functions coverage stays unchanged and is: 92.7%

Comment thread common/source/LinuxUtils.cpp Outdated
Comment thread common/source/LinuxUtils.cpp Outdated
Comment thread ipc/common/source/NamedSocket.cpp Outdated
Comment thread ipc/common/source/NamedSocket.cpp Outdated
Comment thread media/server/gstplayer/source/GstWebAudioPlayer.cpp Outdated
Comment thread media/server/ipc/source/MediaPipelineModuleService.cpp Outdated
Comment thread media/server/main/source/MainThread.cpp Outdated
Comment thread serverManager/service/source/ConfigReader.cpp Outdated
@trupthi1403
trupthi1403 force-pushed the feature/RDKEMW-17769 branch from 26794cc to 5ee72a3 Compare July 6, 2026 10:22
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

Coverage statistics of your commit:
Lines coverage stays unchanged and is: 84.3%
Functions coverage stays unchanged and is: 92.7%

Copilot AI review requested due to automatic review settings July 7, 2026 11:44
@trupthi1403
trupthi1403 force-pushed the feature/RDKEMW-17769 branch from 5ee72a3 to 04e4f47 Compare July 7, 2026 11:44

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 17 out of 17 changed files in this pull request and generated 5 comments.

Comment thread ipc/common/source/NamedSocket.cpp
Comment thread ipc/common/source/NamedSocket.cpp
Comment thread common/source/LinuxUtils.cpp
Comment thread common/source/LinuxUtils.cpp
@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown

Coverage statistics of your commit:
Lines coverage stays unchanged and is: 84.3%
Functions coverage stays unchanged and is: 92.7%

Copilot AI review requested due to automatic review settings July 14, 2026 07:33
@trupthi1403
trupthi1403 force-pushed the feature/RDKEMW-17769 branch from 04e4f47 to dd8e828 Compare July 14, 2026 07:33

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 17 out of 17 changed files in this pull request and generated 1 comment.

Comment thread media/server/main/source/MainThread.cpp
@github-actions

Copy link
Copy Markdown

Coverage statistics of your commit:
Lines coverage stays unchanged and is: 84.3%
Functions coverage stays unchanged and is: 92.7%

Summary: Fix for AUTO_CAUSES_COPY, BAD_CHECK_OF_WAIT_COND,
CHECKED_RETURN, COPY_INSTEAD_OF_MOVE, DEADCODE, PW.INCLUDE_RECURSION,
PW.USELESS_TYPE_QUALIFIER_ON_RETURN_TYPE, UNCAUGHT_EXCEPT, UNINIT_CTOR
Type: Fix
Test Plan: UT/CT, Fullstack
Jira: RDKEMW-17769
Summary: Resolving CI failures
Type: Fix
Test Plan: UT/CT, Fullstack
Jira: RDKEMW-17769
Copilot AI review requested due to automatic review settings July 14, 2026 13:19
@trupthi1403
trupthi1403 force-pushed the feature/RDKEMW-17769 branch from dd8e828 to 991af73 Compare July 14, 2026 13:19

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 17 out of 17 changed files in this pull request and generated 3 comments.

Comment thread media/server/main/interface/IMainThread.h
Comment thread media/server/main/source/MainThread.cpp
Comment thread media/server/gstplayer/source/GstWebAudioPlayer.cpp
@github-actions

Copy link
Copy Markdown

Coverage statistics of your commit:
Lines coverage stays unchanged and is: 84.3%
Functions coverage stays unchanged and is: 92.7%

@DouglasAdler
DouglasAdler merged commit bb0058e into master Jul 15, 2026
40 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 15, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants