Re: [PATCH 1/2] mmdevapi: IsFormatSupported fills closest match iff it returns S_FALSE.

[Octavian Voicu]

--- a/dlls/winealsa.drv/mmdevdrv.c+++ b/dlls/winealsa.drv/mmdevdrv.c>
@@ -1310,17 +1313,14 @@ exit:
>  LeaveCriticalSection(&This->lock);
>  HeapFree(GetProcessHeap(), 0, formats);
>
> - if(hr == S_OK || !out){
> -
>    CoTaskMemFree(closest);
> -
>    if(out)
> -
>        *out = NULL;
> - }else if(closest){
> - if(hr == S_FALSE && out) {
>     closest->nBlockAlign =
>         closest->nChannels * closest->wBitsPerSample / 8;
>     closest->nAvgBytesPerSec =
>         closest->nBlockAlign * closest->nSamplesPerSec;
>     *out = closest;
> - }
> - } else
> -
>    CoTaskMemFree(closest);

You are dereferencing 'closest' without checking if it's non-NULL. It
can be NULL if control jumps to the 'exit' label for several cases.
This only applies to winealsa; for wineoss there are no gotos.

Octavian