Changes in v2: * Split the patch in several parts so the various submaintainers can look at it.
When looking at address_space_read_continue I noticed the 'len' variable was a signed int, while it clearly should be a hwaddr or some other unsigned type. I started looking around and saw that this error had spread all over the code. I replaced the signed int by their unsigned counterparts as much as possible. At the suggestion of Paolo I'm splitting the patch in several parts according to the MAINTAINERS file. cpus.c | 2 +- disas/cris.c | 2 +- disas/s390.c | 3 ++- disas/sh4.c | 2 +- dma-helpers.c | 2 +- exec.c | 29 +++++++++++++++-------------- gdbstub.c | 3 ++- hw/arm/boot.c | 2 +- hw/arm/musicpal.c | 2 +- hw/audio/marvell_88w8618.c | 2 +- hw/audio/milkymist-ac97.c | 4 ++-- hw/dma/sun4m_iommu.c | 4 ++-- hw/net/dp8393x.c | 8 ++++---- hw/net/mcf_fec.c | 4 ++-- hw/net/vmware_utils.h | 17 +++++++++-------- hw/net/xgmac.c | 4 ++-- hw/s390x/virtio-ccw.c | 2 +- include/exec/cpu-all.h | 2 +- include/exec/cpu-common.h | 10 +++++----- include/exec/memory.h | 10 +++++----- include/hw/sparc/sun4m.h | 6 +++--- include/qom/cpu.h | 2 +- kvm-all.c | 2 +- monitor.c | 3 ++- scripts/coverity-model.c | 2 +- target-s390x/mmu_helper.c | 4 ++-- target-sparc/cpu.h | 2 +- target-sparc/mmu_helper.c | 6 +++--- 28 files changed, 73 insertions(+), 68 deletions(-) -- 2.7.1
