Paul Vriens : comdlg32: Move NULL check before use (Coverity).
Alexandre Julliard
julliard at wine.codeweavers.com
Sat Oct 7 11:32:47 CDT 2006
Module: wine
Branch: master
Commit: 06ec62d9c7ef16f74ea6473408dd21809cdb4d99
URL: http://source.winehq.org/git/wine.git/?a=commit;h=06ec62d9c7ef16f74ea6473408dd21809cdb4d99
Author: Paul Vriens <Paul.Vriens at xs4all.nl>
Date: Sat Oct 7 16:48:49 2006 +0200
comdlg32: Move NULL check before use (Coverity).
---
dlls/comdlg32/fontdlg16.c | 7 ++++---
1 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/dlls/comdlg32/fontdlg16.c b/dlls/comdlg32/fontdlg16.c
index a90dca2..3f3264f 100644
--- a/dlls/comdlg32/fontdlg16.c
+++ b/dlls/comdlg32/fontdlg16.c
@@ -174,12 +174,13 @@ BOOL16 WINAPI ChooseFont16(LPCHOOSEFONT1
LOGFONT16 *font16;
SEGPTR lpTemplateName;
- cf32w.lpLogFont=&lf32w;
- CFn_CHOOSEFONT16to32W(lpChFont, &cf32w);
-
TRACE("ChooseFont\n");
+
if (!lpChFont) return FALSE;
+ cf32w.lpLogFont=&lf32w;
+ CFn_CHOOSEFONT16to32W(lpChFont, &cf32w);
+
if (TRACE_ON(commdlg))
_dump_cf_flags(lpChFont->Flags);
More information about the wine-cvs
mailing list