WinXP winsock test broken by commit e39dca6af6c87a30ab1d907c0468de2602a6e442

[Kai Blin]

The tests added in patch
commit e39dca6af6c87a30ab1d907c0468de2602a6e442
Author: Damjan Jovanovic 
Date:   Thu Mar 22 08:03:28 2007 +0200

ws2_32: WSASendTo should always re-enable the FD_WRITE event.

break the ws2_32 sock.c tests on WinXP SP2.
sock.c:1832: Test failed: WaitForSingleObject failed, error 258

I'm afraid I don't really know what's going wrong there.

Cheers,
Kai

[Damjan Jovanovic]

On 6/28/07, Kai Blin  wrote:
> The tests added in patch
> commit e39dca6af6c87a30ab1d907c0468de2602a6e442
> Author: Damjan Jovanovic 
> Date:   Thu Mar 22 08:03:28 2007 +0200
> ws2_32: WSASendTo should always re-enable the FD_WRITE event.
>
> break the ws2_32 sock.c tests on WinXP SP2.
> sock.c:1832: Test failed: WaitForSingleObject failed, error 258

Really strange, I remember that test passing when I wrote it.
> I'm afraid I don't really know what's going wrong there.

I'll run the tests myself this weekend.
> Cheers,
> Kai
>
> --
> Kai Blin
> WorldForge developer  http://www.worldforge.org/
> Wine developer        http://wiki.winehq.org/KaiBlin
> Samba team member     http://www.samba.org/samba/team/
> --
> Will code for cotton.

Damjan

WinXP winsock test broken by commit e39dca6af6c87a30ab1d907c0468de2602a6e442

[Paul Vriens]

Damjan Jovanovic wrote:
> On 6/28/07, Kai Blin  wrote:
> > The tests added in patch
> > commit e39dca6af6c87a30ab1d907c0468de2602a6e442
> > Author: Damjan Jovanovic 
> > Date:   Thu Mar 22 08:03:28 2007 +0200
> > ws2_32: WSASendTo should always re-enable the FD_WRITE event.
> >
> > break the ws2_32 sock.c tests on WinXP SP2.
> > sock.c:1832: Test failed: WaitForSingleObject failed, error 258
>
> Really strange, I remember that test passing when I wrote it.
> > I'm afraid I don't really know what's going wrong there.
>
> I'll run the tests myself this weekend.
> > Cheers,
> > Kai
> >
> > -- 
> > Kai Blin
> > WorldForge developer  http://www.worldforge.org/
> > Wine developer        http://wiki.winehq.org/KaiBlin
> > Samba team member     http://www.samba.org/samba/team/
> > -- 
> > Will code for cotton.
>
> Damjan

Briefly looking at http://test.winehq.org/data agrees with Kai's findings. Ever 
since the March 23 version of winetest, this test is failing for all NT4+ systems.

Cheers,

PAul.