Changeset 110 for trunk/src/helpers/wphandle.c

Timestamp:

Oct 15, 2001, 11:17:25 PM (24 years ago)

Author:

umoeller

Message:

Misc updates.

File:

• trunk/src/helpers/wphandle.c (modified) (7 diffs)

trunk/src/helpers/wphandle.c

@@ -99 +99 @@
 
 APIRET wphQueryActiveHandles(HINI hiniSystem,
-                             PSZ *ppszActiveHandles)
+                             PSZ *ppszActiveHandles)        // out: active handles string (new buffer)
 {
     PSZ pszActiveHandles;
@@ -231 +231 @@
                 // store PNODE in hash table
                 pHandlesBuf->NodeHashTable[pNode->usHandle] = pNode;
-                pCur += sizeof (NODE) + pNode->usNameSize;
+                pCur += sizeof(NODE) + pNode->usNameSize;
             }
             else
@@ -341 +341 @@
                         ULONG   cbTotalOld = cbTotal;
                         cbTotal += cbBlockThis;
-                        if (!(pbData = realloc(pbData, cbTotal)))
+                        if (!(pbData = (BYTE*)realloc(pbData, cbTotal)))
                                 // on first call, pbData is NULL and this
                                 // behaves like malloc()
@@ -417 +417 @@
     return (arc);
 }
-
-/*
- *@@ wphReadAllBlocks:
- *      this reads all the BLOCK's in the active handles
- *      section in OS2SYS.INI into one common buffer, for
- *      which sufficient memory is allocated.
- *
- *      Each of these blocks is a maximum of 64 KB, hell
- *      knows why, maybe the INIs can't handle more. This
- *      func combines all the BLOCK's into one buffer anyways,
- *      which can then be searched.
- *
- *      Use wphQueryActiveHandles to pass the active handles
- *      section to this function.
- *
- *      This gets called by the one-shot function
- *      wphQueryHandleFromPath.
- */
-
-/* BOOL wphReadAllBlocks(HINI hiniSystem,       // in: can be HINI_USER or HINI_SYSTEM
-                      PSZ pszActiveHandles,  // in: active handles section
-                      PBYTE* ppBlock,        // in/out: pointer to buffer, which
-                                             //    will point to the allocated
-                                             //    memory afterwards
-                      PULONG pulSize)        // out: size of the allocated buffer
-{
-    PBYTE   pbAllBlocks,
-            pszBlockName,
-            p;
-    ULONG   ulBlockSize;
-    ULONG   ulTotalSize;
-    BYTE    rgfBlock[100];
-    BYTE    szAppName[10];
-    USHORT  usBlockNo;
-
-    pbAllBlocks = wphEnumProfileKeys(hiniSystem, pszActiveHandles, &ulBlockSize);
-    if (!pbAllBlocks)
-       return FALSE;
-
-    // query size of all individual blocks and calculate total
-    memset(rgfBlock, 0, sizeof rgfBlock);
-    ulTotalSize = 0L;
-    pszBlockName = pbAllBlocks;
-    while (*pszBlockName)
-    {
-        if (!memicmp(pszBlockName, "BLOCK", 5))
-        {
-           usBlockNo = atoi(pszBlockName + 5);
-           if (usBlockNo < 100)
-           {
-               rgfBlock[usBlockNo] = TRUE;
-
-               if (!PrfQueryProfileSize(hiniSystem,
-                                        pszActiveHandles,
-                                        pszBlockName,
-                                        &ulBlockSize))
-               {
-                   free(pbAllBlocks);
-                   return FALSE;
-               }
-               ulTotalSize += ulBlockSize;
-           }
-        }
-        pszBlockName += strlen(pszBlockName) + 1;
-    }
-
-    // *pulSize now contains the total size of all blocks
-    *ppBlock = (PBYTE)malloc(ulTotalSize);
-    if (!(*ppBlock))
-    {
-         free(pbAllBlocks);
-         return FALSE;
-    }
-
-    // now get all blocks into the memory we allocated
-    p = *ppBlock;
-    (*pulSize) = 0;
-    for (usBlockNo = 1; usBlockNo < 100; usBlockNo++)
-    {
-        if (!rgfBlock[usBlockNo])
-            break;
-
-        sprintf(szAppName, "BLOCK%u", (UINT)usBlockNo);
-        ulBlockSize = ulTotalSize;
-        if (!PrfQueryProfileData(hiniSystem,
-                                 pszActiveHandles,
-                                 szAppName,
-                                 p,
-                                 &ulBlockSize))
-        {
-            free(pbAllBlocks);
-            free(*ppBlock);
-            return FALSE;
-        }
-        p += ulBlockSize;
-        (*pulSize) += ulBlockSize;
-        ulTotalSize -= ulBlockSize;
-    }
-
-    free(pbAllBlocks);
-    return TRUE;
-} */
 
 /* ******************************************************************
@@ -688 +586 @@
 {
     APIRET      arc = NO_ERROR;
-    PVOID       pvData;
+
+    PSZ         pszActiveHandles = NULL;
+    PHANDLESBUF pHandlesBuf = NULL;
 
     TRY_LOUD(excpt1)
     {
         // not found there: check the handles then
-        PSZ pszActiveHandles;
+
         if (arc = wphQueryActiveHandles(hiniSystem, &pszActiveHandles))
             _Pmpf((__FUNCTION__ ": wphQueryActiveHandles returned %d", arc));
         else
         {
-            PHANDLESBUF pHandlesBuf;
             if (arc = wphLoadHandles(hiniUser,
                                      hiniSystem,
@@ -719 +618 @@
                 else
                     arc = ERROR_FILE_NOT_FOUND;
-
-                wphFreeHandles(&pHandlesBuf);
-            }
-
-            free(pszActiveHandles);
+            }
         }
     }
@@ -730 +625 @@
         arc = ERROR_WPH_CRASHED;
     } END_CATCH();
+
+    if (pszActiveHandles)
+        free(pszActiveHandles);
+    if (pHandlesBuf)
+        wphFreeHandles(&pHandlesBuf);
 
     return (arc);