winmm/tests: Initialize whole MIDIHDR struct before using it.

Saulius Krasuckas saulius2 at ar.fi.lt
Fri Oct 1 08:39:18 CDT 2010


I am not sure whether it's better to initialize every field separately.  
So I used memset().


S.

---
 dlls/winmm/tests/midi.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/dlls/winmm/tests/midi.c b/dlls/winmm/tests/midi.c
index e1f5104..972899e 100644
--- a/dlls/winmm/tests/midi.c
+++ b/dlls/winmm/tests/midi.c
@@ -127,6 +127,7 @@ static void test_midiIn_device(UINT udev, HWND hwnd)
 
     test_notification(hwnd, "midiInOpen", MIM_OPEN, 0);
 
+    memset(&mhdr, 0, sizeof(mhdr));
     mhdr.dwFlags = 0;
     mhdr.dwUser = 0x56FA552C;
     mhdr.dwBufferLength = 70000; /* > 64KB! */
@@ -283,6 +284,7 @@ static void test_midiOut_device(UINT udev, HWND hwnd)
         if (!rc) Sleep(400); /* Hear note */
     }
 
+    memset(&mhdr, 0, sizeof(mhdr));
     mhdr.dwFlags = 0;
     mhdr.dwUser   = 0x56FA552C;
     mhdr.dwOffset = 0xDEADBEEF;
@@ -407,6 +409,7 @@ static void test_midiStream(UINT udev, HWND hwnd)
     ok(!rc, "midiStreamProperty TIMEDIV rc=%s\n", mmsys_error(rc));
     todo_wine ok(24==LOWORD(midiprop.tdiv.dwTimeDiv), "default stream time division %u\n", midiprop.tdiv.dwTimeDiv);
 
+    memset(&mhdr, 0, sizeof(mhdr));
     mhdr.dwFlags = 0;
     mhdr.dwUser   = 0x56FA552C;
     mhdr.dwOffset = 1234567890;
-- 
1.5.6.5




More information about the wine-patches mailing list