Re: [PULL 00/22] -Werror=maybe-uninitialized fixes

[Thomas Huth <thuth@redhat.com>]

On 02/10/2024 10.36, marcandre.lureau@redhat.com wrote:
> From: Marc-André Lureau <marcandre.lureau@redhat.com>
> 
> The following changes since commit 718780d20470c66a3a36d036b29148d5809dc855:
> 
>    Merge tag 'pull-nvme-20241001' of https://gitlab.com/birkelund/qemu into staging (2024-10-01 11:34:07 +0100)
> 
> are available in the Git repository at:
> 
>    https://gitlab.com/marcandre.lureau/qemu.git tags/warn-pull-request
> 
> for you to fetch changes up to baad82ecfb27474f17f4318502725622ab7170fc:
> 
>    qom/object: fix -Werror=maybe-uninitialized (2024-10-02 11:36:12 +0400)
> 
> ----------------------------------------------------------------
> -Werror=maybe-uninitialized fixes
> 
> ----------------------------------------------------------------
> 
> Marc-André Lureau (22):
>    util/coroutine: fix -Werror=maybe-uninitialized false-positive
>    util/timer: fix -Werror=maybe-uninitialized false-positive
>    hw/qxl: fix -Werror=maybe-uninitialized false-positives
>    nbd: fix -Werror=maybe-uninitialized false-positive
>    block/mirror: fix -Werror=maybe-uninitialized false-positive
>    block/mirror: fix -Werror=maybe-uninitialized false-positive
>    block/stream: fix -Werror=maybe-uninitialized false-positives
>    hw/ahci: fix -Werror=maybe-uninitialized false-positive
>    hw/vhost-scsi: fix -Werror=maybe-uninitialized
>    hw/sdhci: fix -Werror=maybe-uninitialized false-positive
>    block/block-copy: fix -Werror=maybe-uninitialized false-positive
>    migration: fix -Werror=maybe-uninitialized false-positives
>    hw/virtio-blk: fix -Werror=maybe-uninitialized false-positive
>    migration: fix -Werror=maybe-uninitialized false-positive
>    linux-user/hppa: fix -Werror=maybe-uninitialized false-positive
>    target/loongarch: fix -Werror=maybe-uninitialized false-positive
>    tests: fix -Werror=maybe-uninitialized false-positive
>    hw/virtio: fix -Werror=maybe-uninitialized
>    hw/virtio: freeing leaked memory from vhost_svq_get_buf in
>      vhost_svq_poll
>    block: fix -Werror=maybe-uninitialized false-positive
>    fsdep/9p: fix -Werror=maybe-uninitialized false-positive
>    qom/object: fix -Werror=maybe-uninitialized
> 
>   block/block-copy.c                 |  2 +-
>   block/file-posix.c                 |  2 +-
>   block/mirror.c                     |  8 ++++----
>   block/stream.c                     |  6 +++---
>   fsdev/9p-iov-marshal.c             | 15 ++++++++++++---
>   hw/block/virtio-blk.c              |  2 +-
>   hw/display/qxl.c                   |  4 ++--
>   hw/ide/ahci.c                      |  3 ++-
>   hw/scsi/vhost-scsi.c               |  2 +-
>   hw/sd/sdhci.c                      |  2 +-
>   hw/virtio/vhost-shadow-virtqueue.c |  6 ++++--
>   linux-user/hppa/cpu_loop.c         |  2 ++
>   migration/dirtyrate.c              |  4 ++--
>   migration/migration.c              |  2 +-
>   migration/ram.c                    |  2 +-
>   nbd/client-connection.c            |  2 +-
>   qom/object.c                       |  2 +-
>   target/loongarch/gdbstub.c         | 26 ++++++++++++++------------
>   tests/unit/test-bdrv-drain.c       |  2 +-
>   tests/unit/test-block-iothread.c   |  2 +-
>   util/qemu-coroutine.c              |  2 +-
>   util/qemu-timer.c                  |  6 +++---
>   roms/openbios                      |  2 +-

Replying here as well, so that Peter has a chance to see it:

The update to roms/openbios in patch 03 seems to have slipped in by 
accident, so I'd suggest to fix that first before merging this.

  Thomas