qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] tci: don't write zero for reloc in tci_out_labe


From: Blue Swirl
Subject: Re: [Qemu-devel] [PATCH] tci: don't write zero for reloc in tci_out_label
Date: Sun, 24 Jun 2012 12:27:20 +0000

Thanks, applied.

On Tue, Jun 19, 2012 at 2:31 AM, Scott Wood <address@hidden> wrote:
> If tci_out_label is called in the context of tcg_gen_code_search_pc, we
> could be overwriting an already patched relocation with zero -- and not
> repatch it because the set_label is past search_pc, causing a QEMU crash
> when it tries to branch to a zero label.
>
> Not writing anything to the relocation area seems to be in line with what
> other backends do from the couple I looked at (x86, ppc).
>
> Signed-off-by: Scott Wood <address@hidden>
> ---
>  tcg/tci/tcg-target.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/tcg/tci/tcg-target.c b/tcg/tci/tcg-target.c
> index 453f187..3c6b0f5 100644
> --- a/tcg/tci/tcg-target.c
> +++ b/tcg/tci/tcg-target.c
> @@ -487,7 +487,7 @@ static void tci_out_label(TCGContext *s, TCGArg arg)
>         assert(label->u.value);
>     } else {
>         tcg_out_reloc(s, s->code_ptr, sizeof(tcg_target_ulong), arg, 0);
> -        tcg_out_i(s, 0);
> +        s->code_ptr += sizeof(tcg_target_ulong);
>     }
>  }
>
> --
> 1.7.5.4
>
>



reply via email to

[Prev in Thread] Current Thread [Next in Thread]