* sysdeps/unix/sysv/linux/timer_routines.c (timer_helper_thread):
authorUlrich Drepper <drepper@redhat.com>
Thu, 27 Apr 2006 18:17:50 +0000 (18:17 +0000)
committerUlrich Drepper <drepper@redhat.com>
Thu, 27 Apr 2006 18:17:50 +0000 (18:17 +0000)
Allocate new object which is passed to timer_sigev_thread so that
the timer can be deleted before the new thread is scheduled.

nptl/ChangeLog
nptl/sysdeps/unix/sysv/linux/timer_routines.c

index c725b8d..132b2d0 100644 (file)
@@ -1,3 +1,9 @@
+2006-04-27  Ulrich Drepper  <drepper@redhat.com>
+
+       * sysdeps/unix/sysv/linux/timer_routines.c (timer_helper_thread):
+       Allocate new object which is passed to timer_sigev_thread so that
+       the timer can be deleted before the new thread is scheduled.
+
 2006-04-26  Roland McGrath  <roland@redhat.com>
 
        * sysdeps/x86_64/tls.h: Include <asm/prctl.h> inside [! __ASSEMBLER__].
index c180b27..a5eb442 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (C) 2003, 2004, 2005 Free Software Foundation, Inc.
+/* Copyright (C) 2003, 2004, 2005, 2006 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
    Contributed by Ulrich Drepper <drepper@redhat.com>, 2003.
 
 #include "kernel-posix-timers.h"
 
 
+struct thread_start_data
+{
+  void (*thrfunc) (sigval_t);
+  sigval_t sival;
+};
+
+
 #ifdef __NR_timer_create
 /* Helper thread to call the user-provided function.  */
 static void *
@@ -40,10 +47,16 @@ timer_sigev_thread (void *arg)
   INTERNAL_SYSCALL_DECL (err);
   INTERNAL_SYSCALL (rt_sigprocmask, err, 4, SIG_SETMASK, &ss, NULL, _NSIG / 8);
 
-  struct timer *tk = (struct timer *) arg;
+  struct thread_start_data *td = (struct thread_start_data *) arg;
+
+  void (*thrfunc) (sigval_t) = td->thrfunc;
+  sigval_t sival = td->sival;
+
+  /* The TD object was allocated in timer_helper_thread.  */
+  free (td);
 
   /* Call the user-provided function.  */
-  tk->thrfunc (tk->sival);
+  thrfunc (sival);
 
   return NULL;
 }
@@ -82,10 +95,19 @@ timer_helper_thread (void *arg)
          if (si.si_code == SI_TIMER)
            {
              struct timer *tk = (struct timer *) si.si_ptr;
-
-             /* That the signal we are waiting for.  */
-             pthread_t th;
-             (void) pthread_create (&th, &tk->attr, timer_sigev_thread, tk);
+             struct thread_start_data *td = malloc (sizeof (*td));
+
+             /* There is not much we can do if the allocation fails.  */
+             if (td != NULL)
+               {
+                 /* That is the signal we are waiting for.  */
+                 td->thrfunc = tk->thrfunc;
+                 td->sival = tk->sival;
+
+                 pthread_t th;
+                 (void) pthread_create (&th, &tk->attr, timer_sigev_thread,
+                                        td);
+               }
            }
          else if (si.si_code == SI_TKILL)
            /* The thread is canceled.  */