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

Hi,

While reviewing some related code, I noticed array properties are leaking.

I also wonder why I refactored object_property_del_all() with a HashTable, and
whether it's really necessary. Here is also a RFC to simplify that code.

Marc-André Lureau (6):
  qdev: make release_string() idempotent
  qdev: make release_drive() idempotent
  qdev: make release_tpm() idempotent
  qdev: make release_prop_array() free array and idempotent
  tests: add /qdev/free-properties test
  RFC: qom/object: simplify object_property_del_all()

 backends/tpm/tpm_util.c          |  1 +
 hw/core/qdev-properties-system.c |  1 +
 hw/core/qdev-properties.c        | 18 +++---
 qom/object.c                     | 18 ++----
 tests/unit/test-qdev.c           | 96 ++++++++++++++++++++++++++++++++
 tests/unit/meson.build           |  1 +
 6 files changed, 114 insertions(+), 21 deletions(-)
 create mode 100644 tests/unit/test-qdev.c

-- 
2.49.0


Reply via email to