On Sun, 3 Feb 2013 21:33:16 +0100
Stefan Weil s...@weilnetz.de wrote:
gcc with -Wextra complains about an ordered pointer comparison:
target-s390x/helper.c:660:27: warning:
ordered comparison of pointer with integer zero [-Wextra]
Obviously the index was missing in the code.
Signed-off-by: Stefan Weil s...@weilnetz.de
---
I hope my analysis and the fix is correct - please review.
For local builds, I always compile with -Wextra. This bug shows that -Wextra
would be good as a default compiler option for QEMU. Of course some extra
warnings must be explicitly disabled then. I'll send a patch for this after
1.4.
Regards,
Stefan W.
target-s390x/helper.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/target-s390x/helper.c b/target-s390x/helper.c
index 3180b90..8bd84ef 100644
--- a/target-s390x/helper.c
+++ b/target-s390x/helper.c
@@ -657,7 +657,7 @@ static void do_io_interrupt(CPUS390XState *env)
cpu_unmap_lowcore(lowcore);
env-io_index[isc]--;
-if (env-io_index = 0) {
+if (env-io_index[isc] = 0) {
disable = 0;
}
break;
Oops, obvious bug when you look at it :)
Acked-by: Cornelia Huck cornelia.h...@de.ibm.com