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,0,0,&ptr1,&len1,NULL,NULL,0);
> +    ok(rc==DSERR_INVALIDPARAM,"IDirectSoundCaptureBuffer_Lock() returned: %08lx\n", rc);
> +    ok(!ptr1 && !len1, "got %p, %lu.\n", ptr1, len1);
> +
> +    ptr1 = ptr2 = (void *)0xdeadbeef;
> +    len1 = len2 = 0xdeadbeef;
> +    rc=IDirectSoundCaptureBuffer_Lock(state->dscbo,state->buffer_size,1,&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,state->buffer_size,1,&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);
> Same for these two. Did the second instance intend to test with NULL `ptr2` and `len2`?

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