Skip to content

Conversation

jade-cho
Copy link
Contributor

Details:

  • Fix coverity issues

Tickets:

  • 174701

@jade-cho jade-cho requested review from a team as code owners October 17, 2025 09:07
@github-actions github-actions bot added the category: GPU OpenVINO GPU plugin label Oct 17, 2025
cldnn::mem_lock<uint8_t> dst_lock(merged_memory, stream);
for (size_t i = 0; i < user_tensors.size(); i++) {
auto input_tensor = std::dynamic_pointer_cast<RemoteTensorImpl>(user_tensors[i]._ptr);
OPENVINO_ASSERT(input_tensor != nullptr, "[GPU] Failed to cast user tensor to RemoteTensorImpl");
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks redundant,
bool is_remote_buffer = all_remote_buffers(user_tensors); - checked at line 699. It is better to mark as false positive

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Got it. Marked it as false positive.

@p-durandin p-durandin added this to the 2025.4 milestone Oct 17, 2025
@jade-cho jade-cho force-pushed the fix_coverity_issues branch from dd0a66c to 678a8a0 Compare October 20, 2025 00:38
Copy link
Contributor

@hyunback hyunback left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@ahnyoung-paul ahnyoung-paul left a comment

Choose a reason for hiding this comment

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

LGTM

@isanghao isanghao added this pull request to the merge queue Oct 20, 2025
Merged via the queue into openvinotoolkit:master with commit a46f046 Oct 20, 2025
187 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: GPU OpenVINO GPU plugin Code Freeze

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants