Changeset 21649 for trunk/src

Timestamp:

Jun 23, 2011, 2:02:58 PM (14 years ago)

Author:

dmik

Message:

general: Exit gracefully instead of crashing if KERNEL32.DLL initialization fails. Note that "gracefully" means "silently" in this case since it seems that the only way how the OS/2 loader reports a failure to initialize a required DLL for a started EXE is by returning a non-zero exit code.

Location:

trunk/src

Files:

• NTDLL/initntdll.c (modified) (1 diff)
• crypt32/initcrypt32.cpp (modified) (1 diff)
• gdi32/initgdi32.cpp (modified) (1 diff)
• kernel32/initkernel32.cpp (modified) (4 diffs)
• kernel32/initterm.cpp (modified) (4 diffs)
• mscms/initmscms.cpp (modified) (1 diff)
• oleacc/initoleacc.cpp (modified) (1 diff)
• rsaenh/initrsaenh.cpp (modified) (1 diff)
• schannel/initschannel.cpp (modified) (1 diff)
• secur32/initsecur32.cpp (modified) (1 diff)
• user32/controls/initodinctrl.cpp (modified) (1 diff)
• user32/inituser32.cpp (modified) (1 diff)

trunk/src/NTDLL/initntdll.c

@@ -54 +54 @@
 
          //Initialize kernel32 first (circular dependency between kernel32 & ntdll)
-         InitializeKernel32();
+         if (InitializeKernel32() == 0)
+             return 0;
 
          CheckVersionFromHMOD(PE2LX_VERSION, hModule); /*PLF Wed  98-03-18 05:28:48*/

trunk/src/crypt32/initcrypt32.cpp

@@ -88 +88 @@
       case 0 :
 
-         InitializeKernel32();
+         if (InitializeKernel32() == 0)
+             return 0;
+
          //CheckVersionFromHMOD(PE2LX_VERSION, hModule); /*PLF Wed  98-03-18 05:28:48*/
          dllHandle = RegisterLxDll(hModule, OdinLibMain, (PVOID)&crypt32_PEResTab);

trunk/src/gdi32/initgdi32.cpp

@@ -99 +99 @@
          ParseLogStatusGDI32();
 
-         InitializeKernel32();
+         if (InitializeKernel32() == 0)
+             return 0;
+
          CheckVersionFromHMOD(PE2LX_VERSION, hModule); /*PLF Wed  98-03-18 05:28:48*/
          if(InitRegionSpace() == FALSE) {

trunk/src/kernel32/initkernel32.cpp

@@ -112 +112 @@
     /*-------------------------------------------------------------------------*/
 
-    if(fInit == TRUE && ulFlag == 0) {
-        return 1; //already initialized
-    }
-    fInit = TRUE;
-
     switch (ulFlag)
     {
-        case 0 :
+        case 0:
         {
+            if (fInit)
+                return 1; // already initialized
+
             // This always must be the first thing to do.
             RasInitialize (hModule);
@@ -257 +255 @@
             RasEntry (RAS_EVENT_Kernel32InitComplete, &dllHandle, sizeof (dllHandle));
 
+            fInit = TRUE;
+
             break;
         }
 
-        case 1 :
+        case 1:
             if (dllHandle)
             {
@@ -267 +267 @@
             break;
 
-        default  :
-            return 0UL;
+        default:
+            return 0;
     }
 
@@ -280 +280 @@
 void APIENTRY cleanupKernel32(ULONG ulReason)
 {
-    dprintf(("kernel32 exit %d\n", ulReason));
+    if (!fInit)
+    {
+        // The initialization sequence was not complete; attempting to
+        // uninitialize some things may crash (yeah, dirty code that doesn't
+        // analyze its own state)
+        return;
+    }
+
+    dprintf(("kernel32 exit %d", ulReason));
 
     TEB *teb = GetThreadTEB();

trunk/src/kernel32/initterm.cpp

@@ -95 +95 @@
     /*-------------------------------------------------------------------------*/
 
-    if(fInit == TRUE && ulFlag == 0) {
-        return 1; //already initialized
-    }
-
     switch (ulFlag)
     {
-        case 0 :
+        case 0:
         {
+            if (fInit)
+                return 1; // already initialized
+
             ctordtorInit();
 
@@ -112 +111 @@
             rc = DosExitList(EXITLIST_KERNEL32|EXLST_ADD, cleanup);
             if (rc)
-                return 0UL;
+                return 0;
 
-            rc = DosQuerySysInfo(QSV_VERSION_MAJOR, QSV_VERSION_MINOR, version, sizeof(version));
-            if(rc == 0){
-                if(version[0] >= 20 && version[1] <= 30) {
+            rc = DosQuerySysInfo(QSV_VERSION_MAJOR, QSV_VERSION_MINOR,
+                                 version, sizeof(version));
+            if (rc == 0)
+                if(version[0] >= 20 && version[1] <= 30)
                     fVersionWarp3 = TRUE;
-                }
-            }
 
             rc = inittermKernel32(hModule, ulFlag);
@@ -125 +123 @@
         }
 
-        case 1 :
+        case 1:
             rc = inittermKernel32(hModule, ulFlag);
             break;
 
-        default  :
-            return 0UL;
+        default:
+            return 0;
     }
 
@@ -156 +154 @@
     HMODULE hModule;
 
-    if(!fInit) {
+    if (!fInit)
         loadNr = globLoadNr++;
-    }
-    DosQueryModuleHandleStrict("WGSS50", &hModule);
-    _O32__DLL_InitTerm(hModule, 0);
-    DosQueryModuleHandleStrict("KERNEL32", &hModule);
-    return DLLENTRYPOINT_NAME(hModule, 0);
+
+    if (DosQueryModuleHandleStrict("WGSS50", &hModule) == NO_ERROR)
+        if (_O32__DLL_InitTerm(hModule, 0) != 0)
+            if (DosQueryModuleHandleStrict("KERNEL32", &hModule) == NO_ERROR)
+                return DLLENTRYPOINT_NAME(hModule, 0);
+
+    return 0; // failure
 }
 //******************************************************************************

trunk/src/mscms/initmscms.cpp

@@ -88 +88 @@
       case 0 :
 
-         InitializeKernel32();
+         if (InitializeKernel32() == 0)
+             return 0;
+
          //CheckVersionFromHMOD(PE2LX_VERSION, hModule); /*PLF Wed  98-03-18 05:28:48*/
          dllHandle = RegisterLxDll(hModule, OdinLibMain, NULL);

trunk/src/oleacc/initoleacc.cpp

@@ -88 +88 @@
       case 0 :
 
-         InitializeKernel32();
+         if (InitializeKernel32() == 0)
+             return 0;
+
          CheckVersionFromHMOD(PE2LX_VERSION, hModule); /*PLF Wed  98-03-18 05:28:48*/
          dllHandle = RegisterLxDll(hModule, OdinLibMain, (PVOID)&oleacc_PEResTab);

trunk/src/rsaenh/initrsaenh.cpp

@@ -88 +88 @@
       case 0 :
 
-         InitializeKernel32();
+         if (InitializeKernel32() == 0)
+             return 0;
+
          //CheckVersionFromHMOD(PE2LX_VERSION, hModule); /*PLF Wed  98-03-18 05:28:48*/
          dllHandle = RegisterLxDll(hModule, OdinLibMain, NULL);

trunk/src/schannel/initschannel.cpp

@@ -88 +88 @@
       case 0 :
 
-         InitializeKernel32();
+         if (InitializeKernel32() == 0)
+             return 0;
+
          //CheckVersionFromHMOD(PE2LX_VERSION, hModule); /*PLF Wed  98-03-18 05:28:48*/
          dllHandle = RegisterLxDll(hModule, OdinLibMain, NULL);

trunk/src/secur32/initsecur32.cpp

@@ -88 +88 @@
       case 0 :
 
-         InitializeKernel32();
+         if (InitializeKernel32() == 0)
+             return 0;
+
          //CheckVersionFromHMOD(PE2LX_VERSION, hModule); /*PLF Wed  98-03-18 05:28:48*/
          dllHandle = RegisterLxDll(hModule, OdinLibMain, NULL);

trunk/src/user32/controls/initodinctrl.cpp

@@ -67 +67 @@
          ParseLogStatusOdinCtrl();
 
-         InitializeKernel32();
+         if (InitializeKernel32() == 0)
+             return 0;
+
          CheckVersionFromHMOD(PE2LX_VERSION, hModule); /*PLF Wed  98-03-18 05:28:48*/
          break;
-     
+
       case 1 :
          break;

trunk/src/user32/inituser32.cpp

@@ -174 +174 @@
          ParseLogStatusUSER32();
 
-         InitializeKernel32();
+         if (InitializeKernel32() == 0)
+             return 0;
+
          CheckVersionFromHMOD(PE2LX_VERSION, hModule); /*PLF Wed  98-03-18 05:28:48*/