[PATCH] xaudio2: Fix uninitialized variable access (Valgrind)

Andrew Eikum aeikum at codeweavers.com
Thu Feb 28 09:18:00 CST 2019


Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=46724
Signed-off-by: Andrew Eikum <aeikum at codeweavers.com>

On Tue, Feb 26, 2019 at 06:25:38PM -0500, elee at codeweavers.com wrote:
> From: Ethan Lee <elee at codeweavers.com>
> 
> Signed-off-by: Ethan Lee <elee at codeweavers.com>
> ---
>  dlls/xaudio2_7/compat.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/dlls/xaudio2_7/compat.c b/dlls/xaudio2_7/compat.c
> index 318f668589..bc33cb92ed 100644
> --- a/dlls/xaudio2_7/compat.c
> +++ b/dlls/xaudio2_7/compat.c
> @@ -923,7 +923,7 @@ static HRESULT WINAPI XA20SUB_SetOutputVoices(IXAudio20SubmixVoice *iface,
>          const XAUDIO23_VOICE_SENDS *pSendList)
>  {
>      XA2VoiceImpl *This = impl_from_IXAudio20SubmixVoice(iface);
> -    XAUDIO2_VOICE_SENDS sends, *psends;
> +    XAUDIO2_VOICE_SENDS sends, *psends = NULL;
>      HRESULT hr;
>  
>      TRACE("%p, %p\n", This, pSendList);
> @@ -1483,7 +1483,7 @@ static HRESULT WINAPI XA20M_SetOutputVoices(IXAudio20MasteringVoice *iface,
>          const XAUDIO23_VOICE_SENDS *pSendList)
>  {
>      XA2VoiceImpl *This = impl_from_IXAudio20MasteringVoice(iface);
> -    XAUDIO2_VOICE_SENDS sends, *psends;
> +    XAUDIO2_VOICE_SENDS sends, *psends = NULL;
>      HRESULT hr;
>  
>      TRACE("%p, %p\n", This, pSendList);
> @@ -2115,7 +2115,7 @@ static HRESULT WINAPI XA20_CreateSubmixVoice(IXAudio20 *iface,
>          const XAUDIO2_EFFECT_CHAIN *pEffectChain)
>  {
>      IXAudio2Impl *This = impl_from_IXAudio20(iface);
> -    XAUDIO2_VOICE_SENDS sends, *psends;
> +    XAUDIO2_VOICE_SENDS sends, *psends = NULL;
>      HRESULT hr;
>  
>      if(pSendList){
> @@ -2350,7 +2350,7 @@ static HRESULT WINAPI XA22_CreateSubmixVoice(IXAudio22 *iface,
>          const XAUDIO2_EFFECT_CHAIN *pEffectChain)
>  {
>      IXAudio2Impl *This = impl_from_IXAudio22(iface);
> -    XAUDIO2_VOICE_SENDS sends, *psends;
> +    XAUDIO2_VOICE_SENDS sends, *psends = NULL;
>      HRESULT hr;
>  
>      if(pSendList){
> -- 
> 2.17.2
> 
> 
> 



More information about the wine-devel mailing list