Re: [PATCH 1/3] ntdll: NtReadFileScatter results are reported asynchronously

[Jacek Caban]

Hi Andrew,

On 30.11.2017 18:23, Andrew Eikum wrote:
> diff --git a/dlls/ntdll/file.c b/dlls/ntdll/file.c
> index ca2afa0e89..12b1d0f6c5 100644
> --- a/dlls/ntdll/file.c
> +++ b/dlls/ntdll/file.c
> @@ -1105,7 +1105,7 @@ NTSTATUS WINAPI NtReadFileScatter( HANDLE file, HANDLE event, PIO_APC_ROUTINE ap
>  if (send_completion) NTDLL_AddCompletion( file, cvalue, status, total );
>
> - return status;
> - return STATUS_PENDING;

Are you sure it should always be a case? Your tests use overlaped file
handle, I guess that's what makes differences. At least that's what I've
seen with other similar APIs (eg. recent Windows versions always return
STATUS_PENDING for ioctls if handle is overlapped).

Thanks,

Jacek