<div dir="ltr">Thanks, I should read the doc's a little harder...</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 5, 2018 at 1:16 PM, Zebediah Figura <span dir="ltr"><<a href="mailto:z.figura12@gmail.com" target="_blank">z.figura12@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 05/03/18 12:53, Tom Watson wrote:<br>
><br>
> Over the weekend I seem to have triggered a number of segfaults while<br>
> writing tests by sending 0 as the optional bytes written parameter;<br>
><br>
>    WriteFile(handle, buffer, sizeof(buffer), 0, NULL);<br>
><br>
> Test patch/runs on testbot<br>
> here: <a href="https://testbot.winehq.org/JobDetails.pl?Key=36430" rel="noreferrer" target="_blank">https://testbot.winehq.<wbr>org/JobDetails.pl?Key=36430</a> <br>
><br>
> I download the resulting .exe and ran it against current git source<br>
> locally and it works fine.<br>
><br>
> Is this a known issue? Something going on with testbot? Something I'm<br>
> doing wrong? Or does it require some more investigation?<br>
><br>
> Cheers.<br>
><br>
><br>
><br>
<br>
</div></div>Hello Tom,<br>
<br>
The testbot runs each test against a Windows machine. On Windows, it's<br>
not allowed to pass a NULL parameter in for the number of bytes written,<br>
unless the overlapped parameter is used:<br>
<br>
"This parameter can be NULL only when the lpOverlapped parameter is not<br>
NULL."<br>
<br>
That this works on Wine is not really correct behaviour, but it's not<br>
worth "fixing" either.<br>
<br>
ἔρρωσο,<br>
Zeb<br>
<br>
<br>
</blockquote></div><br></div>