| Message ID | 1455556977-3644-5-git-send-email-peter.maydell@linaro.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On 15.02.2016 20:22, Peter Maydell wrote: > When linux-user code is calling cpsr_write(), use a restrictive > mask to ensure we are limiting the set of CPSR bits we update. > In particular, don't allow the mode bits to be changed. > > Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Reviewed-by: Sergey Fedorov <serge.fdrv@gmail.com> > --- > linux-user/main.c | 3 ++- > linux-user/signal.c | 2 +- > 2 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/linux-user/main.c b/linux-user/main.c > index 1269470..c467fa7 100644 > --- a/linux-user/main.c > +++ b/linux-user/main.c > @@ -4446,7 +4446,8 @@ int main(int argc, char **argv, char **envp) > #elif defined(TARGET_ARM) > { > int i; > - cpsr_write(env, regs->uregs[16], 0xffffffff, CPSRWriteByInstr); > + cpsr_write(env, regs->uregs[16], CPSR_USER | CPSR_EXEC, > + CPSRWriteByInstr); > for(i = 0; i < 16; i++) { > env->regs[i] = regs->uregs[i]; > } > diff --git a/linux-user/signal.c b/linux-user/signal.c > index 82f81c7..962111c 100644 > --- a/linux-user/signal.c > +++ b/linux-user/signal.c > @@ -1611,7 +1611,7 @@ setup_return(CPUARMState *env, struct target_sigaction *ka, > env->regs[13] = frame_addr; > env->regs[14] = retcode; > env->regs[15] = handler & (thumb ? ~1 : ~3); > - cpsr_write(env, cpsr, 0xffffffff, CPSRWriteByInstr); > + cpsr_write(env, cpsr, CPSR_IT | CPSR_T, CPSRWriteByInstr); > } > > static abi_ulong *setup_sigframe_v2_vfp(abi_ulong *regspace, CPUARMState *env)
diff --git a/linux-user/main.c b/linux-user/main.c index 1269470..c467fa7 100644 --- a/linux-user/main.c +++ b/linux-user/main.c @@ -4446,7 +4446,8 @@ int main(int argc, char **argv, char **envp) #elif defined(TARGET_ARM) { int i; - cpsr_write(env, regs->uregs[16], 0xffffffff, CPSRWriteByInstr); + cpsr_write(env, regs->uregs[16], CPSR_USER | CPSR_EXEC, + CPSRWriteByInstr); for(i = 0; i < 16; i++) { env->regs[i] = regs->uregs[i]; } diff --git a/linux-user/signal.c b/linux-user/signal.c index 82f81c7..962111c 100644 --- a/linux-user/signal.c +++ b/linux-user/signal.c @@ -1611,7 +1611,7 @@ setup_return(CPUARMState *env, struct target_sigaction *ka, env->regs[13] = frame_addr; env->regs[14] = retcode; env->regs[15] = handler & (thumb ? ~1 : ~3); - cpsr_write(env, cpsr, 0xffffffff, CPSRWriteByInstr); + cpsr_write(env, cpsr, CPSR_IT | CPSR_T, CPSRWriteByInstr); } static abi_ulong *setup_sigframe_v2_vfp(abi_ulong *regspace, CPUARMState *env)
When linux-user code is calling cpsr_write(), use a restrictive mask to ensure we are limiting the set of CPSR bits we update. In particular, don't allow the mode bits to be changed. Signed-off-by: Peter Maydell <peter.maydell@linaro.org> --- linux-user/main.c | 3 ++- linux-user/signal.c | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-)