tree 5ab2b61abda89577aa74d5713957976c3f551c6a
parent 3bf0d02f2817c48b6ee61a95b52a6d76ad543be9
author Alexey Izbyshev <izbyshev@ispras.ru> 1604208788 +0300
committer GitHub <noreply@github.com> 1604208788 -0700
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJfnkiUCRBK7hj4Ov3rIwAAdHIIAGbDoYF9X40g+DdTfGHyjzjr
 qoYU/aV+2Dm9uO8cETq/+N8wHayOnYng3f9RWPpZVZhQIaeoa/aqzQClIWXqLFjt
 B2P7Fr96pdTDucWvaAyx+6FKQDFMJKe1sqy54G9lmbGEv7LsaigSrAvv0w2z5obz
 qcpmynNOYlBtEJQE/kUZBmwMH8fqwoSDqlcVPRPGyrk4UC5Mx38ngl+8NRGpS3k+
 VFIOjBe6Yy0en5a9zEmW2IFZdpqSVXtVzy7BrU2uHkYqOREuUbtt+UBtgXyrb079
 VjnRpTlMrKihZ/Fn4TRxKdM/p8GYu38+Pbfb2xpZWfyW7WlY26JgFPY7d0jBrJU=
 =RUcA
 -----END PGP SIGNATURE-----
 

bpo-42146: Unify cleanup in subprocess_fork_exec() (GH-22970)

* bpo-42146: Unify cleanup in subprocess_fork_exec()

Also ignore errors from _enable_gc():
* They are always suppressed by the current code due to a bug.
* _enable_gc() is only used if `preexec_fn != None`, which is unsafe.
* We don't have a good way to handle errors in case we successfully
  created a child process.

Co-authored-by: Gregory P. Smith <greg@krypto.org>