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

[Andrew Eikum]

On Tue, Dec 05, 2017 at 05:43:00PM +0100, Alexandre Julliard wrote:
> Andrew Eikum <aeikum(a)codeweavers.com> 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.

I see what you mean. I'll try to come up with some tests for that,
thanks.

Andrew