Skip to content

[core] Fix Coverity issue in inference component of core#33702

Merged
mlukasze merged 7 commits intoopenvinotoolkit:masterfrom
praasz:coverity/remove-not-required-copies-inference
Jan 26, 2026
Merged

[core] Fix Coverity issue in inference component of core#33702
mlukasze merged 7 commits intoopenvinotoolkit:masterfrom
praasz:coverity/remove-not-required-copies-inference

Conversation

@praasz
Copy link
Contributor

@praasz praasz commented Jan 20, 2026

Details:

  • Remove not required copies
  • Fix set move state on const ref variable
  • Fix member initialization

Tickets:

  • N/A

Signed-off-by: Raasz, Pawel <pawel.raasz@intel.com>
Signed-off-by: Raasz, Pawel <pawel.raasz@intel.com>
Signed-off-by: Raasz, Pawel <pawel.raasz@intel.com>
@praasz praasz requested a review from a team as a code owner January 20, 2026 12:49
@github-actions github-actions bot added the category: inference OpenVINO Runtime library - Inference label Jan 20, 2026
Comment on lines +43 to +47
ov::IPlugin::IPlugin()
: m_plugin_name(),
m_core(),
m_executor_manager(ov::threading::executor_manager()),
m_version() {}
Copy link
Contributor

@t-jankowski t-jankowski Jan 23, 2026

Choose a reason for hiding this comment

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

Why does coverity complain here? These three members are default initialized anyway?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The m_version members are not initialized. Added explicit all members.

@praasz praasz added this to the 2026.1 milestone Jan 23, 2026
@mlukasze mlukasze added this pull request to the merge queue Jan 26, 2026
Merged via the queue into openvinotoolkit:master with commit 78f6462 Jan 26, 2026
258 of 262 checks passed
Naseer-010 pushed a commit to Naseer-010/openvino that referenced this pull request Feb 18, 2026
…lkit#33702)

### Details:
 - Remove not required copies
 - Fix set move state on const ref variable
 - Fix member initialization

### Tickets:
 - N/A

---------

Signed-off-by: Raasz, Pawel <pawel.raasz@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: inference OpenVINO Runtime library - Inference

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants