<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Mon, Apr 24, 2017 at 2:57 PM John Keeping <<a href="mailto:john@keeping.me.uk">john@keeping.me.uk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">As described in commit 2efb59e (ui-patch: Flush stdout after outputting<br>
data, 2014-06-11), we need to ensure that stdout is flushed before<br>
restoring the file descriptor when writing to the cache.  It turns out<br>
that it's not just ui-patch that is affected by this but also raw diff<br>
which writes to stdout internally.<br>
<br>
Let's avoid risking more places doing this by ensuring that stdout is<br>
flushed after writing in fill_slot().<br></blockquote><div><br></div><div>Thanks, John, this is working and fixes the truncation bug for both patches and rawdiffs.</div><div><br></div><div>-Konstantin</div></div></div><div dir="ltr">-- <br></div><div data-smartmail="gmail_signature"><div dir="ltr"><div>Konstantin Ryabitsev<br></div>Director, IT Infrastructure Security<div>The Linux Foundation<div>+1-971-258-2363</div></div></div></div>