Skip to content

BUG: Decrement ref count in gentype_reduce if allocated memory not used#22557

Merged
seberg merged 1 commit intonumpy:mainfrom
aayushagrawal135:gentype-reduce-mem-leak
Nov 9, 2022
Merged

BUG: Decrement ref count in gentype_reduce if allocated memory not used#22557
seberg merged 1 commit intonumpy:mainfrom
aayushagrawal135:gentype-reduce-mem-leak

Conversation

@aayushagrawal135
Copy link
Copy Markdown
Contributor

Resolves #21495

Done as part of PyData NYC 2022.

@aayushagrawal135
Copy link
Copy Markdown
Contributor Author

@seberg @ganesh-k13 Could you please review the pull request? Thank you.

@charris charris added the 09 - Backport-Candidate PRs tagged should be backported label Nov 9, 2022
@seberg
Copy link
Copy Markdown
Member

seberg commented Nov 9, 2022

Thanks, just had to have a close enough look with the refcounting glasses 🧮 👓 . Looks all good though, thanks @aayushagrawal135.
None of the code-paths can be reasonably hit, so no point in new tests unfortunately.

@seberg seberg merged commit 1aa73ca into numpy:main Nov 9, 2022
@aayushagrawal135
Copy link
Copy Markdown
Contributor Author

Yay🎉

Thanks for taking a look and merging it!

@charris charris changed the title BUG: Decrement ref count in gentype_reduce when allocated memory not … BUG: Decrement ref count in gentype_reduce if allocated memory not used Nov 15, 2022
@charris charris removed the 09 - Backport-Candidate PRs tagged should be backported label Nov 15, 2022
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.

BUG: Potential memory leak in function gentype_reduce (a static analyzer report)

3 participants