Aric Stewart : winealsa: Don' t dereference a potentially NULL value in the WARN statement (Coverity 93).
Alexandre Julliard
julliard at winehq.org
Mon Oct 6 09:35:18 CDT 2008
Module: wine
Branch: master
Commit: 2cc258f6c97891afc6feaec1b8bbaba9e7b5755e
URL: http://source.winehq.org/git/wine.git/?a=commit;h=2cc258f6c97891afc6feaec1b8bbaba9e7b5755e
Author: Aric Stewart <aric at codeweavers.com>
Date: Sat Oct 4 13:03:17 2008 -0500
winealsa: Don't dereference a potentially NULL value in the WARN statement (Coverity 93).
---
dlls/winealsa.drv/midi.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/dlls/winealsa.drv/midi.c b/dlls/winealsa.drv/midi.c
index 068165e..c9710a2 100644
--- a/dlls/winealsa.drv/midi.c
+++ b/dlls/winealsa.drv/midi.c
@@ -1050,8 +1050,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