summaryrefslogtreecommitdiffstats
path: root/user-exec.c
diff options
context:
space:
mode:
authorAlexander Graf <agraf@suse.de>2013-07-06 14:17:49 +0200
committerRiku Voipio <riku.voipio@linaro.org>2013-07-23 17:28:28 +0300
commit732f9e89a1c737f738c445ff24929a1bc137d1a9 (patch)
tree59c38eafdb70fd6f6eb444c775b842af8185ffdf /user-exec.c
parent82f05b69e6b701157b4a2e7d76ae6cf5542d66c9 (diff)
downloadhqemu-732f9e89a1c737f738c445ff24929a1bc137d1a9.zip
hqemu-732f9e89a1c737f738c445ff24929a1bc137d1a9.tar.gz
linux-user: fix segmentation fault passing with h2g(x) != x
When forwarding a segmentation fault into the guest process, we were passing the host's address directly into the guest process's signal descriptor. That obviously confused the guest process, since it didn't know what to make of the (usually 32-bit truncated) address. Passing in h2g(address) makes the guest process a lot happier. To make the code more obvious, introduce a h2g_nocheck() macro that does the same as h2g(), but allows us to convert addresses that may be outside of guest mapped range into the guest's view of address space. This fixes java running in arm-linux-user for me. Signed-off-by: Alexander Graf <agraf@suse.de> Signed-off-by: Riku Voipio <riku.voipio@linaro.org>
Diffstat (limited to 'user-exec.c')
-rw-r--r--user-exec.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/user-exec.c b/user-exec.c
index d45ca8e..82bfa66 100644
--- a/user-exec.c
+++ b/user-exec.c
@@ -95,6 +95,10 @@ static inline int handle_cpu_signal(uintptr_t pc, unsigned long address,
return 1;
}
+ /* Convert forcefully to guest address space, invalid addresses
+ are still valid segv ones */
+ address = h2g_nocheck(address);
+
env = current_cpu->env_ptr;
/* see if it is an MMU fault */
ret = cpu_handle_mmu_fault(env, address, is_write, MMU_USER_IDX);
OpenPOWER on IntegriCloud