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

A negative size in one of the split sizes allowed the computed size o… #52701

Merged
merged 1 commit into from Oct 28, 2021
Merged

A negative size in one of the split sizes allowed the computed size o… #52701

merged 1 commit into from Oct 28, 2021

Conversation

pranve
Copy link

@pranve pranve commented Oct 27, 2021

…f another

to exceed the total dimension, leading to a segfault and security vulnerability.
Adding a check for negative sizes prevents this.

PiperOrigin-RevId: 401035665
Change-Id: I79bbe329787dac82aa4bf60397a9129b716aedab

…f another

to exceed the total dimension, leading to a segfault and security vulnerability.
Adding a check for negative sizes prevents this.

PiperOrigin-RevId: 401035665
Change-Id: I79bbe329787dac82aa4bf60397a9129b716aedab
@google-cla google-cla bot added the cla: yes label Oct 27, 2021
@gbaned gbaned self-assigned this Oct 27, 2021
@gbaned gbaned added comp:core issues related to core part of tensorflow size:S CL Change Size: Small labels Oct 27, 2021
@gbaned gbaned added this to Assigned Reviewer in PR Queue via automation Oct 27, 2021
@gbaned gbaned assigned mihaimaruseac and unassigned gbaned Oct 27, 2021
PR Queue automation moved this from Assigned Reviewer to Approved by Reviewer Oct 28, 2021
@mihaimaruseac mihaimaruseac merged commit 74355ba into tensorflow:r2.6 Oct 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes comp:core issues related to core part of tensorflow size:S CL Change Size: Small
Projects
PR Queue
  
Approved by Reviewer
Development

Successfully merging this pull request may close these issues.

None yet

4 participants