> On Sep 11, 2019, at 4:37 AM, Christos Zoulas <chris...@astron.com> wrote: > > The implementation I posted requires O_EXEC because this is the only way > to set FEXEC which is part of the check_exec() tests. Of course we can > elide this test and not require it in the fd case. It just seems nicely > symmetric to me the way it is now.
It may be "nicely symmetrical", but it's not a correct implementation. O_RDONLY (without O_EXEC) is explicitly allowed. -- thorpej
