Re: [3/3] ws2_32/tests: Get rid of the do_oob_send helper function

[Marvin]

Hi,

While running your changed tests on Windows, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?
Full results can be found at
https://newtestbot.winehq.org/JobDetails.pl?Key=5420

Your paranoid android.

=== wvistau64 (64 bit sock) ===
sock: unhandled exception c0000005 at 000007FEFF0720C0

=== w2008s64 (64 bit sock) ===
sock: unhandled exception c0000005 at 000007FEFE4220C0

[3/3] ws2_32/tests: Get rid of the do_oob_send helper function

[Bruno Jesus]

On Wed, Mar 5, 2014 at 10:24 PM, Marvin  wrote:
> === wvistau64 (64 bit sock) ===
> sock: unhandled exception c0000005 at 000007FEFF0720C0
>
> === w2008s64 (64 bit sock) ===
> sock: unhandled exception c0000005 at 000007FEFE4220C0

It looks like the crashes are not related to WSACleanup then... I
don't have any idea then... Maybe valgrind can help finding a leak or
something like that. It didn't happen in the old testbot but I'm not
sure these Windows versions where present there.

Regards,
Bruno

[3/3] ws2_32/tests: Get rid of the do_oob_send helper function

[André Hentschel]

Am 06.03.2014 13:35, schrieb Bruno Jesus:
> On Wed, Mar 5, 2014 at 10:24 PM, Marvin  wrote:
> > === wvistau64 (64 bit sock) ===
> > sock: unhandled exception c0000005 at 000007FEFF0720C0
> >
> > === w2008s64 (64 bit sock) ===
> > sock: unhandled exception c0000005 at 000007FEFE4220C0
>
> It looks like the crashes are not related to WSACleanup then... I
> don't have any idea then... Maybe valgrind can help finding a leak or
> something like that. It didn't happen in the old testbot but I'm not
> sure these Windows versions where present there.
>
> Regards,
> Bruno

Yeah it crashes anyway when ending:
https://testbot.winehq.org/JobDetails.pl?Key=5446
that seems odd...

OK... so i removed the Exit function and it works:
https://testbot.winehq.org/JobDetails.pl?Key=5449
Now i'll test if it's the TlsFree, the WSACleanup or something else...

[3/3] ws2_32/tests: Get rid of the do_oob_send helper function

[Bruno Jesus]

On Thu, Mar 6, 2014 at 7:46 PM, André Hentschel  wrote:
> OK... so i removed the Exit function and it works:
> https://testbot.winehq.org/JobDetails.pl?Key=5449
> Now i'll test if it's the TlsFree, the WSACleanup or something else...

I remember testing the TlsFree, I don't think it's that either.

[3/3] ws2_32/tests: Get rid of the do_oob_send helper function

[André Hentschel]

Am 07.03.2014 00:06, schrieb Bruno Jesus:
> On Thu, Mar 6, 2014 at 7:46 PM, André Hentschel  wrote:
> > OK... so i removed the Exit function and it works:
> > https://testbot.winehq.org/JobDetails.pl?Key=5449
> > Now i'll test if it's the TlsFree, the WSACleanup or something else...
>
> I remember testing the TlsFree, I don't think it's that either.

seems like it is:
https://testbot.winehq.org/JobDetails.pl?Key=5450
vs.
https://testbot.winehq.org/JobDetails.pl?Key=5451