<div><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 27 Apr 2021 at 15:38, Dmitry Timoshkov <<a href="mailto:dmitry@baikal.ru">dmitry@baikal.ru</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)">Dmitry Timoshkov <<a href="mailto:dmitry@baikal.ru" target="_blank">dmitry@baikal.ru</a>> wrote:<br>
<br>
> "Zebediah Figura (she/her)" <<a href="mailto:zfigura@codeweavers.com" target="_blank">zfigura@codeweavers.com</a>> wrote:<br>
> <br>
> > >> You can also see the results at:<br>
> > >> <a href="https://testbot.winehq.org/JobDetails.pl?Key=89319" rel="noreferrer" target="_blank">https://testbot.winehq.org/JobDetails.pl?Key=89319</a><br>
> > > <br>
> > > Same patch sent separately runs without failures:<br>
> > > <a href="https://testbot.winehq.org/JobDetails.pl?Key=89342" rel="noreferrer" target="_blank">https://testbot.winehq.org/JobDetails.pl?Key=89342</a><br>
> > > <br>
> > <br>
> > The tests crash with STATUS_HEAP_CORRUPTION, which strikes me as <br>
> > something that could cause inconsistent behaviour. Are you sure that <br>
> > you're not introducing a failure here?<br>
> <br>
> Thanks for the hint. Running with warn+heap helped to find the reason<br>
> of the heap corruption, it's 'p[GlobalSize(global)] = 0;' statements.<br>
> This revealed another bug in msxml3 - it doesn't NUL terminate the stream on<br>
> ::save(). After fixing that and removing 'p[] = 0' the processing instruction<br>
> tests stopped to crash.<br>
> <br>
> P.S.<br>
> warn+heap still makes msxml3 domdoc tests crash, however that looks like<br>
> another bug somewhere.<br>
<br>
That another crash happens at the end of test_get_ownerDocument() because<br>
priv->properties of doc1 and doc_owner have the same pointer value, and<br>
releasing doc_owner after doc1 leads to a double free and heap corruption.<br>
<br>
Nikolay, could you please have a look?</blockquote><div dir="auto"><br></div><div dir="auto">I won’t have time for that in coming days. </div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)" dir="auto"><br>
<br>
-- <br>
Dmitry.<br>
<br>
</blockquote></div></div>