Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

relationals: fix out of bounds accesses in shuffle test #1961

Merged
merged 1 commit into from
Jun 4, 2024

Conversation

svenvh
Copy link
Member

@svenvh svenvh commented May 3, 2024

The values in src are indices into an array in get_order_string(). Not initializing src resulted in out of bounds accesses there.

It seems that when the out of bounds accesses happened, the result of get_order_string() was not actually used, so at least the test was not using random data. Fix the issue as it prevents a clean run of this test with e.g. AddressSanitizer.

The values in `src` are indices into an array in `get_order_string()`.
Not initializing `src` resulted in out of bounds accesses there.

It seems that when the out of bounds accesses happened, the result of
`get_order_string()` was not actually used, so at least the test was
not using random data.  Fix the issue as it prevents a clean run of
this test with e.g. AddressSanitizer.

Signed-off-by: Sven van Haastregt <sven.vanhaastregt@arm.com>
@neiltrevett neiltrevett merged commit a130f3d into KhronosGroup:main Jun 4, 2024
7 checks passed
@svenvh svenvh deleted the relationals-asan branch June 5, 2024 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants