<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Apr 22, 2022 at 12:30 AM Rob Landley <<a href="mailto:rob@landley.net">rob@landley.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 4/21/22 19:58, enh via Toybox wrote:<br>
> O_CLOEXEC is the default in toybox, so although the code as written<br>
> _looked_ correct, we need to |O_CLOEXEC to have the fd survive the<br>
> exec().<br>
<br>
Oops.<br>
<br>
> Found by folks building the kernel with toybox.<br>
<br>
Yay!<br>
<br>
I'm building the kernel with toybox for a bunch of architectures here, and<br>
didn't see it. I'm curious what context the breakage was in?<br></blockquote><div><br></div><div>yeah, sorry, i did add the link to the commit message but it was so long i took it back out again. aiui they were trying this:</div><div><br></div><div><a href="https://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-kbuild.git/commit/scripts/gen_autoksyms.sh?h=kbuild&id=2f6b64906adf99b4c5ea9356df793766d290cfb4">https://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-kbuild.git/commit/scripts/gen_autoksyms.sh?h=kbuild&id=2f6b64906adf99b4c5ea9356df793766d290cfb4</a><br></div><div><br></div><div>if you're thinking "i thought Android didn't allow kernel modules", that _was_ true, but part of the Generic Kernel Image (GKI) stuff is that every device should have the same kernel ... with modules being the escape hatch. (which then means everyone working in that area is worried about ABI one way or another!)</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Thanks,<br>
<br>
Rob<br>
</blockquote></div></div>