2013-08-17 02:43:59 +02:00
|
|
|
From 8b6c3923497e577546cb1a388decfa1e63496d10 Mon Sep 17 00:00:00 2001
|
2012-12-13 14:31:35 +01:00
|
|
|
From: Alexander Graf <agraf@suse.de>
|
|
|
|
Date: Thu, 13 Dec 2012 14:29:22 +0100
|
2013-07-16 04:49:49 +02:00
|
|
|
Subject: [PATCH] linux-user: lseek: explicitly cast non-set offsets to signed
|
2012-12-13 14:31:35 +01:00
|
|
|
|
2013-07-16 04:49:49 +02:00
|
|
|
When doing lseek, SEEK_SET indicates that the offset is an unsigned variable.
|
|
|
|
Other seek types have parameters that can be negative.
|
2012-12-13 14:31:35 +01:00
|
|
|
|
|
|
|
When converting from 32bit to 64bit parameters, we need to take this into
|
2013-07-16 04:49:49 +02:00
|
|
|
account and enable SEEK_END and SEEK_CUR to be negative, while SEEK_SET stays
|
|
|
|
absolute positioned which we need to maintain as unsigned.
|
2012-12-13 14:31:35 +01:00
|
|
|
|
|
|
|
Signed-off-by: Alexander Graf <agraf@suse.de>
|
|
|
|
---
|
2013-10-08 20:29:33 +02:00
|
|
|
linux-user/syscall.c | 9 +++++++--
|
|
|
|
1 file changed, 7 insertions(+), 2 deletions(-)
|
2012-12-13 14:31:35 +01:00
|
|
|
|
|
|
|
diff --git a/linux-user/syscall.c b/linux-user/syscall.c
|
2013-07-30 14:36:48 +02:00
|
|
|
index ab30b58..4136873 100644
|
2012-12-13 14:31:35 +01:00
|
|
|
--- a/linux-user/syscall.c
|
|
|
|
+++ b/linux-user/syscall.c
|
2013-07-30 14:36:48 +02:00
|
|
|
@@ -5548,9 +5548,14 @@ abi_long do_syscall(void *cpu_env, int num, abi_ulong arg1,
|
2012-12-13 14:31:35 +01:00
|
|
|
case TARGET_NR_oldstat:
|
|
|
|
goto unimplemented;
|
|
|
|
#endif
|
|
|
|
- case TARGET_NR_lseek:
|
|
|
|
- ret = get_errno(lseek(arg1, arg2, arg3));
|
|
|
|
+ case TARGET_NR_lseek: {
|
|
|
|
+ off_t off = arg2;
|
2013-07-16 04:49:49 +02:00
|
|
|
+ if (arg3 != SEEK_SET) {
|
2012-12-13 14:31:35 +01:00
|
|
|
+ off = (abi_long)arg2;
|
|
|
|
+ }
|
|
|
|
+ ret = get_errno(lseek(arg1, off, arg3));
|
|
|
|
break;
|
|
|
|
+ }
|
|
|
|
#if defined(TARGET_NR_getxpid) && defined(TARGET_ALPHA)
|
|
|
|
/* Alpha specific */
|
|
|
|
case TARGET_NR_getxpid:
|