Aric Stewart : wineoss: Don' t dereference a potentially NULL value in the WARN statement (Coverity 93).

Alexandre Julliard julliard at winehq.org
Tue Oct 7 08:54:27 CDT 2008


Module: wine
Branch: master
Commit: e3e9bc30fdd4889d97cc1a81f38d8a89fbcd93af
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=e3e9bc30fdd4889d97cc1a81f38d8a89fbcd93af

Author: Aric Stewart <aric at codeweavers.com>
Date:   Tue Oct  7 07:17:27 2008 -0500

wineoss: Don't dereference a potentially NULL value in the WARN statement (Coverity 93).

---

 dlls/wineoss.drv/midi.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/wineoss.drv/midi.c b/dlls/wineoss.drv/midi.c
index a7ec4fd..ade1928 100644
--- a/dlls/wineoss.drv/midi.c
+++ b/dlls/wineoss.drv/midi.c
@@ -1620,8 +1620,8 @@ static DWORD modPrepare(WORD wDevID, LPMIDIHDR lpMidiHdr, DWORD dwSize)
     if (dwSize < sizeof(MIDIHDR) || lpMidiHdr == 0 ||
 	lpMidiHdr->lpData == 0 || (lpMidiHdr->dwFlags & MHDR_INQUEUE) != 0 ||
 	lpMidiHdr->dwBufferLength >= 0x10000ul) {
-	WARN("%p %p %08x %d/%d\n", lpMidiHdr, lpMidiHdr->lpData,
-	           lpMidiHdr->dwFlags, sizeof(MIDIHDR), dwSize);
+	WARN("%p %p %08x %d/%d\n", lpMidiHdr, lpMidiHdr ? lpMidiHdr->lpData : NULL,
+	           lpMidiHdr ? lpMidiHdr->dwFlags : 0, sizeof(MIDIHDR), dwSize);
 	return MMSYSERR_INVALPARAM;
     }
 




More information about the wine-cvs mailing list