Skip to content

[core] Fix Coverity CID 2536763#33927

Merged
olpipi merged 5 commits intoopenvinotoolkit:masterfrom
praasz:coverity/overflowed-integer-argument-in-tensor-read
Feb 6, 2026
Merged

[core] Fix Coverity CID 2536763#33927
olpipi merged 5 commits intoopenvinotoolkit:masterfrom
praasz:coverity/overflowed-integer-argument-in-tensor-read

Conversation

@praasz
Copy link
Copy Markdown
Contributor

@praasz praasz commented Feb 2, 2026

Details:

  • Fix overflow by implicit cast to signed in read file as tensor by explicit convert.
  • If cast result is negative the file read return 0 as gcount() which triggers exception.

Tickets:

Signed-off-by: Pawel Raasz <pawel.raasz@intel.com>
@praasz praasz added this to the 2026.1 milestone Feb 2, 2026
@praasz praasz requested a review from a team as a code owner February 2, 2026 08:22
@github-actions github-actions bot added the category: Core OpenVINO Core (aka ngraph) label Feb 2, 2026
Co-authored-by: Roman Kazantsev <roman.kazantsev@intel.com>
@praasz praasz requested a review from rkazants February 2, 2026 10:49
@praasz praasz mentioned this pull request Feb 6, 2026
@olpipi olpipi added this pull request to the merge queue Feb 6, 2026
Merged via the queue into openvinotoolkit:master with commit b5901db Feb 6, 2026
233 of 235 checks passed
@praasz praasz deleted the coverity/overflowed-integer-argument-in-tensor-read branch February 16, 2026 06:14
Naseer-010 pushed a commit to Naseer-010/openvino that referenced this pull request Feb 18, 2026
### Details:
- Fix overflow by implicit cast to signed in read file as tensor by
explicit convert.
- If cast result is negative the file `read` return `0` as `gcount()`
which triggers exception.

### Tickets:
 - CVS-180108

---------

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

Labels

category: Core OpenVINO Core (aka ngraph)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants