Changeset 22040 for trunk/src/kernel32/oslibmisc.cpp

Timestamp:

Oct 25, 2012, 12:39:39 PM (13 years ago)

Author:

dmik

Message:

kernel32: Fix Win32ImageBase::matchModName().

As opposed to the declared functionality, it was comparing the name part
of the argument with the full path of the stored module name and this would
obviously not work if the module name were a full path.

One of the failig cases was a frequent attempt to retrieve the resources of the
current executable by doing LoadLibraryEx() with the full executable name.

File:

• trunk/src/kernel32/oslibmisc.cpp (modified) (1 diff)

trunk/src/kernel32/oslibmisc.cpp

@@ -267 +267 @@
 //******************************************************************************
 //******************************************************************************
-char *OSLibStripPath(char *path)
-{
-  /* @@@PH what does this function do ? Strip the path from a FQFN name ? */
-  char *pszFilename;
-  char *pszFilename1;
-
-  pszFilename  = strrchr(path, '\\');                 /* find rightmost backslash */
-  pszFilename1 = strrchr(path, '/');                  /* find rightmost slash */
-  if(pszFilename > pszFilename1 && pszFilename != NULL)
-    return (++pszFilename);              /* return pointer to next character */
-
-  if (pszFilename1 != NULL)
-    return (++pszFilename1);              /* return pointer to next character */
-
-  return (path);                                     /* default return value */
+const char *OSLibStripPath(const char *path)
+{
+    const char *pszName;
+    register char ch;
+
+    /*
+     * Search the filename string finding the modulename start and end.
+     * The loop ends when we have passed one char left of the module name.
+     */
+    pszName = path + strlen(path) - 1;
+    while (pszName >= path
+           && (ch = *pszName) != '\\'
+           && ch != '/'
+           && ch != ':'
+           )
+    {
+        pszName--;
+    }
+    pszName++;
+
+    return pszName;
 }
 //******************************************************************************