aboutsummaryrefslogtreecommitdiff
path: root/linux-user
diff options
context:
space:
mode:
authorBlue Swirl <blauwirbel@gmail.com>2010-09-18 05:53:14 +0000
committerBlue Swirl <blauwirbel@gmail.com>2010-09-18 05:53:14 +0000
commitf562e716c962451af359e5a59ee9dbbd66b9fee8 (patch)
treef5f3e4d75dc059a4b0529a9d6b8f2638e91c7a18 /linux-user
parentd0b3e4f5f4a29d48250887e5c0b3c65bc4dc6d13 (diff)
downloadqemu-f562e716c962451af359e5a59ee9dbbd66b9fee8.zip
qemu-f562e716c962451af359e5a59ee9dbbd66b9fee8.tar.gz
qemu-f562e716c962451af359e5a59ee9dbbd66b9fee8.tar.bz2
linux-user: improve flatload error checking
Because of the use of unsigned type, possible errors during load were ignored. Fix by using a signed type. This also avoids a warning with GCC flag -Wtype-limits. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
Diffstat (limited to 'linux-user')
-rw-r--r--linux-user/flatload.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/linux-user/flatload.c b/linux-user/flatload.c
index 8ad130a..8f9f4a5 100644
--- a/linux-user/flatload.c
+++ b/linux-user/flatload.c
@@ -383,7 +383,8 @@ static int load_flat_file(struct linux_binprm * bprm,
struct lib_info *libinfo, int id, abi_ulong *extra_stack)
{
struct flat_hdr * hdr;
- abi_ulong textpos = 0, datapos = 0, result;
+ abi_ulong textpos = 0, datapos = 0;
+ abi_long result;
abi_ulong realdatastart = 0;
abi_ulong text_len, data_len, bss_len, stack_len, flags;
abi_ulong memp = 0; /* for finding the brk area */