<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Feb 13, 2019, 03:53 Rob Landley <<a href="mailto:rob@landley.net">rob@landley.net</a> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 2/12/19 11:48 PM, enh via Toybox wrote:<br>
> Very few places actually check for errors from emit, and I actually see<br>
> the same endless loop from "sed (GNU sed) 4.4" on current Debian, so I'm<br>
> not sure this isn't Broken As Designed, but an endless loop spewing<br>
> "short write" (or saying nothing, in the case of GNU sed) really doesn't<br>
> feel like useful behavior in face of EPIPE, which really isn't going to<br>
> fix itself. Certainly not being able to run the sed tests to completion<br>
> is pretty annoying --- which is why, unless we remove this test as<br>
> invalid, we should probably also add a SKIP_HOST=1 to the "b loop" test.<br>
> <br>
> Note that even with this fix you'll see the error twice:<br>
> <br>
>   sed: short write: Broken pipe<br>
>   sed: short write: Broken pipe<br>
<br>
I'm not seeing this error, make test_sed currently runs to completion<br>
successfully for me? (Applied anyway, but how do I trigger what you're seeing in<br>
toybox? We don't seem to have that test in sed.tests?)<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Yeah, you do, it's the one called "b loop". I mentioned it on the list when it started happening. It's been completely reproducible for me on Debian for months now, but it did seem to come from nowhere. </div><div dir="auto"><br></div><div dir="auto">As I discovered last night, it's actually affecting GNU sed too, which strace shows just ignoring EPIPE from write(2) over and over. What happens for you (before this patch)? Death by SIGPIPE? </div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Rob<br>
</blockquote></div></div></div>