The following commit authors need to sign the Contributor License Agreement:

• indoor47@gmail.com
• sharkdp@users.noreply.github.com
• rvolosatovs@users.noreply.github.com

Most changes to Pythonrequire a NEWS entry. Add one using theblurb_it web app or theblurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply theskip news label instead.

Please:

• add regression tests and check that the test failed before your change
• remove blank lines in NEWS

By the way,@dr-carlos already suggested to open a PR. It is polite to ask them if they want to contribute themselves. As such, I'm going to close this one unless they are fine with you making the PR (we don't really want people "sniping" work of others)

Thanks for the feedback. I missed that@dr-carlos suggested to fix it.

I’m happy to close this PR if@dr-carlos is already working on it.

Thanks for the feedback. I missed that@dr-carlos suggested to fix it.

I’m happy to close this PR if@dr-carlos is already working on it.

Hi, thanks for asking!
I'm happy for you to continue with the PR :)

Here are the changes I made:

• I added akw local pointer, as a similar segfault happens for keywords
• I added anfn local pointer so thatrepr uses its original state when generating its final representation.
• I got rid of theerror goto and merged it with thedone goto as I needed to decrement the reference count offn,args, andkw, and I found that decrementing them in thedone goto was the easiest.
  Update: I changed the goto logic to reduce repetitive calls toPy_DECREF
• I added a test based on@Qanux's original code in issueheap-buffer-overflow in functools.partial.__repr__() #144475. I extended it to also check for changes in thefn andkw arguments.