[4/4] kernel32: GetModuleHandleEx already clears the module handle on failure.

Hans Leidekker hans at codeweavers.com
Thu Apr 5 04:16:37 CDT 2012


I checked all uses of GetModuleHandleEx in Wine, none depend on the old behavior.
---
 dlls/kernel32/module.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/kernel32/module.c b/dlls/kernel32/module.c
index d4978cf..6b91a4c 100644
--- a/dlls/kernel32/module.c
+++ b/dlls/kernel32/module.c
@@ -570,7 +570,7 @@ HMODULE WINAPI DECLSPEC_HOTPATCH GetModuleHandleA(LPCSTR module)
 {
     HMODULE ret;
 
-    if (!GetModuleHandleExA( GET_MODULE_HANDLE_EX_FLAG_UNCHANGED_REFCOUNT, module, &ret )) ret = 0;
+    GetModuleHandleExA( GET_MODULE_HANDLE_EX_FLAG_UNCHANGED_REFCOUNT, module, &ret );
     return ret;
 }
 
@@ -583,7 +583,7 @@ HMODULE WINAPI GetModuleHandleW(LPCWSTR module)
 {
     HMODULE ret;
 
-    if (!GetModuleHandleExW( GET_MODULE_HANDLE_EX_FLAG_UNCHANGED_REFCOUNT, module, &ret )) ret = 0;
+    GetModuleHandleExW( GET_MODULE_HANDLE_EX_FLAG_UNCHANGED_REFCOUNT, module, &ret );
     return ret;
 }
 
-- 
1.7.5.4






More information about the wine-patches mailing list