Skip to content

Conversation

@gomathishankar37
Copy link
Contributor

No description provided.

@gomathishankar37 gomathishankar37 requested a review from a team as a code owner May 15, 2025 13:16
@yogeswaransky yogeswaransky changed the title Add middleware runtime WhoAmI checks in MaintenanceManager RDK-57914: Dynamic WhoAmI feature control for Manintenance Manager Jun 11, 2025
@gomathishankar37
Copy link
Contributor Author

LGTM

@gomathishankar37 gomathishankar37 requested a review from tdeva14 June 20, 2025 13:57
@gomathishankar37
Copy link
Contributor Author

Please resolve conflicts. LGTM

@rdkcmf-jenkins
Copy link
Contributor

rdkcmf-jenkins commented Jun 25, 2025

Coverity Issue - Data race condition

Accessing "this->g_task_status" without holding lock "WPEFramework::Plugin::MaintenanceManager.m_statusMutex". Elsewhere, "WPEFramework::Plugin::MaintenanceManager.g_task_status" is written to with "MaintenanceManager.m_statusMutex" held 13 out of 16 times.

Medium Impact, CWE-366
MISSING_LOCK

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
MaintenanceManager/MaintenanceManager.cpp:523

@rdkcmf-jenkins
Copy link
Contributor

rdkcmf-jenkins commented Jun 25, 2025

Coverity Issue - Data race condition

Accessing "this->m_task_map" without holding lock "WPEFramework::Plugin::MaintenanceManager.m_statusMutex". Elsewhere, "WPEFramework::Plugin::MaintenanceManager.m_task_map" is written to with "MaintenanceManager.m_statusMutex" held 11 out of 15 times.

Medium Impact, CWE-366
MISSING_LOCK

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
MaintenanceManager/MaintenanceManager.cpp:497

@yogeswaransky
Copy link

Comments addressed and Conflicts resolved

Copy link
Contributor

@tdeva14 tdeva14 left a comment

Choose a reason for hiding this comment

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

LGTM

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Unchecked return value

Calling "Invoke" without checking return value (as is done elsewhere 1 out of 1 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:
MaintenanceManager/MaintenanceManager.cpp:612

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Variable copied when it could be moved

"joGetResult" is passed-by-value as parameter to "WPEFramework::Core::JSON::VariantContainer::VariantContainer(WPEFramework::Core::JSON::VariantContainer const &)", when it could be moved instead.

Low Impact, CWE-none
COPY_INSTEAD_OF_MOVE

How to fix

Use "std::move(""joGetResult"")" instead of "joGetResult".

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
MaintenanceManager/MaintenanceManager.cpp:619

Copy link
Contributor

@tdeva14 tdeva14 left a comment

Choose a reason for hiding this comment

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

LGTM

@tdeva14 tdeva14 merged commit 54f17f6 into develop Jun 25, 2025
6 of 8 checks passed
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.

5 participants