Re: [PATCH-for-5.2 0/4] misc: Trivial format string fixes

[Dov Murik]

On 03/11/2020 15:58, Dov Murik wrote:
> On 03/11/2020 13:25, Philippe Mathieu-Daudé wrote:
> > IIUC qemu-trivial@ doesn't queue patches during freeze,
> >
> > so it might be easier if patches are queued by respective
> >
> > subsystem maintainers.
> >
> >
> >
> > Philippe Mathieu-Daudé (4):
> >
> >    hw/display/cirrus_vga: Remove debugging code commented out
> >
> >    hw/display/cirrus_vga: Fix hexadecimal format string specifier
> >
> >    hw/ppc/spapr_tpm_proxy: Fix hexadecimal format string specifier
> >
> >    migration/ram: Fix hexadecimal format string specifier
> >
> >
> >
> >   hw/display/cirrus_vga.c | 20 +-------------------
> >
> >   migration/ram.c         |  2 +-
> >
> >   hw/ppc/trace-events     |  2 +-
> >
> >   3 files changed, 3 insertions(+), 21 deletions(-)
>
>
> There's at least one more easy fix:
>
> hw/misc/trace-events:106:mos6522_get_next_irq_time(uint16_t latch, 
> int64_t d, int64_t delta) "latch=%d counter=0x%"PRId64 " 
> delta_next=0x%"PRId64
>
> but I have no idea how to test this.
>
> -Dov

... and one more in hw/usb/u2f-passthru.c:348 :

        error_report("%s: Bad written size (req 0x%zu, val 0x%zd)",
                     TYPE_U2F_PASSTHRU, sizeof(host_packet), written);


Again, I have no idea how to test/trigger these areas in the code.

-Dov