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.

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

moved array sanity check to before the array access #117

Merged
merged 1 commit into from
May 13, 2022
Merged

moved array sanity check to before the array access #117

merged 1 commit into from
May 13, 2022

Conversation

RealAstolfo
Copy link
Contributor

Fix referring to #110

@CLAassistant
Copy link

CLAassistant commented May 12, 2022 •
edited

CLA assistant check
All committers have signed the CLA.

mmaneetsingh
mmaneetsingh approved these changes May 12, 2022
View reviewed changes
Copy link
Collaborator

@mmaneetsingh mmaneetsingh left a comment

Choose a reason for hiding this comment

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

Thanks for contributing this change.
Change looks good, taking it in the internal development branch.

@mmaneetsingh mmaneetsingh merged commit a947dc4 into NVIDIA:main May 13, 2022
@mmaneetsingh
Copy link
Collaborator

Merged. Thanks for the contribution.

@RealAstolfo RealAstolfo deleted the array-index-ub-change branch May 14, 2022 02:34
@PAR2020 PAR2020 added the Implemented Fixed, in test prior to release integration label May 24, 2022
@PAR2020 PAR2020 added this to the 515.48.07 milestone May 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mmaneetsingh mmaneetsingh mmaneetsingh approved these changes

Assignees
No one assigned
Labels
Implemented Fixed, in test prior to release integration
Projects
None yet
Milestone
515.48.07
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@RealAstolfo @CLAassistant @mmaneetsingh @PAR2020