Re: [Qemu-arm] [PATCH] target-arm/arm-semi.c: In SYS_HEAPINFO use correct type for 'limit'

[Laurent Desnogues]

On Mon, Jul 4, 2016 at 6:49 PM, Peter Maydell <address@hidden> wrote:
> In commit f5666418c4 most of the SYS_HEAPINFO implementation was
> fixed to use target_ulong rather than uint32_t, but the 'limit'
> variable was not changed.
>
> Reported-by: Laurent Desnogues <address@hidden>
> Signed-off-by: Peter Maydell <address@hidden>

Reviewed-by: Laurent Desnogues <address@hidden>

Thanks,

Laurent

> ---
> This was actually pointed out in code review for the original
> patch, but I accidentally forgot to make the change when I
> applied the patch to the target-arm queue :-(
> ---
>  target-arm/arm-semi.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target-arm/arm-semi.c b/target-arm/arm-semi.c
> index d50726f..7cac873 100644
> --- a/target-arm/arm-semi.c
> +++ b/target-arm/arm-semi.c
> @@ -565,7 +565,7 @@ target_ulong do_arm_semihosting(CPUARMState *env)
>      case TARGET_SYS_HEAPINFO:
>          {
>              target_ulong retvals[4];
> -            uint32_t limit;
> +            target_ulong limit;
>              int i;
>
>              GET_ARG(0);
> --
> 1.9.1
>