[Rpm-maint] [rpm-software-management/rpm] lua: add rpm.execute() (#390)
Jeff Johnson
notifications at github.com
Wed Feb 21 16:29:50 UTC 2018
The real flaw with looping over close(fdno) before exec in rpm is that it masks the root cause of the problem: applications that are not setting FD_CLOEXEC.
The loop also prevents an application from intentionally passing an open fdno to a child of rpm (if that is/was intended).
I'd suggest removing the loop entirely (or logging leaky fdnos) so that applications can be fixed directly: it's not a difficult patch in most cases.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/390#issuecomment-367384963
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20180221/de5743d9/attachment.html>
More information about the Rpm-maint
mailing list