From 3b8ee1c51209b3fa49c0bfb26aba3d04f6f4cd35 Mon Sep 17 00:00:00 2001 From: davidxu Date: Thu, 7 Oct 2004 13:52:18 +0000 Subject: Follow kernel change, restore signal mask correctly by using a command of kse_thr_interrupt. --- lib/libkse/thread/thr_execve.c | 38 +++++++++++++++++--------------------- 1 file changed, 17 insertions(+), 21 deletions(-) (limited to 'lib/libkse') diff --git a/lib/libkse/thread/thr_execve.c b/lib/libkse/thread/thr_execve.c index db5b858..2904a2f 100644 --- a/lib/libkse/thread/thr_execve.c +++ b/lib/libkse/thread/thr_execve.c @@ -39,30 +39,26 @@ __weak_reference(_execve, execve); int _execve(const char *name, char *const *argv, char *const *envp) { - sigset_t omask; + struct kse_execve_args args; struct pthread *curthread; - kse_critical_t crit; - int saved_errno; int ret; - /* - * When exec'ing, set the kernel signal mask to the thread's - * signal mask to satisfy POSIX requirements. We have to enter - * a critical region so that the kernel thread doesn't get - * changed out from under us after setting the signal mask. - */ - curthread = _get_curthread(); - crit = _kse_critical_enter(); - __sys_sigprocmask(SIG_SETMASK, &curthread->sigmask, &omask); - ret = __sys_execve(name, argv, envp); + if (!_kse_isthreaded()) + ret = __sys_execve(name, argv, envp); + else { + /* + * When exec'ing, set the kernel signal mask to the thread's + * signal mask to satisfy POSIX requirements. + */ + curthread = _get_curthread(); + args.sigmask = curthread->sigmask; + args.sigpend = curthread->sigpend; + args.path = (char *)name; + args.argv = (char **)argv; + args.envp = (char **)envp; + args.reserved = NULL; + ret = kse_thr_interrupt(NULL, KSE_INTR_EXECVE, (long)&args); + } - /* - * If something went wrong, set the signal mask back but don't - * destroy errno. - */ - saved_errno = errno; - __sys_sigprocmask(SIG_SETMASK, &omask, NULL); - errno = saved_errno; - _kse_critical_leave(crit); return (ret); } -- cgit v1.1