Update.

2003-03-08  Ulrich Drepper  <drepper@redhat.com>

	* allocatestack.c (allocate_stack): If mprotect() fails free the
	TLS memory.

diff --git a/nptl/ChangeLog b/nptl/ChangeLog
index 562e38b..4f5dc87 100644 (file)
--- a/nptl/ChangeLog
+++ b/nptl/ChangeLog
@@ -1,3 +1,8 @@
+2003-03-08  Ulrich Drepper  <drepper@redhat.com>
+
+       * allocatestack.c (allocate_stack): If mprotect() fails free the
+       TLS memory.
+
 2003-03-07  Ulrich Drepper  <drepper@redhat.com>
 
        * sysdeps/i386/i486/bits/atomic.h: Fix a few unused definitions.

diff --git a/nptl/allocatestack.c b/nptl/allocatestack.c
index 4aec875..31cff2a 100644 (file)
--- a/nptl/allocatestack.c
+++ b/nptl/allocatestack.c
@@ -439,9 +439,12 @@ allocate_stack (const struct pthread_attr *attr, struct pthread **pdp,
 
              lll_unlock (stack_cache_lock);
 
-             /* Free the memory regardless of whether the size of the
-                cache is over the limit or not.  If this piece of
-                memory caused problems we better do not use it
+             /* Get rid of the TLS block we allocated.  */
+             _dl_deallocate_tls (pd, false);
+
+             /* Free the stack memory regardless of whether the size
+                of the cache is over the limit or not.  If this piece
+                of memory caused problems we better do not use it
                 anymore.  Uh, and we ignore possible errors.  There
                 is nothing we could do.  */
              (void) munmap (mem, size);