Re: [PATCH v3 0/1] MR10966: dsound: Fail IDirectSoundCaptureBufferImpl_Lock() for invalid buffer range.

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/dsound/tests/capture.c:
> +    len1 = len2 = 0xdeadbeef;
> +    rc=IDirectSoundCaptureBuffer_Lock(state->dscbo,(DWORD)-32,0,&ptr1,&len1,&ptr2,&len2,0);
> +    ok(rc==DSERR_INVALIDPARAM,"IDirectSoundCaptureBuffer_Lock() failed: %08lx\n", rc);
> +    ok(!ptr1 && !ptr2 && !len1 && !len2, "got %p, %lu, %p, %lu.\n", ptr1, len1, ptr2, len2);
> +
> +    ptr1 = ptr2 = (void *)0xdeadbeef;
> +    len1 = len2 = 0xdeadbeef;
> +    rc=IDirectSoundCaptureBuffer_Lock(state->dscbo,(DWORD)-32,64,&ptr1,&len1,&ptr2,&len2,0);
> +    ok(rc==DSERR_INVALIDPARAM,"IDirectSoundCaptureBuffer_Lock() returned: %08lx\n", rc);
> +    ok(!ptr1 && !ptr2 && !len1 && !len2, "got %p, %lu, %p, %lu.\n", ptr1, len1, ptr2, len2);
> +
> +    ptr1 = ptr2 = (void *)0xdeadbeef;
> +    len1 = len2 = 0xdeadbeef;
> +    rc=IDirectSoundCaptureBuffer_Lock(state->dscbo,(DWORD)-32,64,&ptr1,&len1,&ptr2,&len2,0);
> +    ok(rc==DSERR_INVALIDPARAM,"IDirectSoundCaptureBuffer_Lock() returned: %08lx\n", rc);
> +    ok(!ptr1 && !ptr2 && !len1 && !len2, "got %p, %lu, %p, %lu.\n", ptr1, len1, ptr2, len2);
> These two tests look identical.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10966#note_141662