qemu-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] d9b3ad: libvhost-user: Zero memory allocated


From: Peter Maydell
Subject: [Qemu-commits] [qemu/qemu] d9b3ad: libvhost-user: Zero memory allocated for VuVirtqIn...
Date: Thu, 21 Nov 2019 03:27:10 -0800

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: d9b3ade54ea7465093578cd0f792437198b572ac
      
https://github.com/qemu/qemu/commit/d9b3ade54ea7465093578cd0f792437198b572ac
  Author: Xie Yongji <address@hidden>
  Date:   2019-11-20 (Wed, 20 Nov 2019)

  Changed paths:
    M contrib/libvhost-user/libvhost-user.c

  Log Message:
  -----------
  libvhost-user: Zero memory allocated for VuVirtqInflightDesc

Use a zero-initialized VuVirtqInflightDesc struct to avoid
that scan-build reports that vq->resubmit_list[0].counter may
be garbage value in vu_check_queue_inflights().

Fixes: 5f9ff1eff ("libvhost-user: Support tracking inflight I/O in
shared memory")
Reported-by: Marc-André Lureau <address@hidden>
Signed-off-by: Xie Yongji <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Marc-André Lureau <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: ec244b17391f8f9a16e91905a91a52dd77d3759f
      
https://github.com/qemu/qemu/commit/ec244b17391f8f9a16e91905a91a52dd77d3759f
  Author: Stefan Hajnoczi <address@hidden>
  Date:   2019-11-20 (Wed, 20 Nov 2019)

  Changed paths:
    M contrib/vhost-user-input/main.c

  Log Message:
  -----------
  vhost-user-input: use free(elem) instead of g_free(elem)

The virtqueue element returned by vu_queue_pop() is allocated using
malloc(3) by virtqueue_alloc_element().  Use the matching free(3)
function instead of glib's g_free().

Signed-off-by: Stefan Hajnoczi <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>


  Commit: 6c77aa909e0eec6531e2398d0e9e60ec6a9339c6
      
https://github.com/qemu/qemu/commit/6c77aa909e0eec6531e2398d0e9e60ec6a9339c6
  Author: Igor Mammedov <address@hidden>
  Date:   2019-11-20 (Wed, 20 Nov 2019)

  Changed paths:
    M tests/bios-tables-test.c

  Log Message:
  -----------
  tests: acpi: always retain dumped ACPI tables in case of error

If IASL wasn't able to parse expected file, test will just
print warning
  "Warning! iasl couldn't parse the expected aml\n"
and remove temporary table dumped from guest.

Typically expected tables are always valid, with an exception
when patchset introduces new tables.
Make sure dumped tables are retained even if expected files
are not valid, so one could have a chance to manualy check new
tables.

Signed-off-by: Igor Mammedov <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: bf12043acc6458e7b71d19a9f030b31ec22fba12
      
https://github.com/qemu/qemu/commit/bf12043acc6458e7b71d19a9f030b31ec22fba12
  Author: Peter Maydell <address@hidden>
  Date:   2019-11-20 (Wed, 20 Nov 2019)

  Changed paths:
    M contrib/libvhost-user/libvhost-user.c
    M contrib/vhost-user-input/main.c
    M tests/bios-tables-test.c

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into staging

virtio, acpi: fixes

A couple of bugfixes.

Signed-off-by: Michael S. Tsirkin <address@hidden>

# gpg: Signature made Wed 20 Nov 2019 09:57:44 GMT
# gpg:                using RSA key 5D09FD0871C8F85B94CA8A0D281F0DB8D28D5469
# gpg:                issuer "address@hidden"
# gpg: Good signature from "Michael S. Tsirkin <address@hidden>" [full]
# gpg:                 aka "Michael S. Tsirkin <address@hidden>" [full]
# Primary key fingerprint: 0270 606B 6F3C DF3D 0B17  0970 C350 3912 AFBE 8E67
#      Subkey fingerprint: 5D09 FD08 71C8 F85B 94CA  8A0D 281F 0DB8 D28D 5469

* remotes/mst/tags/for_upstream:
  tests: acpi: always retain dumped ACPI tables in case of error
  vhost-user-input: use free(elem) instead of g_free(elem)
  libvhost-user: Zero memory allocated for VuVirtqInflightDesc

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/39e2821077e6...bf12043acc64



reply via email to

[Prev in Thread] Current Thread [Next in Thread]