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

From: Ethan Lee

Signed-off-by: Ethan Lee 
---
 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){

[Marvin]

Hi,

While running your changed tests on Windows, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?

Full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=48301

Your paranoid android.

=== debian9 (32 bit report) ===

Report errors:
The report seems to have been truncated

=== debian9 (32 bit WoW report) ===

Report errors:
The report seems to have been truncated

[Andrew Eikum]

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=46724
Signed-off-by: Andrew Eikum

On Tue, Feb 26, 2019 at 06:25:38PM -0500, [e-mail] wrote:
> From: Ethan Lee
> Signed-off-by: Ethan Lee
>
> 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