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
Possible reference leak in MAKE_FUNCTION #71178
Comments
*names* is not Py_DECREFed in MAKE_FUNCTION in error case. |
Sorry, wrong patch. Attach the right one. |
LGTM. Would be nice to have a test (just compile a function with annotations in a loop). |
I'm willing to. But I don't know where is the suitable place to put such a test and since the reference leak only appears in error case, can compile a function with annotations in a loop accomplish the test goal? |
I don't know what is the best place for this function. You can just break compiling functions with annotations, run tests, found tests that use functions with annotations, and add new test near one of them. You can test your test by removing the fix and running the test with the "-R :" option. |
New changeset eaa3a71a6f62 by Benjamin Peterson in branch '3.5': New changeset 7270701cf5bc by Benjamin Peterson in branch 'default': |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: