kernel32/tests: check return value of alarmThread in test_NamedPipe_2() (try 2)

Andrew Nguyen arethusa26 at gmail.com
Sat May 22 09:44:11 CDT 2010


On Sat, May 22, 2010 at 5:02 AM, Austin English <austinenglish at gmail.com> wrote:
> Changes from try 1:
> Fix another typo.
> Set LastError before checking it
> Print LastError as a decimal
> Don't check for INVALID_HANDLE_VALUE as a return value of
> CreateThread, instead make sure serverThread is not NULL.
>
> --
> -Austin
>
>
>
>

You're calling SetLastError(0xdeadbeef) at the wrong time in the pipe
server subtests. If CreateThread fails, you will unconditionally print
the decimal representation of 0xdeadbeef in the failure messages.
SetLastError has to be called before CreateThread to ensure that the
failure messages print a last error that the preceding CreateThread
call sets.



More information about the wine-devel mailing list