Commit d317091d authored by Peter Maydell's avatar Peter Maydell
Browse files

linux-user: Make semihosting heap/stack fields abi_ulongs 



The fields in the TaskState heap_base, heap_limit and stack_base
are all guest addresses (representing the locations of the heap
and stack for the guest binary), so they should be abi_ulong
rather than uint32_t. (This only in practice affects ARM AArch64
since all the other semihosting implementations are 32-bit.)

Signed-off-by: default avatarPeter Maydell <peter.maydell@linaro.org>
Reviewed-by: default avatarLaurent Desnogues <laurent.desnogues@gmail.com>
Message-id: 1466783381-29506-2-git-send-email-peter.maydell@linaro.org
parent e2c8f9e4

linux-user/qemu.h

+3 −3
Original line number Diff line number Diff line
@@ -116,10 +116,10 @@ typedef struct TaskState { 
#endif

#if defined(TARGET_ARM) || defined(TARGET_M68K) || defined(TARGET_UNICORE32)

    /* Extra fields for semihosted binaries.  */

    uint32_t heap_base;

    uint32_t heap_limit;

    abi_ulong heap_base;

    abi_ulong heap_limit;

#endif

    uint32_t stack_base;

    abi_ulong stack_base;

    int used; /* non zero if used */

    struct image_info *info;

    struct linux_binprm *bprm;