On Mon, Jul 24, 2017 at 11:38 AM, Eric Blake <[email protected]> wrote: > On 07/24/2017 01:27 PM, Philippe Mathieu-Daudé wrote: >> since a negative value means it errored. >> >> hw/core/loader.c:149:9: warning: Loss of sign in implicit conversion >> if (size > max_sz) { >> ^~~~ >> hw/core/loader.c:171:9: warning: Loss of sign in implicit conversion >> if (size > memory_region_size(mr)) { >> ^~~~ >> >> Reported-by: Clang Static Analyzer >> Signed-off-by: Philippe Mathieu-Daudé <[email protected]> >> --- >> hw/core/loader.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) > > Reviewed-by: Eric Blake <[email protected]>
Reviewed-by: Alistair Francis <[email protected]> Thanks, Alistair > > -- > Eric Blake, Principal Software Engineer > Red Hat, Inc. +1-919-301-3266 > Virtualization: qemu.org | libvirt.org >
