Skip to content

[query] local_to_global with bad data will read uninitialized memory #13479

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.

Sign up for GitHub

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

Jump to bottom
Closed
chrisvittal opened this issue Aug 22, 2023 · 0 comments · Fixed by #13485
Closed

[query] local_to_global with bad data will read uninitialized memory #13479

chrisvittal opened this issue Aug 22, 2023 · 0 comments · Fixed by #13485
Assignees
@chrisvittal
Labels
bug

Comments

@chrisvittal
Copy link
Collaborator

https://hail.zulipchat.com/#narrow/stream/123010-Hail-Query-0.2E2-support/topic/.E2.9C.94.20vds.20LAD.20to.20AD/near/386687039
@chrisvittal chrisvittal self-assigned this Aug 22, 2023
@danking danking changed the title local_to_global with bad data will read uninitialized memory [query] local_to_global with bad data will read uninitialized memory Aug 23, 2023
@danking danking added the bug label Aug 23, 2023
chrisvittal added a commit to chrisvittal/hail that referenced this issue Aug 23, 2023
@chrisvittal
[query] Add bounds checks to local_to_global
fc08b10 
Make sure that all local alleles are less than the n_total_alleles
parameter passed to the function.

This error will often occur if a vds is split with regular split_multi
rather than vds.split_multi

Closes hail-is#13479
@chrisvittal chrisvittal mentioned this issue Aug 23, 2023
Merged
danking pushed a commit that referenced this issue Aug 23, 2023
@chrisvittal
[query] Add bounds checks to local_to_global (#13485)
90886da 
Make sure that all local alleles are less than the n_total_alleles
parameter passed to the function.

This error will often occur if a vds is split with regular split_multi
rather than vds.split_multi

Closes #13479
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chrisvittal chrisvittal

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[query] Add bounds checks to local_to_global chrisvittal/hail
2 participants
@danking @chrisvittal