ldlinux: fixes bug that happens when using fullpath for a COM32 module

When using full path for a com32 module, for example,
/boot/syslinux/ls.c32 it fails without any error message. This patch
fixes it by looking first if the argv[0] is the path to a module before
looking for it at PATH.

Since we're using fopen to open module files (which works for both
absolute paths and paths relative to the current working directory) we
no longer need to include "." in PATH and neither the code to handle it.

Signed-off-by: Andre Ericson <de.ericson@gmail.com>
Signed-off-by: Matt Fleming <matt.fleming@intel.com>

diff --git a/com32/lib/sys/module/common.c b/com32/lib/sys/module/common.c
index eeb2607..19742e6 100644 (file)
--- a/com32/lib/sys/module/common.c
+++ b/com32/lib/sys/module/common.c
@@ -64,6 +64,10 @@ static FILE *findpath(char *name)
        char *p, *n;
        int i;
 
+       f = fopen(name, "rb"); /* for full path */
+       if (f)
+               return f;
+
        p = PATH;
 again:
        i = 0;
@@ -74,15 +78,6 @@ again:
        if (*p == ':')
                p++;
 
-       if (path[0] == '.' && i == 1) {
-               if (!core_getcwd(path, sizeof(path))) {
-                       DBG_PRINT("Could not get cwd\n");
-                       return NULL;
-               }
-
-               i = strlen(path);
-       }
-
        n = name;
        while (*n && i < FILENAME_MAX)
                path[i++] = *n++;

diff --git a/core/include/fs.h b/core/include/fs.h
index 481e085..ded8c15 100644 (file)
--- a/core/include/fs.h
+++ b/core/include/fs.h
@@ -179,7 +179,7 @@ static inline struct file *handle_to_file(uint16_t handle)
     return handle ? &files[handle-1] : NULL;
 }
 
-#define PATH_DEFAULT   ".:/boot/syslinux/:/boot/"
+#define PATH_DEFAULT   "/boot/syslinux/:/boot/"
 extern char *PATH;
 
 /* fs.c */