[BZ #2451]

2006-03-13  Jakub Jelinek  <jakub@redhat.com>
	[BZ #2451]
	* elf/dl-load.c (open_verify): Add free_name argument, if true, free
	name before calling lose.
	(open_path): Adjust caller.
	(_dl_map_object): Adjust callers.  Free name_copy before calling
	_dl_signal_error.
	* elf/Makefile: Add rules to build and run tst-leaks1.
	* elf/tst-leaks1.c: New test.

diff --git a/ChangeLog b/ChangeLog
index 1c85510b4df0e7e347ef0d6d4a85c95bed9cd4a6..6d85208d80f2ca643f5be107b3e924810866b7c8 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,14 @@
+2006-03-13  Jakub Jelinek  <jakub@redhat.com>
+
+       [BZ #2451]
+       * elf/dl-load.c (open_verify): Add free_name argument, if true, free
+       name before calling lose.
+       (open_path): Adjust caller.
+       (_dl_map_object): Adjust callers.  Free name_copy before calling
+       _dl_signal_error.
+       * elf/Makefile: Add rules to build and run tst-leaks1.
+       * elf/tst-leaks1.c: New test.
+
 2006-03-06  Roland McGrath  <roland@redhat.com>
 
        * version.h (VERSION): 2.4

diff --git a/elf/Makefile b/elf/Makefile
index 791341758e9c1b97061d5feabcfb33652f2bc8bb..5e459303f4f64265b1c1082879243323ec31258b 100644 (file)
--- a/elf/Makefile
+++ b/elf/Makefile
@@ -91,7 +91,7 @@ distribute    := rtld-Rules \
                   order2mod1.c order2mod2.c order2mod3.c order2mod4.c \
                   tst-stackguard1.c tst-stackguard1-static.c \
                   tst-array5.c tst-array5-static.c tst-array5dep.c \
-                  tst-array5.exp
+                  tst-array5.exp tst-leaks1.c
 
 CFLAGS-dl-runtime.c = -fexceptions -fasynchronous-unwind-tables
 CFLAGS-dl-lookup.c = -fexceptions -fasynchronous-unwind-tables
@@ -139,7 +139,7 @@ vpath %.c ../locale/programs
 endif
 endif
 
-tests = tst-tls1 tst-tls2 tst-tls9
+tests = tst-tls1 tst-tls2 tst-tls9 tst-leaks1
 ifeq (yes,$(have-initfini-array))
 tests += tst-array1 tst-array2 tst-array3 tst-array4 tst-array5
 endif
@@ -180,6 +180,7 @@ endif
 ifeq (yesyes,$(have-fpie)$(build-shared))
 tests: $(objpfx)tst-pie1.out
 endif
+tests: $(objpfx)tst-leaks1-mem
 modules-names = testobj1 testobj2 testobj3 testobj4 testobj5 testobj6 \
                testobj1_1 failobj constload2 constload3 unloadmod \
                dep1 dep2 dep3 dep4 $(modules-vis-$(have-protected)) \
@@ -895,3 +896,9 @@ order2mod2.so-no-z-defs = yes
 
 tst-stackguard1-ARGS = --command "$(built-program-cmd) --child"
 tst-stackguard1-static-ARGS = --command "$(objpfx)tst-stackguard1-static --child"
+
+$(objpfx)tst-leaks1: $(libdl)
+$(objpfx)tst-leaks1-mem: $(objpfx)tst-leaks1.out
+       $(common-objpfx)malloc/mtrace $(objpfx)tst-leaks1.mtrace > $@
+
+tst-leaks1-ENV = MALLOC_TRACE=$(objpfx)tst-leaks1.mtrace

diff --git a/elf/dl-load.c b/elf/dl-load.c
index bba1c83ba0850d78bfee17f27456f1f7ee70ca43..088954a04f15812796bfe7bf0ceffbc6ca389a8b 100644 (file)
--- a/elf/dl-load.c
+++ b/elf/dl-load.c
@@ -1,5 +1,6 @@
 /* Map in a shared object's segments from the file.
-   Copyright (C) 1995-2002, 2003, 2004, 2005 Free Software Foundation, Inc.
+   Copyright (C) 1995, 1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003, 2004,
+   2005, 2006  Free Software Foundation, Inc.
    This file is part of the GNU C Library.
 
    The GNU C Library is free software; you can redistribute it and/or
@@ -1554,7 +1555,7 @@ print_search_path (struct r_search_path_elem **list,
    user might want to know about this.  */
 static int
 open_verify (const char *name, struct filebuf *fbp, struct link_map *loader,
-            int whatcode, bool *found_other_class)
+            int whatcode, bool *found_other_class, bool free_name)
 {
   /* This is the expected ELF header.  */
 #define ELF32_CLASS ELFCLASS32
@@ -1635,6 +1636,12 @@ open_verify (const char *name, struct filebuf *fbp, struct link_map *loader,
          errstring = (errval == 0
                       ? N_("file too short") : N_("cannot read file data"));
        call_lose:
+         if (free_name)
+           {
+             char *realname = (char *) name;
+             name = strdupa (realname);
+             free (realname);
+           }
          lose (errval, fd, name, NULL, NULL, errstring);
        }
 
@@ -1821,7 +1828,8 @@ open_path (const char *name, size_t namelen, int preloaded,
          if (__builtin_expect (GLRO(dl_debug_mask) & DL_DEBUG_LIBS, 0))
            _dl_debug_printf ("  trying file=%s\n", buf);
 
-         fd = open_verify (buf, fbp, loader, whatcode, found_other_class);
+         fd = open_verify (buf, fbp, loader, whatcode, found_other_class,
+                           false);
          if (this_dir->status[cnt] == unknown)
            {
              if (fd != -1)
@@ -2098,7 +2106,7 @@ _dl_map_object (struct link_map *loader, const char *name, int preloaded,
                {
                  fd = open_verify (cached,
                                    &fb, loader ?: GL(dl_ns)[nsid]._ns_loaded,
-                                   LA_SER_CONFIG, &found_other_class);
+                                   LA_SER_CONFIG, &found_other_class, false);
                  if (__builtin_expect (fd != -1, 1))
                    {
                      realname = local_strdup (cached);
@@ -2136,7 +2144,7 @@ _dl_map_object (struct link_map *loader, const char *name, int preloaded,
        {
          fd = open_verify (realname, &fb,
                            loader ?: GL(dl_ns)[nsid]._ns_loaded, 0,
-                           &found_other_class);
+                           &found_other_class, true);
          if (__builtin_expect (fd, 0) == -1)
            free (realname);
        }
@@ -2166,8 +2174,11 @@ _dl_map_object (struct link_map *loader, const char *name, int preloaded,
          if ((name_copy = local_strdup (name)) == NULL
              || (l = _dl_new_object (name_copy, name, type, loader,
                                      mode, nsid)) == NULL)
-           _dl_signal_error (ENOMEM, name, NULL,
-                             N_("cannot create shared object descriptor"));
+           {
+             free (name_copy);
+             _dl_signal_error (ENOMEM, name, NULL,
+                               N_("cannot create shared object descriptor"));
+           }
          /* Signal that this is a faked entry.  */
          l->l_faked = 1;
          /* Since the descriptor is initialized with zero we do not

diff --git a/elf/tst-leaks1.c b/elf/tst-leaks1.c
new file mode 100644 (file)
index 0000000..c9db5e0
--- /dev/null
+++ b/elf/tst-leaks1.c
@@ -0,0 +1,24 @@
+#include <dlfcn.h>
+#include <mcheck.h>
+#include <stdlib.h>
+
+int
+main (void)
+{
+  mtrace ();
+
+  int ret = 0;
+  for (int i = 0; i < 10; i++)
+    {
+      void *h = dlopen (i < 5 ? "./tst-leaks1.c"
+                             : "$ORIGIN/tst-leaks1.o", RTLD_LAZY);
+      if (h != NULL)
+       {
+         puts ("dlopen unexpectedly succeeded");
+         ret = 1;
+         dlclose (h);
+       }
+    }
+
+  return ret;
+}