From ae719405fbbac8c014074af558813cb5338e4ac9 Mon Sep 17 00:00:00 2001
From: Alexandre Julliard <julliard@winehq.org>
Date: Wed, 3 May 2000 18:43:11 +0000
Subject: [PATCH] PE_LoadImage: removed version parameter, set last error
 correctly.

---
 include/pe_image.h |  2 +-
 loader/pe_image.c  | 18 ++++++++----------
 2 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/include/pe_image.h b/include/pe_image.h
index 46fb85248d3..809c289c3ec 100644
--- a/include/pe_image.h
+++ b/include/pe_image.h
@@ -38,7 +38,7 @@ extern DWORD PE_SizeofResource(HMODULE,HRSRC);
 extern struct _wine_modref *PE_LoadLibraryExA(LPCSTR, DWORD);
 extern void PE_UnloadLibrary(struct _wine_modref *);
 extern HGLOBAL PE_LoadResource(struct _wine_modref *wm,HRSRC);
-extern HMODULE PE_LoadImage( HANDLE hFile, LPCSTR filename, WORD *version );
+extern HMODULE PE_LoadImage( HANDLE hFile, LPCSTR filename );
 extern struct _wine_modref *PE_CreateModule( HMODULE hModule, LPCSTR filename,
                                              DWORD flags, BOOL builtin );
 extern BOOL PE_CreateProcess( HANDLE hFile, LPCSTR filename, LPCSTR cmd_line, LPCSTR env, 
diff --git a/loader/pe_image.c b/loader/pe_image.c
index 91992e513d1..50182361ce0 100644
--- a/loader/pe_image.c
+++ b/loader/pe_image.c
@@ -468,7 +468,7 @@ static void do_relocations( unsigned int load_addr, IMAGE_BASE_RELOCATION *r )
  * BUT we have to map the whole image anyway, for Win32 programs sometimes
  * want to access them. (HMODULE32 point to the start of it)
  */
-HMODULE PE_LoadImage( HANDLE hFile, LPCSTR filename, WORD *version )
+HMODULE PE_LoadImage( HANDLE hFile, LPCSTR filename )
 {
     HMODULE	hModule;
     HANDLE	mapping;
@@ -505,6 +505,7 @@ HMODULE PE_LoadImage( HANDLE hFile, LPCSTR filename, WORD *version )
     if ( *(WORD*)hModule !=IMAGE_DOS_SIGNATURE)
     {
         WARN("%s image doesn't have DOS signature, but 0x%04x\n", filename,*(WORD*)hModule);
+        SetLastError( ERROR_BAD_EXE_FORMAT );
         goto error;
     }
 
@@ -514,6 +515,7 @@ HMODULE PE_LoadImage( HANDLE hFile, LPCSTR filename, WORD *version )
     if ( nt->Signature != IMAGE_NT_SIGNATURE )
     {
         WARN("%s image doesn't have PE signature, but 0x%08lx\n", filename, nt->Signature );
+        SetLastError( ERROR_BAD_EXE_FORMAT );
         goto error;
     }
 
@@ -533,6 +535,7 @@ HMODULE PE_LoadImage( HANDLE hFile, LPCSTR filename, WORD *version )
         default: MESSAGE("Unknown-%04x", nt->FileHeader.Machine); break;
         }
         MESSAGE(")\n");
+        SetLastError( ERROR_BAD_EXE_FORMAT );
         goto error;
     }
 
@@ -555,6 +558,7 @@ HMODULE PE_LoadImage( HANDLE hFile, LPCSTR filename, WORD *version )
         ERR("PE module is too small (header: %d, filesize: %d), "
                     "probably truncated download?\n", 
                     rawsize, file_size );
+        SetLastError( ERROR_BAD_EXE_FORMAT );
         goto error;
     }
 
@@ -616,6 +620,7 @@ HMODULE PE_LoadImage( HANDLE hFile, LPCSTR filename, WORD *version )
                    filename,
                    (nt->FileHeader.Characteristics&IMAGE_FILE_RELOCS_STRIPPED)?
                    "stripped during link" : "unknown reason" );
+            SetLastError( ERROR_BAD_EXE_FORMAT );
             goto error;
         }
 
@@ -699,10 +704,6 @@ HMODULE PE_LoadImage( HANDLE hFile, LPCSTR filename, WORD *version )
     if ( reloc )
         do_relocations( load_addr, (IMAGE_BASE_RELOCATION *)RVA(reloc) );
 
-    /* Get expected OS / Subsystem version */
-    *version =   ( (nt->OptionalHeader.MajorSubsystemVersion & 0xff) << 8 )
-               |   (nt->OptionalHeader.MinorSubsystemVersion & 0xff);
-
     /* We don't need the orignal mapping any more */
     UnmapViewOfFile( (LPVOID)hModule );
     return (HMODULE)load_addr;
@@ -919,7 +920,6 @@ WINE_MODREF *PE_LoadLibraryExA (LPCSTR name, DWORD flags)
 	WINE_MODREF	*wm;
 	char        	filename[256];
 	HANDLE		hFile;
-	WORD		version = 0;
 
 	/* Search for and open PE file */
 	if ( SearchPathA( NULL, name, ".DLL", 
@@ -930,11 +930,10 @@ WINE_MODREF *PE_LoadLibraryExA (LPCSTR name, DWORD flags)
 	if ( hFile == INVALID_HANDLE_VALUE ) return NULL;
 	
 	/* Load PE module */
-	hModule32 = PE_LoadImage( hFile, filename, &version );
+	hModule32 = PE_LoadImage( hFile, filename );
 	if (!hModule32)
 	{
                 CloseHandle( hFile );
-		SetLastError( ERROR_OUTOFMEMORY );	/* Not entirely right, but good enough */
 		return NULL;
 	}
 
@@ -993,13 +992,12 @@ BOOL PE_CreateProcess( HANDLE hFile, LPCSTR filename, LPCSTR cmd_line, LPCSTR en
                        BOOL inherit, DWORD flags, LPSTARTUPINFOA startup,
                        LPPROCESS_INFORMATION info )
 {
-    WORD version = 0;
     HMODULE16 hModule16;
     HMODULE hModule32;
     NE_MODULE *pModule;
 
     /* Load file */
-    if ( (hModule32 = PE_LoadImage( hFile, filename, &version )) < 32 )
+    if ( (hModule32 = PE_LoadImage( hFile, filename )) < 32 )
     {
         SetLastError( hModule32 );
         return FALSE;
-- 
2.11.4.GIT