[PATCH 1/2] ntdll: Set socket IO status before calling server if operation is completed.

[Zebediah Figura (she/her)]

On 12/16/21 12:43, Paul Gofman wrote:
> Signed-off-by: Paul Gofman <pgofman at codeweavers.com>
> ---
>      If the operation is completed before the server call then the completion
>      will be sent during the server call and another thread may receive it
>      before the iosb status is set. It looks like we shouldn't normally have failure
>      status from server once our operation already succeeded.

That doesn't sound right. The server shouldn't queue completion during 
the recv_socket server call, but rather during the subsequent wait on 
the async object [i.e. wait_async()]. In that case we should have 
already filled the IOSB.