]> git.ipfire.org Git - thirdparty/glibc.git/commitdiff
* dlfcn/dlmopen.c (dlmopen_doit): Don't allow RTLD_GLOBAL to be
authorUlrich Drepper <drepper@redhat.com>
Wed, 26 Jul 2006 08:21:54 +0000 (08:21 +0000)
committerUlrich Drepper <drepper@redhat.com>
Wed, 26 Jul 2006 08:21:54 +0000 (08:21 +0000)
used when the namespace is not the base namespace.

ChangeLog
dlfcn/dlmopen.c

index fde42b3dfa6ec9d4a0b716a98a7d2e0f3521ba5a..2852fb1fdfdbbfc9fabc0f33f325336a01ff8c90 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2006-07-26  Ulrich Drepper  <drepper@redhat.com>
+
+       * dlfcn/dlmopen.c (dlmopen_doit): Don't allow RTLD_GLOBAL to be
+       used when the namespace is not the base namespace.
+
 2006-07-26  Gavin Romig-Koch  <gavin@redhat.com>
 
        * stdlib/cxa_atexit.c (__new_exitfn_called): New variable.
index 0a6d47ea2e234bc0e576db6fa440a055debe8544..0c6915493b908182fa3c6463fdfeb63f2c3dcb8d 100644 (file)
@@ -1,5 +1,5 @@
 /* Load a shared object at run time.
-   Copyright (C) 1995,96,97,98,99,2000,2003,2004 Free Software Foundation, Inc.
+   Copyright (C) 1995-2000,2003,2004,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
@@ -55,12 +55,19 @@ dlmopen_doit (void *a)
 
   /* Non-shared code has no support for multiple namespaces.  */
   if (args->nsid != LM_ID_BASE)
+    {
 # ifdef SHARED
-    /* If trying to open the link map for the main executable the namespace
-       must be the main one.  */
-    if (args->file == NULL)
+      /* If trying to open the link map for the main executable the namespace
+        must be the main one.  */
+      if (args->file == NULL)
 # endif
-      GLRO(dl_signal_error) (EINVAL, NULL, NULL, N_("invalid namespace"));
+       GLRO(dl_signal_error) (EINVAL, NULL, NULL, N_("invalid namespace"));
+
+      /* It makes no sense to use RTLD_GLOBAL when loading a DSO into
+        a namespace other than the base namespace.  */
+      if (__builtin_expect (args->mode & RTLD_GLOBAL, 0))
+       GLRO(dl_signal_error) (EINVAL, NULL, NULL, N_("invalid mode"));
+    }
 
   args->new = GLRO(dl_open) (args->file ?: "", args->mode | __RTLD_DLOPEN,
                             args->caller,