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