]> git.ipfire.org Git - thirdparty/libvirt.git/commit
virJSONValueArrayAppend: Clear pointer when taking ownership of passed value
authorPeter Krempa <pkrempa@redhat.com>
Thu, 11 Feb 2021 16:57:45 +0000 (17:57 +0100)
committerPeter Krempa <pkrempa@redhat.com>
Sat, 20 Feb 2021 12:26:37 +0000 (13:26 +0100)
commit6431b20c3ef2fdf31bcce0a04439c2dc3d2a61bc
tree8d94f8bcc43175db152e1a9bea38e80bef9f3a02
parent962d80e75107858c239472616c9ea5cfccd6e87c
virJSONValueArrayAppend: Clear pointer when taking ownership of passed value

The parent array takes ownership of the inserted value once all checks
pass. Don't make the callers second-guess when that happens and modify
the function to take a double pointer so that it can be cleared once the
ownership is taken.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
19 files changed:
src/locking/lock_daemon.c
src/logging/log_handler.c
src/network/leaseshelper.c
src/node_device/node_device_driver.c
src/qemu/qemu_agent.c
src/qemu/qemu_block.c
src/qemu/qemu_command.c
src/qemu/qemu_firmware.c
src/qemu/qemu_migration_cookie.c
src/qemu/qemu_migration_params.c
src/qemu/qemu_monitor_json.c
src/rpc/virnetserver.c
src/rpc/virnetserverservice.c
src/util/virjson.c
src/util/virjson.h
src/util/virlease.c
src/util/virlockspace.c
src/util/virmacmap.c
tests/testutilsqemuschema.c