Changeset 22028 for trunk/src/peldr/pe.c

Timestamp:

Oct 22, 2012, 2:48:13 PM (13 years ago)

Author:

dmik

Message:

pe: A better fix for adding .EXE than r22016.

It will only add .EXE if there is no (any) extention already specified,
regardless of whether the path is present or not. This matches the
behavior of CMD.EXE on both Windows and OS/2.

File:

• trunk/src/peldr/pe.c (modified) (1 diff)

trunk/src/peldr/pe.c

@@ -199 +199 @@
         win32cmdline = pszTemp;
 
-        // add the .EXE extension if missing (only if there's no path)
-        if (*str_find_char(exeName, '.') == '\0' &&
-            *str_find_char(exeName, '\\') == '\0' &&
-            *str_find_char(exeName, '/') == '\0')
+        // add the .EXE extension if missing (only if there's no extension already)
+        if (*str_find_last_char(exeName, '.') == '\0')
         {
             int i = strlen(exeName);
-            if (i < 4 || !((exeName[i-1] == 'E' || exeName[i-1] == 'e') &&
-                           (exeName[i-2] == 'X' || exeName[i-2] == 'x') &&
-                           (exeName[i-3] == 'E' || exeName[i-3] == 'e') &&
-                           exeName[i-4] == '.'))
-            {
-                if (i + 4 < CCHMAXPATH)
-                    strcat(exeName, ".exe");
-            }
+            if (i + 4 < CCHMAXPATH)
+                strcat(exeName, ".exe");
         }