djdelorie / rpms / glibc

Forked from rpms/glibc 3 years ago
Clone
Siddhesh Poyarekar f64bffe
2014-08-21  Florian Weimer  <fweimer@redhat.com>
Siddhesh Poyarekar f64bffe
Siddhesh Poyarekar f64bffe
	[BZ #17187]
Siddhesh Poyarekar f64bffe
	* iconv/gconv_trans.c (struct known_trans, search_tree, lock,
Siddhesh Poyarekar f64bffe
	trans_compare, open_translit, __gconv_translit_find):
Siddhesh Poyarekar f64bffe
	Remove module loading code.
Siddhesh Poyarekar f64bffe
Siddhesh Poyarekar f64bffe
diff --git a/iconv/gconv_trans.c b/iconv/gconv_trans.c
Siddhesh Poyarekar f64bffe
index 1e25854..d71c029 100644
Siddhesh Poyarekar f64bffe
--- a/iconv/gconv_trans.c
Siddhesh Poyarekar f64bffe
+++ b/iconv/gconv_trans.c
Siddhesh Poyarekar f64bffe
@@ -238,181 +238,11 @@ __gconv_transliterate (struct __gconv_step *step,
Siddhesh Poyarekar f64bffe
   return __GCONV_ILLEGAL_INPUT;
Siddhesh Poyarekar f64bffe
 }
Siddhesh Poyarekar f64bffe
 
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-/* Structure to represent results of found (or not) transliteration
Siddhesh Poyarekar f64bffe
-   modules.  */
Siddhesh Poyarekar f64bffe
-struct known_trans
Siddhesh Poyarekar f64bffe
-{
Siddhesh Poyarekar f64bffe
-  /* This structure must remain the first member.  */
Siddhesh Poyarekar f64bffe
-  struct trans_struct info;
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-  char *fname;
Siddhesh Poyarekar f64bffe
-  void *handle;
Siddhesh Poyarekar f64bffe
-  int open_count;
Siddhesh Poyarekar f64bffe
-};
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-/* Tree with results of previous calls to __gconv_translit_find.  */
Siddhesh Poyarekar f64bffe
-static void *search_tree;
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-/* We modify global data.   */
Siddhesh Poyarekar f64bffe
-__libc_lock_define_initialized (static, lock);
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-/* Compare two transliteration entries.  */
Siddhesh Poyarekar f64bffe
-static int
Siddhesh Poyarekar f64bffe
-trans_compare (const void *p1, const void *p2)
Siddhesh Poyarekar f64bffe
-{
Siddhesh Poyarekar f64bffe
-  const struct known_trans *s1 = (const struct known_trans *) p1;
Siddhesh Poyarekar f64bffe
-  const struct known_trans *s2 = (const struct known_trans *) p2;
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-  return strcmp (s1->info.name, s2->info.name);
Siddhesh Poyarekar f64bffe
-}
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-/* Open (maybe reopen) the module named in the struct.  Get the function
Siddhesh Poyarekar f64bffe
-   and data structure pointers we need.  */
Siddhesh Poyarekar f64bffe
-static int
Siddhesh Poyarekar f64bffe
-open_translit (struct known_trans *trans)
Siddhesh Poyarekar f64bffe
-{
Siddhesh Poyarekar f64bffe
-  __gconv_trans_query_fct queryfct;
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-  trans->handle = __libc_dlopen (trans->fname);
Siddhesh Poyarekar f64bffe
-  if (trans->handle == NULL)
Siddhesh Poyarekar f64bffe
-    /* Not available.  */
Siddhesh Poyarekar f64bffe
-    return 1;
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-  /* Find the required symbol.  */
Siddhesh Poyarekar f64bffe
-  queryfct = __libc_dlsym (trans->handle, "gconv_trans_context");
Siddhesh Poyarekar f64bffe
-  if (queryfct == NULL)
Siddhesh Poyarekar f64bffe
-    {
Siddhesh Poyarekar f64bffe
-      /* We cannot live with that.  */
Siddhesh Poyarekar f64bffe
-    close_and_out:
Siddhesh Poyarekar f64bffe
-      __libc_dlclose (trans->handle);
Siddhesh Poyarekar f64bffe
-      trans->handle = NULL;
Siddhesh Poyarekar f64bffe
-      return 1;
Siddhesh Poyarekar f64bffe
-    }
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-  /* Get the context.  */
Siddhesh Poyarekar f64bffe
-  if (queryfct (trans->info.name, &trans->info.csnames, &trans->info.ncsnames)
Siddhesh Poyarekar f64bffe
-      != 0)
Siddhesh Poyarekar f64bffe
-    goto close_and_out;
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-  /* Of course we also have to have the actual function.  */
Siddhesh Poyarekar f64bffe
-  trans->info.trans_fct = __libc_dlsym (trans->handle, "gconv_trans");
Siddhesh Poyarekar f64bffe
-  if (trans->info.trans_fct == NULL)
Siddhesh Poyarekar f64bffe
-    goto close_and_out;
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-  /* Now the optional functions.  */
Siddhesh Poyarekar f64bffe
-  trans->info.trans_init_fct =
Siddhesh Poyarekar f64bffe
-    __libc_dlsym (trans->handle, "gconv_trans_init");
Siddhesh Poyarekar f64bffe
-  trans->info.trans_context_fct =
Siddhesh Poyarekar f64bffe
-    __libc_dlsym (trans->handle, "gconv_trans_context");
Siddhesh Poyarekar f64bffe
-  trans->info.trans_end_fct =
Siddhesh Poyarekar f64bffe
-    __libc_dlsym (trans->handle, "gconv_trans_end");
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-  trans->open_count = 1;
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-  return 0;
Siddhesh Poyarekar f64bffe
-}
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
 int
Siddhesh Poyarekar f64bffe
 internal_function
Siddhesh Poyarekar f64bffe
 __gconv_translit_find (struct trans_struct *trans)
Siddhesh Poyarekar f64bffe
 {
Siddhesh Poyarekar f64bffe
-  struct known_trans **found;
Siddhesh Poyarekar f64bffe
-  const struct path_elem *runp;
Siddhesh Poyarekar f64bffe
-  int res = 1;
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-  /* We have to have a name.  */
Siddhesh Poyarekar f64bffe
-  assert (trans->name != NULL);
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-  /* Acquire the lock.  */
Siddhesh Poyarekar f64bffe
-  __libc_lock_lock (lock);
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-  /* See whether we know this module already.  */
Siddhesh Poyarekar f64bffe
-  found = __tfind (trans, &search_tree, trans_compare);
Siddhesh Poyarekar f64bffe
-  if (found != NULL)
Siddhesh Poyarekar f64bffe
-    {
Siddhesh Poyarekar f64bffe
-      /* Is this module available?  */
Siddhesh Poyarekar f64bffe
-      if ((*found)->handle != NULL)
Siddhesh Poyarekar f64bffe
-	{
Siddhesh Poyarekar f64bffe
-	  /* Maybe we have to reopen the file.  */
Siddhesh Poyarekar f64bffe
-	  if ((*found)->handle != (void *) -1)
Siddhesh Poyarekar f64bffe
-	    /* The object is not unloaded.  */
Siddhesh Poyarekar f64bffe
-	    res = 0;
Siddhesh Poyarekar f64bffe
-	  else if (open_translit (*found) == 0)
Siddhesh Poyarekar f64bffe
-	    {
Siddhesh Poyarekar f64bffe
-	      /* Copy the data.  */
Siddhesh Poyarekar f64bffe
-	      *trans = (*found)->info;
Siddhesh Poyarekar f64bffe
-	      (*found)->open_count++;
Siddhesh Poyarekar f64bffe
-	      res = 0;
Siddhesh Poyarekar f64bffe
-	    }
Siddhesh Poyarekar f64bffe
-	}
Siddhesh Poyarekar f64bffe
-    }
Siddhesh Poyarekar f64bffe
-  else
Siddhesh Poyarekar f64bffe
-    {
Siddhesh Poyarekar f64bffe
-      size_t name_len = strlen (trans->name) + 1;
Siddhesh Poyarekar f64bffe
-      int need_so = 0;
Siddhesh Poyarekar f64bffe
-      struct known_trans *newp;
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-      /* We have to continue looking for the module.  */
Siddhesh Poyarekar f64bffe
-      if (__gconv_path_elem == NULL)
Siddhesh Poyarekar f64bffe
-	__gconv_get_path ();
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-      /* See whether we have to append .so.  */
Siddhesh Poyarekar f64bffe
-      if (name_len <= 4 || memcmp (&trans->name[name_len - 4], ".so", 3) != 0)
Siddhesh Poyarekar f64bffe
-	need_so = 1;
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-      /* Create a new entry.  */
Siddhesh Poyarekar f64bffe
-      newp = (struct known_trans *) malloc (sizeof (struct known_trans)
Siddhesh Poyarekar f64bffe
-					    + (__gconv_max_path_elem_len
Siddhesh Poyarekar f64bffe
-					       + name_len + 3)
Siddhesh Poyarekar f64bffe
-					    + name_len);
Siddhesh Poyarekar f64bffe
-      if (newp != NULL)
Siddhesh Poyarekar f64bffe
-	{
Siddhesh Poyarekar f64bffe
-	  char *cp;
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-	  /* Clear the struct.  */
Siddhesh Poyarekar f64bffe
-	  memset (newp, '\0', sizeof (struct known_trans));
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-	  /* Store a copy of the module name.  */
Siddhesh Poyarekar f64bffe
-	  newp->info.name = cp = (char *) (newp + 1);
Siddhesh Poyarekar f64bffe
-	  cp = __mempcpy (cp, trans->name, name_len);
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-	  newp->fname = cp;
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-	  /* Search in all the directories.  */
Siddhesh Poyarekar f64bffe
-	  for (runp = __gconv_path_elem; runp->name != NULL; ++runp)
Siddhesh Poyarekar f64bffe
-	    {
Siddhesh Poyarekar f64bffe
-	      cp = __mempcpy (__stpcpy ((char *) newp->fname, runp->name),
Siddhesh Poyarekar f64bffe
-			      trans->name, name_len);
Siddhesh Poyarekar f64bffe
-	      if (need_so)
Siddhesh Poyarekar f64bffe
-		memcpy (cp, ".so", sizeof (".so"));
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-	      if (open_translit (newp) == 0)
Siddhesh Poyarekar f64bffe
-		{
Siddhesh Poyarekar f64bffe
-		  /* We found a module.  */
Siddhesh Poyarekar f64bffe
-		  res = 0;
Siddhesh Poyarekar f64bffe
-		  break;
Siddhesh Poyarekar f64bffe
-		}
Siddhesh Poyarekar f64bffe
-	    }
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-	  if (res)
Siddhesh Poyarekar f64bffe
-	    newp->fname = NULL;
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-	  /* In any case we'll add the entry to our search tree.  */
Siddhesh Poyarekar f64bffe
-	  if (__tsearch (newp, &search_tree, trans_compare) == NULL)
Siddhesh Poyarekar f64bffe
-	    {
Siddhesh Poyarekar f64bffe
-	      /* Yickes, this should not happen.  Unload the object.  */
Siddhesh Poyarekar f64bffe
-	      res = 1;
Siddhesh Poyarekar f64bffe
-	      /* XXX unload here.  */
Siddhesh Poyarekar f64bffe
-	    }
Siddhesh Poyarekar f64bffe
-	}
Siddhesh Poyarekar f64bffe
-    }
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-  __libc_lock_unlock (lock);
Siddhesh Poyarekar f64bffe
-
Siddhesh Poyarekar f64bffe
-  return res;
Siddhesh Poyarekar f64bffe
+  /* This function always fails.  Transliteration module loading is
Siddhesh Poyarekar f64bffe
+     not implemented.  */
Siddhesh Poyarekar f64bffe
+  return 1;
Siddhesh Poyarekar f64bffe
 }
Siddhesh Poyarekar f64bffe
-- 
Siddhesh Poyarekar f64bffe
1.9.3
Siddhesh Poyarekar f64bffe