Re: [PATCH v5 0/7] MR1714: cryptowinrt: Implement IKeyCredentialManagerStatics_IsSupportedAsync.

[Mohamad Al-Jaf (＠maljaf)]

On Fri Dec  9 04:32:05 2022 +0000, Mohamad Al-Jaf wrote:
> I had to increase the timeout to 5000 ms because neither 1000 nor 2000
> ms were enough for the operation to complete, specifically for the
> `tmp_handler == NULL` check on line 301. It was failing randomly on the
> testbot VMs, 1 time on w10pro64_zh_CN and another on w10pro64_ja.
> - w10pro64_ja: https://testbot.winehq.org/JobDetails.pl?Key=127445
> - w10pro64_zh_CN: https://testbot.winehq.org/JobDetails.pl?Key=127443

Okay, it looks like 5000 ms wasn't enough either. I tested it several times with 5000 and it worked fine. Seems like there's still a race condition happening. Is my test still incorrect?

https://testbot.winehq.org/JobDetails.pl?Key=127451