[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