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

Prevent array write out-of-bounds. #49815

Merged
merged 1 commit into from May 27, 2021

Conversation

geetachavan1
Copy link
Contributor

If user passes an invalid axis, then we copy one too many dimensions to the output in the loop below these checks. Even if we didn't do that, there will be further issues with an invalid axis, so we check for that right now.

PiperOrigin-RevId: 371023299
Change-Id: I9eca37ffc2b29e8e48710f500701270ef0790224

If user passes an invalid axis, then we copy one too many dimensions to the output in the loop below these checks. Even if we didn't do that, there will be further issues with an invalid axis, so we check for that right now.

PiperOrigin-RevId: 371023299
Change-Id: I9eca37ffc2b29e8e48710f500701270ef0790224
@google-cla google-cla bot added the cla: yes label May 26, 2021
@gbaned gbaned self-assigned this May 27, 2021
@gbaned gbaned added comp:lite TF Lite related issues size:XS CL Change Size: Extra Small labels May 27, 2021
@gbaned gbaned added this to Assigned Reviewer in PR Queue via automation May 27, 2021
@gbaned gbaned requested a review from mihaimaruseac May 27, 2021 10:39
@gbaned gbaned assigned mihaimaruseac and unassigned gbaned May 27, 2021
PR Queue automation moved this from Assigned Reviewer to Approved by Reviewer May 27, 2021
@mihaimaruseac mihaimaruseac merged commit c427d9b into tensorflow:r2.2 May 27, 2021
PR Queue automation moved this from Approved by Reviewer to Merged May 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes comp:lite TF Lite related issues size:XS CL Change Size: Extra Small
Projects
PR Queue
  
Merged
Development

Successfully merging this pull request may close these issues.

None yet

3 participants