Message379529
> 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. |
|
Date |
User |
Action |
Args |
2020-10-24 17:29:41 | izbyshev | set | recipients:
+ izbyshev, gregory.p.smith, ronaldoussoren, koobs, pablogsal, Yonatan Goldschmidt |
2020-10-24 17:29:41 | izbyshev | set | messageid: <1603560581.04.0.591683874199.issue35823@roundup.psfhosted.org> |
2020-10-24 17:29:41 | izbyshev | link | issue35823 messages |
2020-10-24 17:29:40 | izbyshev | create | |
|