Skip to content

Fixed freeing unallocated pointer when resizing with height too large#4116

Merged
hugovk merged 1 commit intopython-pillow:masterfrom
radarhere:memory
Nov 13, 2019
Merged

Fixed freeing unallocated pointer when resizing with height too large#4116
hugovk merged 1 commit intopython-pillow:masterfrom
radarhere:memory

Conversation

@radarhere
Copy link
Copy Markdown
Member

Resolves #4115

#3393 added two free lines that should not be present. bounds_vert and kk_vert are assigned by precompute_coeffs. If precompute_coeffs returns 0, then they were not assigned, and should not be freed.

I have added a test that fails without this PR to demonstrate.

@radarhere radarhere mentioned this pull request Oct 4, 2019
@hugovk hugovk merged commit b64d96d into python-pillow:master Nov 13, 2019
@radarhere radarhere deleted the memory branch November 13, 2019 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Freeing pointers

2 participants