[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Typo in get_reloc_expr (dyngen.c)
From: |
Ben Taylor |
Subject: |
Re: [Qemu-devel] Typo in get_reloc_expr (dyngen.c) |
Date: |
Tue, 6 Jun 2006 7:52:47 -0400 |
Fabrice or Paul:
Can we get this added? This patch has been posted to the list 4 or 5 times
in various incarnations in the last year or soj. I posted a long message with
the logs from the compile on why the current code is wrong, and why this patch
fixes
it. The basic problem is that code uses a sizeof construct, instead of the
size of
the string. What happens is all the labels end up being truncated, and the op.c
won't compile on sparc.
see analysis of this problem in
http://lists.gnu.org/archive/html/qemu-devel/2006-04/msg00494.html
Ben
---- Stuart Brady <address@hidden> wrote:
> Hi,
>
> There's a small bug in get_reloc_expr in dyngen.c.
>
> It should only affect SPARC hosts, but it can't be causing any real
> problems, or someone would have noticed it by now.
>
> I'm trying to port QEMU to PA-RISC[0], and needed this code, so it did
> actually affect me. I've attached the (one-line) patch.
> --
> Stuart Brady
>
> [0] http://homepage.ntlworld.com/wholehog/stuart/qemu/guest.html