bpo-42085: Fix memory leak introduced by GH-22780

[asvetlov]

./python -m test test_asyncgen -R 3:3 -m test.test_asyncgen.AsyncGenAsyncioTest.test_async_gen_asyncio_03 doesn't report about leaks now.

https://bugs.python.org/issue42085

[asvetlov]

@vstinner FYI: I have a fix for the memory leak.
Thank you very much for pointing on the problem.
Would you review this PR (in case if it is urgent) or wait for @1st1 and others?

@vladima you would be interested in this PR as the author of the initial change.

[1st1]

Thanks so much, Andrew!

[asvetlov]

Welcome!

[bedevere-bot]

⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️

Hi! The buildbot s390x RHEL8 LTO 3.x has failed when building commit cda99b4.

What do you need to do:

1. Don't panic.
2. Check the buildbot page in the devguide if you don't know what the buildbots are or how they work.
3. Go to the page of the buildbot that failed (https://buildbot.python.org/all/#builders/567/builds/972) and take a look at the build logs.
4. Check if the failure is related to this commit (cda99b4) or if it is a false positive.
5. If the failure is related to this commit, please, reflect that on the issue and make a new Pull Request with a fix.

You can take a look at the buildbot page here:

https://buildbot.python.org/all/#builders/567/builds/972

Summary of the results of the build (if available):

== Tests result: ENV CHANGED ==

410 tests OK.

10 slowest tests:

• test_concurrent_futures: 2 min 52 sec
• test_peg_generator: 2 min 50 sec
• test_gdb: 1 min 41 sec
• test_multiprocessing_spawn: 1 min 12 sec
• test_nntplib: 1 min 6 sec
• test_multiprocessing_forkserver: 1 min 1 sec
• test_multiprocessing_fork: 54.9 sec
• test_signal: 49.5 sec
• test_asyncio: 48.2 sec
• test_tokenize: 35.1 sec

1 test altered the execution environment:
test_asyncio

14 tests skipped:
test_devpoll test_ioctl test_kqueue test_msilib test_nis
test_ossaudiodev test_startfile test_tix test_tk test_ttk_guionly
test_winconsoleio test_winreg test_winsound test_zipfile64

Total duration: 4 min 47 sec

Click to see traceback logs

Traceback (most recent call last):
File "/home/dje/cpython-buildarea/3.x.edelsohn-rhel8-z.lto/build/Lib/asyncio/sslproto.py", line 321, in __del__self.close()
File "/home/dje/cpython-buildarea/3.x.edelsohn-rhel8-z.lto/build/Lib/asyncio/sslproto.py", line 316, in closeself._ssl_protocol._start_shutdown()
File "/home/dje/cpython-buildarea/3.x.edelsohn-rhel8-z.lto/build/Lib/asyncio/sslproto.py", line 590, in _start_shutdownself._abort()
File "/home/dje/cpython-buildarea/3.x.edelsohn-rhel8-z.lto/build/Lib/asyncio/sslproto.py", line 731, in _abortself._transport.abort()
File "/home/dje/cpython-buildarea/3.x.edelsohn-rhel8-z.lto/build/Lib/asyncio/selector_events.py", line 680, in abortself._force_close(None)
File "/home/dje/cpython-buildarea/3.x.edelsohn-rhel8-z.lto/build/Lib/asyncio/selector_events.py", line 731, in _force_closeself._loop.call_soon(self._call_connection_lost, exc)
File "/home/dje/cpython-buildarea/3.x.edelsohn-rhel8-z.lto/build/Lib/asyncio/base_events.py", line 746, in call_soonself._check_closed()
File "/home/dje/cpython-buildarea/3.x.edelsohn-rhel8-z.lto/build/Lib/asyncio/base_events.py", line 510, in _check_closedraiseRuntimeError('Event loop is closed')
RuntimeError: Event loop is closed

[asvetlov]

I'm very sorry for the failure; it is not related to the change.

Heh, I should finish #17975 to fix this annoying sslproto issue.

[1st1]

Heh, I should finish #17975 to fix this annoying sslproto issue.

We're almost there!