提交 bf619d82 编写于 作者: R Rich Felker

major improvements to cancellation handling

- there is no longer any risk of spoofing cancellation requests, since
  the cancel flag is set in pthread_cancel rather than in the signal
  handler.

- cancellation signal is no longer unblocked when running the
  cancellation handlers. instead, pthread_create will cause any new
  threads created from a cancellation handler to unblock their own
  cancellation signal.

- various tweaks in preparation for POSIX timer support.
上级 70c31c7b
...@@ -18,6 +18,7 @@ struct __libc { ...@@ -18,6 +18,7 @@ struct __libc {
FILE *ofl_head; FILE *ofl_head;
int ofl_lock; int ofl_lock;
void (*lockfile)(FILE *); void (*lockfile)(FILE *);
void (*sigtimer)();
}; };
......
...@@ -41,10 +41,14 @@ struct pthread { ...@@ -41,10 +41,14 @@ struct pthread {
long off; long off;
void *pending; void *pending;
} robust_list; } robust_list;
int unblock_cancel;
}; };
struct __timer { struct __timer {
struct sigevent sev; int timerid;
union sigval val;
void (*notify)(union sigval);
pthread_t thread;
}; };
#define __SU (sizeof(size_t)/sizeof(int)) #define __SU (sizeof(size_t)/sizeof(int))
...@@ -73,7 +77,6 @@ struct __timer { ...@@ -73,7 +77,6 @@ struct __timer {
#define SIGCANCEL 32 #define SIGCANCEL 32
#define SIGSYSCALL 33 #define SIGSYSCALL 33
#define SIGTIMER 32 /* ?? */
int __set_thread_area(void *); int __set_thread_area(void *);
int __libc_sigaction(int, const struct sigaction *, struct sigaction *); int __libc_sigaction(int, const struct sigaction *, struct sigaction *);
......
...@@ -2,5 +2,6 @@ ...@@ -2,5 +2,6 @@
int pthread_cancel(pthread_t t) int pthread_cancel(pthread_t t)
{ {
t->cancel = 1;
return pthread_kill(t, SIGCANCEL); return pthread_kill(t, SIGCANCEL);
} }
...@@ -47,20 +47,19 @@ void __pthread_unwind_next(struct __ptcb *cb) ...@@ -47,20 +47,19 @@ void __pthread_unwind_next(struct __ptcb *cb)
static void docancel(struct pthread *self) static void docancel(struct pthread *self)
{ {
struct __ptcb cb = { .__next = self->cancelbuf }; struct __ptcb cb = { .__next = self->cancelbuf };
sigset_t set;
self->canceldisable = 1; self->canceldisable = 1;
self->cancelasync = 0; self->cancelasync = 0;
sigemptyset(&set);
sigaddset(&set, SIGCANCEL);
__libc_sigprocmask(SIG_UNBLOCK, &set, 0);
__pthread_unwind_next(&cb); __pthread_unwind_next(&cb);
} }
static void cancel_handler(int sig, siginfo_t *si, void *ctx) static void cancel_handler(int sig, siginfo_t *si, void *ctx)
{ {
struct pthread *self = __pthread_self(); struct pthread *self = __pthread_self();
if (si->si_code > 0 || si->si_pid != self->pid) return; if (!self->cancel) {
self->cancel = 1; if (si->si_code == SI_TIMER && libc.sigtimer)
libc.sigtimer(sig, si, ctx);
return;
}
if (self->canceldisable) return; if (self->canceldisable) return;
if (self->cancelasync || (self->cancelpoint==1 && PC_AT_SYS(ctx))) if (self->cancelasync || (self->cancelpoint==1 && PC_AT_SYS(ctx)))
docancel(self); docancel(self);
...@@ -176,6 +175,12 @@ static void init_threads() ...@@ -176,6 +175,12 @@ static void init_threads()
static int start(void *p) static int start(void *p)
{ {
struct pthread *self = p; struct pthread *self = p;
if (self->unblock_cancel) {
sigset_t set;
sigemptyset(&set);
sigaddset(&set, SIGCANCEL);
__libc_sigprocmask(SIG_UNBLOCK, &set, 0);
}
pthread_exit(self->start(self->start_arg)); pthread_exit(self->start(self->start_arg));
return 0; return 0;
} }
...@@ -220,6 +225,7 @@ int pthread_create(pthread_t *res, const pthread_attr_t *attr, void *(*entry)(vo ...@@ -220,6 +225,7 @@ int pthread_create(pthread_t *res, const pthread_attr_t *attr, void *(*entry)(vo
new->tsd = (void *)tsd; new->tsd = (void *)tsd;
new->detached = attr->_a_detach; new->detached = attr->_a_detach;
new->attr = *attr; new->attr = *attr;
new->unblock_cancel = self->cancel;
memcpy(new->tlsdesc, self->tlsdesc, sizeof new->tlsdesc); memcpy(new->tlsdesc, self->tlsdesc, sizeof new->tlsdesc);
new->tlsdesc[1] = (uintptr_t)new; new->tlsdesc[1] = (uintptr_t)new;
stack = (void *)((uintptr_t)new-1 & ~(uintptr_t)15); stack = (void *)((uintptr_t)new-1 & ~(uintptr_t)15);
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册