diff options
| author | Richard Weinberger <richard@nod.at> | 2014-03-05 09:49:40 -0500 |
|---|---|---|
| committer | Richard Weinberger <richard@sigma-star.at> | 2014-08-06 07:04:31 -0400 |
| commit | 8e2beafa2f7f48b1392c00897578e5daf858fe59 (patch) | |
| tree | e3f650ae3cfe786476d66ca94821a533031b76b1 /arch/openrisc/kernel | |
| parent | a8040ff82ee623bfd070b920fb4b5d133f9d9953 (diff) | |
openrisc: Use sigsp()
Use sigsp() instead of the open coded variant.
Signed-off-by: Richard Weinberger <richard@nod.at>
Diffstat (limited to 'arch/openrisc/kernel')
| -rw-r--r-- | arch/openrisc/kernel/signal.c | 20 |
1 files changed, 3 insertions, 17 deletions
diff --git a/arch/openrisc/kernel/signal.c b/arch/openrisc/kernel/signal.c index 66775bc07a8e..7d1b8235bf90 100644 --- a/arch/openrisc/kernel/signal.c +++ b/arch/openrisc/kernel/signal.c | |||
| @@ -132,30 +132,16 @@ static inline unsigned long align_sigframe(unsigned long sp) | |||
| 132 | * or the alternate stack. | 132 | * or the alternate stack. |
| 133 | */ | 133 | */ |
| 134 | 134 | ||
| 135 | static inline void __user *get_sigframe(struct k_sigaction *ka, | 135 | static inline void __user *get_sigframe(struct ksignal *ksig, |
| 136 | struct pt_regs *regs, size_t frame_size) | 136 | struct pt_regs *regs, size_t frame_size) |
| 137 | { | 137 | { |
| 138 | unsigned long sp = regs->sp; | 138 | unsigned long sp = regs->sp; |
| 139 | int onsigstack = on_sig_stack(sp); | ||
| 140 | 139 | ||
| 141 | /* redzone */ | 140 | /* redzone */ |
| 142 | sp -= STACK_FRAME_OVERHEAD; | 141 | sp -= STACK_FRAME_OVERHEAD; |
| 143 | 142 | sp = sigsp(sp, ksig); | |
| 144 | /* This is the X/Open sanctioned signal stack switching. */ | ||
| 145 | if ((ka->sa.sa_flags & SA_ONSTACK) && !onsigstack) { | ||
| 146 | if (current->sas_ss_size) | ||
| 147 | sp = current->sas_ss_sp + current->sas_ss_size; | ||
| 148 | } | ||
| 149 | |||
| 150 | sp = align_sigframe(sp - frame_size); | 143 | sp = align_sigframe(sp - frame_size); |
| 151 | 144 | ||
| 152 | /* | ||
| 153 | * If we are on the alternate signal stack and would overflow it, don't. | ||
| 154 | * Return an always-bogus address instead so we will die with SIGSEGV. | ||
| 155 | */ | ||
| 156 | if (onsigstack && !likely(on_sig_stack(sp))) | ||
| 157 | return (void __user *)-1L; | ||
| 158 | |||
| 159 | return (void __user *)sp; | 145 | return (void __user *)sp; |
| 160 | } | 146 | } |
| 161 | 147 | ||
| @@ -173,7 +159,7 @@ static int setup_rt_frame(struct ksignal *ksig, sigset_t *set, | |||
| 173 | unsigned long return_ip; | 159 | unsigned long return_ip; |
| 174 | int err = 0; | 160 | int err = 0; |
| 175 | 161 | ||
| 176 | frame = get_sigframe(&ksig->ka, regs, sizeof(*frame)); | 162 | frame = get_sigframe(ksig, regs, sizeof(*frame)); |
| 177 | 163 | ||
| 178 | if (!access_ok(VERIFY_WRITE, frame, sizeof(*frame))) | 164 | if (!access_ok(VERIFY_WRITE, frame, sizeof(*frame))) |
| 179 | return -EFAULT; | 165 | return -EFAULT; |
