Thanks for looking Jordan!<br><br><div class="gmail_quote"><div dir="ltr">On Mon, Sep 17, 2018, 11:16 AM Jordan Rupprecht <<a href="mailto:rupprecht@google.com">rupprecht@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> Interesting, what kind of failures?<br>
><br>
> If they are causing you problems, of course feel free to revert.<br>
><br>
> Dave<br>
<br>
Turns out they are all real issues which running the test under asan<br>
mode flags as global-buffer-overflow. I'm guessing the over-alignment<br>
was hiding the bug as reads there would be zero initialized, but now<br>
they are reading other non-zero data and crashing on that.<br>
<br>
So, this patch just exposes some buggy code. Let's not revert it.<br>
Sorry for the false alarm!<br>
<br>
-- Jordan<br>
</blockquote></div>