Check for a valid module before attempting to read the export

dlls/ntdll/loader.c

@@ -1181,7 +1181,9 @@ NTSTATUS WINAPI LdrGetProcedureAddress(HMODULE module, const ANSI_STRING *name,
 
     RtlEnterCriticalSection( &loader_section );
 
-    if ((exports = RtlImageDirectoryEntryToData( module, TRUE,
+    /* check if the module itself is invalid to return the proper error */
+    if (!get_modref( module )) ret = STATUS_DLL_NOT_FOUND;
+    else if ((exports = RtlImageDirectoryEntryToData( module, TRUE,
                                                       IMAGE_DIRECTORY_ENTRY_EXPORT, &exp_size )))
     {
         void *proc = name ? find_named_export( module, exports, exp_size, name->Buffer, -1 )
@@ -1192,11 +1194,6 @@ NTSTATUS WINAPI LdrGetProcedureAddress(HMODULE module, const ANSI_STRING *name,
             ret = STATUS_SUCCESS;
         }
     }
-    else
-    {
-        /* check if the module itself is invalid to return the proper error */
-        if (!get_modref( module )) ret = STATUS_DLL_NOT_FOUND;
-    }
 
     RtlLeaveCriticalSection( &loader_section );
     return ret;