A compromise solution for insmod

diff --git a/insmod.c b/insmod.c
index 44c34a2e85c2e5e1d486ce2f1eb0fb4235802585..0644affd194b3c021cef30296828b426b83f3e6e 100644 (file)
--- a/insmod.c
+++ b/insmod.c
@@ -124,7 +124,7 @@
 #ifndef MODUTILS_MODULE_H
 static const int MODUTILS_MODULE_H = 1;
 
-#ident "$Id: insmod.c,v 1.62 2001/05/14 18:27:25 kraai Exp $"
+#ident "$Id: insmod.c,v 1.63 2001/05/14 20:03:04 andersen Exp $"
 
 /* This file contains the structures used by the 2.0 and 2.1 kernels.
    We do not use the kernel headers directly because we do not wish
@@ -330,7 +330,7 @@ int delete_module(const char *);
 #ifndef MODUTILS_OBJ_H
 static const int MODUTILS_OBJ_H = 1;
 
-#ident "$Id: insmod.c,v 1.62 2001/05/14 18:27:25 kraai Exp $"
+#ident "$Id: insmod.c,v 1.63 2001/05/14 20:03:04 andersen Exp $"
 
 /* The relocatable object is manipulated using elfin types.  */
 
@@ -3180,18 +3180,29 @@ extern int insmod_main( int argc, char **argv)
                 * but do not error out yet if we fail to find it... */
                if (uname(&myuname) == 0) {
                        char module_dir[FILENAME_MAX];
+                       char real_module_dir[FILENAME_MAX];
                        snprintf (module_dir, sizeof(module_dir), "%s/%s", 
                                        _PATH_MODULES, myuname.release);
-                       recursive_action(module_dir, TRUE, TRUE, FALSE,
+                       /* Jump through hoops in case /lib/modules/`uname -r`
+                        * is a symlink.  We do not want recursive_action to
+                        * follow symlinks, but we do want to follow the
+                        * /lib/modules/`uname -r` dir, So resolve it ourselves
+                        * if it is a link... */
+                       if (realpath (module_dir, real_module_dir) == NULL)
+                               strcpy(real_module_dir, module_dir);
+                       recursive_action(real_module_dir, TRUE, FALSE, FALSE,
                                        check_module_name_match, 0, m_fullName);
                }
 
                /* Check if we have found anything yet */
                if (m_filename[0] == '\0' || ((fp = fopen(m_filename, "r")) == NULL)) 
                {
+                       char module_dir[FILENAME_MAX];
+                       if (realpath (_PATH_MODULES, module_dir) == NULL)
+                               strcpy(module_dir, _PATH_MODULES);
                        /* No module found under /lib/modules/`uname -r`, this
                         * time cast the net a bit wider.  Search /lib/modules/ */
-                       if (recursive_action(_PATH_MODULES, TRUE, TRUE, FALSE,
+                       if (recursive_action(module_dir, TRUE, FALSE, FALSE,
                                                check_module_name_match, 0, m_fullName) == FALSE) 
                        {
                                if (m_filename[0] == '\0'

diff --git a/modutils/insmod.c b/modutils/insmod.c
index 44c34a2e85c2e5e1d486ce2f1eb0fb4235802585..0644affd194b3c021cef30296828b426b83f3e6e 100644 (file)
--- a/modutils/insmod.c
+++ b/modutils/insmod.c
@@ -124,7 +124,7 @@
 #ifndef MODUTILS_MODULE_H
 static const int MODUTILS_MODULE_H = 1;
 
-#ident "$Id: insmod.c,v 1.62 2001/05/14 18:27:25 kraai Exp $"
+#ident "$Id: insmod.c,v 1.63 2001/05/14 20:03:04 andersen Exp $"
 
 /* This file contains the structures used by the 2.0 and 2.1 kernels.
    We do not use the kernel headers directly because we do not wish
@@ -330,7 +330,7 @@ int delete_module(const char *);
 #ifndef MODUTILS_OBJ_H
 static const int MODUTILS_OBJ_H = 1;
 
-#ident "$Id: insmod.c,v 1.62 2001/05/14 18:27:25 kraai Exp $"
+#ident "$Id: insmod.c,v 1.63 2001/05/14 20:03:04 andersen Exp $"
 
 /* The relocatable object is manipulated using elfin types.  */
 
@@ -3180,18 +3180,29 @@ extern int insmod_main( int argc, char **argv)
                 * but do not error out yet if we fail to find it... */
                if (uname(&myuname) == 0) {
                        char module_dir[FILENAME_MAX];
+                       char real_module_dir[FILENAME_MAX];
                        snprintf (module_dir, sizeof(module_dir), "%s/%s", 
                                        _PATH_MODULES, myuname.release);
-                       recursive_action(module_dir, TRUE, TRUE, FALSE,
+                       /* Jump through hoops in case /lib/modules/`uname -r`
+                        * is a symlink.  We do not want recursive_action to
+                        * follow symlinks, but we do want to follow the
+                        * /lib/modules/`uname -r` dir, So resolve it ourselves
+                        * if it is a link... */
+                       if (realpath (module_dir, real_module_dir) == NULL)
+                               strcpy(real_module_dir, module_dir);
+                       recursive_action(real_module_dir, TRUE, FALSE, FALSE,
                                        check_module_name_match, 0, m_fullName);
                }
 
                /* Check if we have found anything yet */
                if (m_filename[0] == '\0' || ((fp = fopen(m_filename, "r")) == NULL)) 
                {
+                       char module_dir[FILENAME_MAX];
+                       if (realpath (_PATH_MODULES, module_dir) == NULL)
+                               strcpy(module_dir, _PATH_MODULES);
                        /* No module found under /lib/modules/`uname -r`, this
                         * time cast the net a bit wider.  Search /lib/modules/ */
-                       if (recursive_action(_PATH_MODULES, TRUE, TRUE, FALSE,
+                       if (recursive_action(module_dir, TRUE, FALSE, FALSE,
                                                check_module_name_match, 0, m_fullName) == FALSE) 
                        {
                                if (m_filename[0] == '\0'