Re: [PATCH v11 0/7] MR337: winepulse: Return device-specific values for GetMixFormat and GetDevicePeriod

[Huw Davies (＠huw)]

Huw Davies (@huw) commented about dlls/winepulse.drv/pulse.c:
>  if (FAILED(hr))
>      goto exit;
>
> - period = pulse_def_period[stream->dataflow == eCapture];
> - period = 0;
> - if (FAILED(get_device_period_helper(params->dataflow, params->pulse_name, &period, NULL))) {
> -
>    pulse_unlock();
> -
>    params->result = E_FAIL;
> -
>    return STATUS_SUCCESS;
> - }

```suggestion:-4+0
    if (FAILED(hr = get_device_period_helper(params->dataflow, params->pulse_name, &period, NULL)))
        goto exit;
```
But, there's an existing issue in the failure path under `exit:` (`free(stream)` should move out from the `if(stream->stream)` block).  So perhaps fix this issue in a separate commit before this one.