Re: [PATCH v3 1/3] ntdll: Report NtReadFileScatter results asynchronously

[Alexandre Julliard]

Andrew Eikum  writes:
> @@ -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;

Is it really correct to do this for error cases too?  In particular,
returning STATUS_PENDING for the non-overlapped case doesn't seem right.