<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jul 19, 2022 at 6:57 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 7/18/22 19:02, enh wrote:>     ack. i tried to take an update but hit another<br>
-Werror=format-security issue<br>
>     with one of your diff.c changes. i've sent a patch (and a separate patch to<br>
>     add that -Werror= to the default toybox configure, since that's one we<br>
>     always have to fix in the end anyway; may as well catch them fresh?).<br>
> <br>
>     i'll try again tomorrow... (i want to try to use `timeout -i` too!)<br>
> <br>
> heh, this doesn't seem intentional (especially because it happens without `-i`),<br>
> but it wasn't obvious to me what the fix is?<br>
> <br>
> ~/toybox$ time ./toybox timeout -s SEGV 10 ./foo<br>
> timeout: exec ./foo: No such file or directory<br>
> <br>
> real 0m10.035s<br>
> user 0m0.009s<br>
> sys 0m0.013s<br>
<br>
It isn't? If the child process exits, the pipe should close? Even if SIGCHLD is<br>
delivered before we can set the signal handler, that shouldn't... zombie process...<br></blockquote><div><br></div><div>sorry, i was in a rush to shut my laptop for the day... the point was</div><div><br></div><div>real 0m10.035s<br></div><div><br></div><div>i don't think a failure should wait for the full timeout to exit. but it wasn't obvious to me where to put the missing exit. check whether the global exit status has been set after xpopen_both()? or should xpopen_both() be handling this?</div><div><br></div><div>i don't think this bug is a blocker, and i'll try to take an update today, but i noticed the first time i tried to use timeout (albeit via user error), so i likely won't be the last :-)</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">
Lemme get back to you on this. There's is an elderly cat.<br>
<br>
Rob<br>
</blockquote></div></div>