This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author izbyshev
Recipients Yonatan Goldschmidt, gregory.p.smith, izbyshev, koobs, pablogsal, ronaldoussoren
Date 2020-10-24.17:29:40
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1603560581.04.0.591683874199.issue35823@roundup.psfhosted.org>
In-reply-to
Content
> Thank you for taking this on!  I'm calling it fixed for now as the buildbots are looking happy with it.  If issues with it arise we can address them.

Thank you for reviewing and merging!

Using POSIX_CALL for pthread_sigmask() is incorrect, however, since it *returns* the error instead of setting errno. I've opened a PR with a fix and additional handling of the first pthread_sigmask() call in the parent (which can be done easily).

I've also noticed a memory leak why doing the above: cwd_obj2 is not released on the error path. It probably slipped with patches adding user/groups support. I'll file a separate issue.

Would you also clarify why you disallowed vfork() in case setsid() is needed? Have you discovered any potential issues with setsid()? Note that setsid() doesn't suffer from thread sync issues like setuid()/setgid()/etc.
History
Date User Action Args
2020-10-24 17:29:41izbyshevsetrecipients: + izbyshev, gregory.p.smith, ronaldoussoren, koobs, pablogsal, Yonatan Goldschmidt
2020-10-24 17:29:41izbyshevsetmessageid: <1603560581.04.0.591683874199.issue35823@roundup.psfhosted.org>
2020-10-24 17:29:41izbyshevlinkissue35823 messages
2020-10-24 17:29:40izbyshevcreate