[PATCH v2 1/4] localspl: Remove useless NULL checks from AddMonitor trace.

Dmitry Timoshkov dmitry at baikal.ru
Thu Sep 5 05:31:33 CDT 2019


AddMonitor dereferences pMonitors later without any checks.

Signed-off-by: Dmitry Timoshkov <dmitry at baikal.ru>
---
 dlls/localspl/provider.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/dlls/localspl/provider.c b/dlls/localspl/provider.c
index cb31b194ae..ccd19811cf 100644
--- a/dlls/localspl/provider.c
+++ b/dlls/localspl/provider.c
@@ -1475,9 +1475,7 @@ static BOOL WINAPI fpAddMonitor(LPWSTR pName, DWORD Level, LPBYTE pMonitors)
 
     mi2w = (LPMONITOR_INFO_2W) pMonitors;
     TRACE("(%s, %d, %p): %s %s %s\n", debugstr_w(pName), Level, pMonitors,
-            debugstr_w(mi2w ? mi2w->pName : NULL),
-            debugstr_w(mi2w ? mi2w->pEnvironment : NULL),
-            debugstr_w(mi2w ? mi2w->pDLLName : NULL));
+        debugstr_w(mi2w->pName), debugstr_w(mi2w->pEnvironment), debugstr_w(mi2w->pDLLName));
 
     if (copy_servername_from_name(pName, NULL)) {
         FIXME("server %s not supported\n", debugstr_w(pName));
-- 
2.20.1




More information about the wine-devel mailing list