Make thread-safe.
authordrepper <drepper>
Fri, 5 Sep 1997 01:30:57 +0000 (01:30 +0000)
committerdrepper <drepper>
Fri, 5 Sep 1997 01:30:57 +0000 (01:30 +0000)
elf/dl-close.c
elf/dl-open.c

index 7e851cd..fa9e92c 100644 (file)
@@ -1,5 +1,5 @@
-/* _dl_close -- Close a shared object opened by `_dl_open'.
-   Copyright (C) 1996 Free Software Foundation, Inc.
+/* Close a shared object opened by `_dl_open'.
+   Copyright (C) 1996, 1997 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
 #include <string.h>
 #include <sys/types.h>
 #include <sys/mman.h>
+#include <libc-lock.h>
 
 
+/* During the program run we must not modify the global data of
+   loaded shared object simultanously in two threads.  Therefore we
+   protect `dlopen' and `dlclose' in dlclose.c.  */
+__libc_lock_define (extern, _dl_load_lock)
+
 #define LOSE(s) _dl_signal_error (0, map->l_name, s)
 
 void
@@ -36,10 +42,16 @@ _dl_close (struct link_map *map)
   if (map->l_opencount == 0)
     LOSE ("shared object not open");
 
+  /* Acquire the lock.  */
+  __libc_lock_lock (_dl_load_lock);
+
   /* Decrement the reference count.  */
   if (--map->l_opencount > 0 || map->l_type != lt_loaded)
-    /* There are still references to this object.  Do nothing more.  */
-    return;
+    {
+      /* There are still references to this object.  Do nothing more.  */
+      __libc_lock_unlock (_dl_load_lock);
+      return;
+    }
 
   /* Notify the debugger we are about to remove some loaded objects.  */
   _r_debug.r_state = RT_DELETE;
@@ -114,4 +126,7 @@ _dl_close (struct link_map *map)
   /* Notify the debugger those objects are finalized and gone.  */
   _r_debug.r_state = RT_CONSISTENT;
   _dl_debug_state ();
+
+  /* Release the lock.  */
+  __libc_lock_unlock (_dl_load_lock);
 }
index 529806c..113e472 100644 (file)
@@ -21,6 +21,7 @@
 #include <dlfcn.h>
 #include <stdlib.h>
 #include <errno.h>
+#include <libc-lock.h>
 
 
 extern ElfW(Addr) _dl_sysdep_start (void **start_argptr,
@@ -38,6 +39,17 @@ extern char **__environ;
 
 size_t _dl_global_scope_alloc;
 
+
+/* During the program run we must not modify the global data of
+   loaded shared object simultanously in two threads.  Therefore we
+   protect `_dl_open' and `_dl_close' in dl-close.c.
+
+   This must be a recursive lock since the initializer function of
+   the loaded object might as well require a call to this function.
+   At this time it is not anymore a problem to modify the tables.  */
+__libc_lock_define_initialized_recursive (, _dl_load_lock)
+
+
 struct link_map *
 _dl_open (const char *file, int mode)
 {
@@ -45,11 +57,17 @@ _dl_open (const char *file, int mode)
   ElfW(Addr) init;
   struct r_debug *r;
 
+  /* Make sure we are alone.  */
+  __libc_lock_lock (_dl_load_lock);
+
   /* Load the named object.  */
   new = _dl_map_object (NULL, file, lt_loaded, 0);
   if (new->l_searchlist)
-    /* It was already open.  */
-    return new;
+    {
+      /* It was already open.  */
+      __libc_lock_unlock (_dl_load_lock);
+      return new;
+    }
 
   /* Load that object's dependencies.  */
   _dl_map_object_deps (new, NULL, 0, 0);
@@ -152,5 +170,8 @@ _dl_open (const char *file, int mode)
        has loaded a dynamic object now has competition.  */
     __libc_multiple_libcs = 1;
 
+  /* Release the lock.  */
+  __libc_lock_unlock (_dl_load_lock);
+
   return new;
 }