[v2] -Werror=maybe-uninitialized fixes

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

marcandre.lureau@redhat.com posted 22 patches 1 month, 3 weeks ago

Diff against v1 v1 v2 v2 v3

Only 0 patches received!

There is a newer version of this series

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 +++---
22 files changed, 60 insertions(+), 44 deletions(-)

Expand all Fold all

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

Posted by marcandre.lureau@redhat.com 1 month, 3 weeks ago

From: Marc-André Lureau <marcandre.lureau@redhat.com>

The following changes since commit 062cfce8d4c077800d252b84c65da8a2dd03fd6f:

  Merge tag 'pull-target-arm-20241001' of https://git.linaro.org/people/pmaydell/qemu-arm into staging (2024-10-01 17:40:25 +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 8f3375434d45e56db51b5ecd4d8a929146ba5641:

  qom/object: fix -Werror=maybe-uninitialized (2024-10-02 16:14:29 +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 +++---
 22 files changed, 60 insertions(+), 44 deletions(-)

-- 
2.45.2.827.g557ae147e6

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

Posted by Peter Maydell 1 month, 3 weeks ago

On Wed, 2 Oct 2024 at 13:34, <marcandre.lureau@redhat.com> wrote:
>
> From: Marc-André Lureau <marcandre.lureau@redhat.com>
>
> The following changes since commit 062cfce8d4c077800d252b84c65da8a2dd03fd6f:
>
>   Merge tag 'pull-target-arm-20241001' of https://git.linaro.org/people/pmaydell/qemu-arm into staging (2024-10-01 17:40:25 +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 8f3375434d45e56db51b5ecd4d8a929146ba5641:
>
>   qom/object: fix -Werror=maybe-uninitialized (2024-10-02 16:14:29 +0400)
>
> ----------------------------------------------------------------
> -Werror=maybe-uninitialized fixes
>
> ----------------------------------------------------------------


Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/9.2
for any user-visible changes.

-- PMM