[PATCH] dbghelp: Add a missing NULL check (Coverity)

Marcus Meissner marcus at jet.franken.de
Tue Dec 8 17:01:29 CST 2009


Hi,

all other functions check the return value of process_find_by_handle,
so this function should do it too.

Ciao, Marcus
---
 dlls/dbghelp/symbol.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/dlls/dbghelp/symbol.c b/dlls/dbghelp/symbol.c
index c322a62..c266ce2 100644
--- a/dlls/dbghelp/symbol.c
+++ b/dlls/dbghelp/symbol.c
@@ -1034,6 +1034,7 @@ static BOOL sym_enum(HANDLE hProcess, ULONG64 BaseOfDll, PCSTR Mask,
     regex_t             mod_regex, sym_regex;
 
     pair.pcs = process_find_by_handle(hProcess);
+    if (!pair.pcs) return FALSE;
     if (BaseOfDll == 0)
     {
         /* do local variables ? */
-- 
1.5.6



More information about the wine-patches mailing list