Skip to content

WIP - Security hits solved#54

Closed
DariaMityagina wants to merge 5 commits intoopenvinotoolkit:developfrom
DariaMityagina:icv/dm/OS-hits
Closed

WIP - Security hits solved#54
DariaMityagina wants to merge 5 commits intoopenvinotoolkit:developfrom
DariaMityagina:icv/dm/OS-hits

Conversation

@DariaMityagina
Copy link
Contributor

Summary

Solving:

Target Platform For Release Notes

Aids the generation of release notes

  • NPU37XX
  • NPU40XX
  • NONE (Not included in release notes)

Classification of this Pull Request

  • Maintenance
  • BUG
  • Feature

Code Review Survey (Copy and Complete in your code review)

  • number_minutes_spent_on_review[0]
  • number_p1_defects_found[0]
  • number_p2_defects_found[0]
  • number_p3_defects_found[0]
  • number_p4_defects_found[0]

@DariaMityagina DariaMityagina self-assigned this Apr 14, 2025
@DariaMityagina DariaMityagina requested a review from a team as a code owner April 14, 2025 13:24
@DariaMityagina DariaMityagina changed the title WIP - Hits solved (2) WIP - Security hits solved (2) Apr 14, 2025

std::vector<AlgorithmParam> block_params(input_blocks * ALGORITHMS);
std::size_t total_blocks = static_cast<std::size_t>(input_blocks) * static_cast<std::size_t>(ALGORITHMS);
std::vector<AlgorithmParam> block_params(total_blocks);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Multiplication result converted to larger type

@DariaMityagina DariaMityagina marked this pull request as draft April 14, 2025 13:28
@DariaMityagina
Copy link
Contributor Author

DariaMityagina commented Apr 15, 2025

@DariaMityagina
Copy link
Contributor Author

image

@DariaMityagina DariaMityagina changed the title WIP - Security hits solved (2) WIP - Security hits solved Apr 16, 2025
@Maxim-Doronin
Copy link
Collaborator

Fixed has been brought through bulk update #63

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.

2 participants