diff options
| -rw-r--r-- | include/linux/sched.h | 2 | ||||
| -rw-r--r-- | kernel/exit.c | 3 | ||||
| -rw-r--r-- | kernel/fork.c | 8 |
3 files changed, 5 insertions, 8 deletions
diff --git a/include/linux/sched.h b/include/linux/sched.h index 99855f694ebd..d04186d8cc68 100644 --- a/include/linux/sched.h +++ b/include/linux/sched.h | |||
| @@ -1151,7 +1151,7 @@ extern void exit_thread(void); | |||
| 1151 | 1151 | ||
| 1152 | extern void exit_files(struct task_struct *); | 1152 | extern void exit_files(struct task_struct *); |
| 1153 | extern void __cleanup_signal(struct signal_struct *); | 1153 | extern void __cleanup_signal(struct signal_struct *); |
| 1154 | extern void cleanup_sighand(struct task_struct *); | 1154 | extern void __cleanup_sighand(struct sighand_struct *); |
| 1155 | extern void exit_itimers(struct signal_struct *); | 1155 | extern void exit_itimers(struct signal_struct *); |
| 1156 | 1156 | ||
| 1157 | extern NORET_TYPE void do_group_exit(int); | 1157 | extern NORET_TYPE void do_group_exit(int); |
diff --git a/kernel/exit.c b/kernel/exit.c index 22399caf7574..bc0ec674d3f4 100644 --- a/kernel/exit.c +++ b/kernel/exit.c | |||
| @@ -114,10 +114,11 @@ static void __exit_signal(struct task_struct *tsk) | |||
| 114 | __unhash_process(tsk); | 114 | __unhash_process(tsk); |
| 115 | 115 | ||
| 116 | tsk->signal = NULL; | 116 | tsk->signal = NULL; |
| 117 | cleanup_sighand(tsk); | 117 | tsk->sighand = NULL; |
| 118 | spin_unlock(&sighand->siglock); | 118 | spin_unlock(&sighand->siglock); |
| 119 | rcu_read_unlock(); | 119 | rcu_read_unlock(); |
| 120 | 120 | ||
| 121 | __cleanup_sighand(sighand); | ||
| 121 | clear_tsk_thread_flag(tsk,TIF_SIGPENDING); | 122 | clear_tsk_thread_flag(tsk,TIF_SIGPENDING); |
| 122 | flush_sigqueue(&tsk->pending); | 123 | flush_sigqueue(&tsk->pending); |
| 123 | if (sig) { | 124 | if (sig) { |
diff --git a/kernel/fork.c b/kernel/fork.c index aa50c848fae7..b3f7a1bb5e55 100644 --- a/kernel/fork.c +++ b/kernel/fork.c | |||
| @@ -803,12 +803,8 @@ static inline int copy_sighand(unsigned long clone_flags, struct task_struct * t | |||
| 803 | return 0; | 803 | return 0; |
| 804 | } | 804 | } |
| 805 | 805 | ||
| 806 | void cleanup_sighand(struct task_struct *tsk) | 806 | void __cleanup_sighand(struct sighand_struct *sighand) |
| 807 | { | 807 | { |
| 808 | struct sighand_struct * sighand = tsk->sighand; | ||
| 809 | |||
| 810 | /* Ok, we're done with the signal handlers */ | ||
| 811 | tsk->sighand = NULL; | ||
| 812 | if (atomic_dec_and_test(&sighand->count)) | 808 | if (atomic_dec_and_test(&sighand->count)) |
| 813 | kmem_cache_free(sighand_cachep, sighand); | 809 | kmem_cache_free(sighand_cachep, sighand); |
| 814 | } | 810 | } |
| @@ -1233,7 +1229,7 @@ bad_fork_cleanup_mm: | |||
| 1233 | bad_fork_cleanup_signal: | 1229 | bad_fork_cleanup_signal: |
| 1234 | cleanup_signal(p); | 1230 | cleanup_signal(p); |
| 1235 | bad_fork_cleanup_sighand: | 1231 | bad_fork_cleanup_sighand: |
| 1236 | cleanup_sighand(p); | 1232 | __cleanup_sighand(p->sighand); |
| 1237 | bad_fork_cleanup_fs: | 1233 | bad_fork_cleanup_fs: |
| 1238 | exit_fs(p); /* blocking */ | 1234 | exit_fs(p); /* blocking */ |
| 1239 | bad_fork_cleanup_files: | 1235 | bad_fork_cleanup_files: |
