(tests): Add tst-iconv2.
[kopensolaris-gnu/glibc.git] / iconv / gconv_conf.c
index 83d9621..b7e7692 100644 (file)
 #include <errno.h>
 #include <limits.h>
 #include <search.h>
+#include <stddef.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include <unistd.h>
 #include <sys/param.h>
 
+#include <bits/libc-lock.h>
 #include <gconv_int.h>
 
 
 /* This is the default path where we look for module lists.  */
 static const char default_gconv_path[] = GCONV_PATH;
 
-/* The path element in use.   */
-const struct path_elem *__gconv_path_elem;
-/* Maximum length of a single path element.  */
+/* The path elements, as determined by the __gconv_get_path function.
+   All path elements end in a slash.  */
+struct path_elem *__gconv_path_elem;
+/* Maximum length of a single path element in __gconv_path_elem.  */
 size_t __gconv_max_path_elem_len;
 
 /* We use the following struct if we couldn't allocate memory.  */
@@ -168,7 +171,7 @@ add_alias (char *rp, void *modules)
 /* Insert a data structure for a new module in the search tree.  */
 static inline void
 internal_function
-insert_module (struct gconv_module *newp)
+insert_module (struct gconv_module *newp, int tobefreed)
 {
   struct gconv_module **rootp = &__gconv_modules_db;
 
@@ -192,8 +195,24 @@ insert_module (struct gconv_module *newp)
            }
 
          if (root != NULL)
-           /* This is a no new conversion.  */
-           return;
+           {
+             /* This is a no new conversion.  But maybe the cost is
+                better.  */
+             if (newp->cost_hi < root->cost_hi
+                 || (newp->cost_hi == root->cost_hi
+                     && newp->cost_lo < root->cost_lo))
+               {
+                 newp->left = root->left;
+                 newp->right = root->right;
+                 newp->same = root->same;
+                 *rootp = newp;
+
+                 free (root);
+               }
+             else if (tobefreed)
+               free (newp);
+             return;
+           }
 
          break;
        }
@@ -274,13 +293,10 @@ add_module (char *rp, const char *directory, size_t dir_len, void **modules,
     return;
   if (module[0] == '/')
     dir_len = 0;
-  else
-    /* Increment by one for the slash.  */
-    ++dir_len;
 
   /* See whether we must add the ending.  */
   need_ext = 0;
-  if (wp - module < sizeof (gconv_module_ext)
+  if (wp - module < (ptrdiff_t) sizeof (gconv_module_ext)
       || memcmp (wp - sizeof (gconv_module_ext), gconv_module_ext,
                 sizeof (gconv_module_ext)) != 0)
     /* We must add the module extension.  */
@@ -301,26 +317,19 @@ add_module (char *rp, const char *directory, size_t dir_len, void **modules,
     {
       char *tmp;
 
-      new_module->from_string = memcpy ((char *) new_module
-                                       + sizeof (struct gconv_module),
-                                       from, to - from);
+      new_module->from_string = tmp = (char *) (new_module + 1);
+      tmp = __mempcpy (tmp, from, to - from);
 
-      new_module->to_string = memcpy ((char *) new_module->from_string
-                                     + (to - from), to, module - to);
+      new_module->to_string = tmp;
+      tmp = __mempcpy (tmp, to, module - to);
 
       new_module->cost_hi = cost_hi;
       new_module->cost_lo = modcounter;
 
-      new_module->module_name = (char *) new_module->to_string + (module - to);
+      new_module->module_name = tmp;
 
-      if (dir_len == 0)
-       tmp = (char *) new_module->module_name;
-      else
-       {
-         tmp = __mempcpy ((char *) new_module->module_name,
-                          directory, dir_len - 1);
-         *tmp++ = '/';
-       }
+      if (dir_len != 0)
+       tmp = __mempcpy (tmp, directory, dir_len);
 
       tmp = __mempcpy (tmp, module, wp - module);
 
@@ -328,7 +337,7 @@ add_module (char *rp, const char *directory, size_t dir_len, void **modules,
        memcpy (tmp - 1, gconv_module_ext, sizeof (gconv_module_ext));
 
       /* Now insert the new module data structure in our search tree.  */
-      insert_module (new_module);
+      insert_module (new_module, 1);
     }
 }
 
@@ -342,7 +351,7 @@ read_conf_file (const char *filename, const char *directory, size_t dir_len,
   FILE *fp = fopen (filename, "r");
   char *line = NULL;
   size_t line_len = 0;
-  int modcounter = 0;
+  static int modcounter;
 
   /* Don't complain if a file is not present or readable, simply silently
      ignore it.  */
@@ -417,14 +426,18 @@ __gconv_get_path (void)
       char *oldp;
       char *cp;
       int nelems;
+      char *cwd;
+      size_t cwdlen;
 
-      user_path = __secure_getenv ("GCONV_PATH");
+      user_path = getenv ("GCONV_PATH");
       if (user_path == NULL)
        {
          /* No user-defined path.  Make a modifiable copy of the
             default path.  */
          gconv_path = strdupa (default_gconv_path);
          gconv_path_len = sizeof (default_gconv_path);
+         cwd = NULL;
+         cwdlen = 0;
        }
       else
        {
@@ -436,7 +449,10 @@ __gconv_get_path (void)
          __mempcpy (__mempcpy (__mempcpy (gconv_path, user_path, user_len),
                                ":", 1),
                     default_gconv_path, sizeof (default_gconv_path));
+         cwd = __getcwd (NULL, 0);
+         cwdlen = strlen (cwd);
        }
+      assert (default_gconv_path[0] == '/');
 
       /* In a first pass we calculate the number of elements.  */
       oldp = NULL;
@@ -453,7 +469,8 @@ __gconv_get_path (void)
       /* Allocate the memory for the result.  */
       result = (struct path_elem *) malloc ((nelems + 1)
                                            * sizeof (struct path_elem)
-                                           + gconv_path_len + nelems);
+                                           + gconv_path_len + nelems
+                                           + (nelems - 1) * (cwdlen + 1));
       if (result != NULL)
        {
          char *strspace = (char *) &result[nelems + 1];
@@ -466,6 +483,12 @@ __gconv_get_path (void)
          do
            {
              result[n].name = strspace;
+             if (elem[0] != '/')
+               {
+                 assert (cwd != NULL);
+                 strspace = __mempcpy (strspace, cwd, cwdlen);
+                 *strspace++ = '/';
+               }
              strspace = __stpcpy (strspace, elem);
              if (strspace[-1] != '/')
                *strspace++ = '/';
@@ -484,6 +507,9 @@ __gconv_get_path (void)
        }
 
       __gconv_path_elem = result ?: &empty_path_elem;
+
+      if (cwd != NULL)
+       free (cwd);
     }
 
   __libc_lock_unlock (lock);
@@ -506,21 +532,18 @@ __gconv_read_conf (void)
 
   for (cnt = 0; __gconv_path_elem[cnt].name != NULL; ++cnt)
     {
-      char real_elem[__gconv_max_path_elem_len + sizeof (gconv_conf_filename)];
-
-      if (__realpath (__gconv_path_elem[cnt].name, real_elem) != NULL)
-       {
-         size_t elem_len = strlen (real_elem);
-         char *filename;
-
-         filename = alloca (elem_len + 1 + sizeof (gconv_conf_filename));
-         __mempcpy (__mempcpy (__mempcpy (filename, real_elem, elem_len),
-                               "/", 1),
-                    gconv_conf_filename, sizeof (gconv_conf_filename));
-
-         /* Read the next configuration file.  */
-         read_conf_file (filename, real_elem, elem_len, &modules, &nmodules);
-       }
+      const char *elem = __gconv_path_elem[cnt].name;
+      size_t elem_len = __gconv_path_elem[cnt].len;
+      char *filename;
+
+      /* No slash needs to be inserted between elem and gconv_conf_filename;
+        elem already ends in a slash.  */
+      filename = alloca (elem_len + sizeof (gconv_conf_filename));
+      __mempcpy (__mempcpy (filename, elem, elem_len),
+                gconv_conf_filename, sizeof (gconv_conf_filename));
+
+      /* Read the next configuration file.  */
+      read_conf_file (filename, elem, elem_len, &modules, &nmodules);
     }
 
   /* Add the internal modules.  */
@@ -536,7 +559,7 @@ __gconv_read_conf (void)
        /* It'll conflict so don't add it.  */
        continue;
 
-      insert_module (&builtin_modules[cnt]);
+      insert_module (&builtin_modules[cnt], 0);
     }
 
   /* Add aliases for builtin conversions.  */