Skip to content

Conversation

sshekhar563
Copy link

This PR fixes a segmentation fault in the dequantize merge optimization pass.
The issue was caused by an out-of-bounds memory read when comparing two memory buffers of different sizes. This change adds a guard condition to check that the buffer sizes are equal before attempting to compare their contents.

Fixes #32295

@sshekhar563 sshekhar563 requested review from a team as code owners October 6, 2025 14:30
@github-actions github-actions bot added the category: GPU OpenVINO GPU plugin label Oct 6, 2025
@sys-openvino-ci sys-openvino-ci added the ExternalPR External contributor label Oct 6, 2025
@p-durandin
Copy link
Contributor

build_jenkins

Copy link
Contributor

@Lyamin-Roman Lyamin-Roman left a comment

Choose a reason for hiding this comment

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

Please, remove extra spaces to fix cpplint
src/plugins/intel_gpu/src/graph/graph_optimizer/prepare_quantization.cpp:375: Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]

Also, there is no need to delete line 387

@Lyamin-Roman Lyamin-Roman changed the title Fix: Prevent out-of-bounds read in dequantize merge [GPU] Fix: Prevent out-of-bounds read in dequantize merge Oct 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: GPU OpenVINO GPU plugin ExternalPR External contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Segmentation fault due to read beyond allocated buffer in GPU plugin graph otimizer
4 participants