Patchwork [3/8] linux-user: syscall should use sanitized arg1

login
register
mail settings
Submitter Peter Maydell
Date June 16, 2011, 4:37 p.m.
Message ID <1308242235-8261-4-git-send-email-peter.maydell@linaro.org>
Download mbox | patch
Permalink /patch/100682/
State New
Headers show

Comments

Peter Maydell - June 16, 2011, 4:37 p.m.
From: Juan Quintela <quintela@redhat.com>

Looking at the other architectures, we should be using "how" not "arg1".

Signed-off-by: Juan Quintela <quintela@redhat.com>
[peter.maydell@linaro.org: remove unnecessary initialisation of how]
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
 linux-user/syscall.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

Patch

diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index f3d03b0..2635b75 100644
--- a/linux-user/syscall.c
+++ b/linux-user/syscall.c
@@ -7058,7 +7058,7 @@  abi_long do_syscall(void *cpu_env, int num, abi_long arg1,
     case TARGET_NR_osf_sigprocmask:
         {
             abi_ulong mask;
-            int how = arg1;
+            int how;
             sigset_t set, oldset;
 
             switch(arg1) {
@@ -7077,7 +7077,7 @@  abi_long do_syscall(void *cpu_env, int num, abi_long arg1,
             }
             mask = arg2;
             target_to_host_old_sigset(&set, &mask);
-            sigprocmask(arg1, &set, &oldset);
+            sigprocmask(how, &set, &oldset);
             host_to_target_old_sigset(&mask, &oldset);
             ret = mask;
         }