[PATCH] krnl386.exe: removed unused owner_exists variable (Coverity)

Marcus Meissner marcus at jet.franken.de
Sun Jan 3 06:33:56 CST 2010


Hi,

not sure if this is right, but owner_exists is only ever set
to FALSE once, so checking for it is not necessary.

Perhaps the hInst=21 can go entirely.

Ciao, Marcus
---
 dlls/krnl386.exe16/ne_module.c |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/dlls/krnl386.exe16/ne_module.c b/dlls/krnl386.exe16/ne_module.c
index 08ed94c..9a0b6cc 100644
--- a/dlls/krnl386.exe16/ne_module.c
+++ b/dlls/krnl386.exe16/ne_module.c
@@ -967,7 +967,6 @@ static HINSTANCE16 MODULE_LoadModule16( LPCSTR libname, BOOL implicit, BOOL lib_
     const char *file_name = NULL;
     char dllname[32], *p;
     const char *basename, *main_module;
-    int owner_exists = FALSE;
 
     /* strip path information */
 
@@ -1042,7 +1041,7 @@ static HINSTANCE16 MODULE_LoadModule16( LPCSTR libname, BOOL implicit, BOOL lib_
         TRACE("Trying native dll '%s'\n", libname);
         hinst = NE_LoadModule(libname, lib_only);
         if (hinst > 32) TRACE_(loaddll)("Loaded module %s : native\n", debugstr_a(libname));
-        if (hinst == ERROR_FILE_NOT_FOUND && owner_exists) hinst = 21;  /* win32 module */
+        if (hinst == ERROR_FILE_NOT_FOUND) hinst = 21;  /* win32 module */
     }
 
     if (hinst > 32 && !implicit)
-- 
1.5.6



More information about the wine-patches mailing list