diff options
| author | Markus Armbruster | 2022-06-30 10:52:19 +0200 |
|---|---|---|
| committer | Laurent Vivier | 2022-08-08 07:19:26 +0200 |
| commit | a32086de4919b9affb2ab2d0112d400eaf89f607 (patch) | |
| tree | 568c1d3c5aaf4192229518e840d715666068b977 /hw/display/xlnx_dp.c | |
| parent | Merge tag 'pull-la-20220805' of https://gitlab.com/rth7680/qemu into staging (diff) | |
| download | qemu-a32086de4919b9affb2ab2d0112d400eaf89f607.tar.gz qemu-a32086de4919b9affb2ab2d0112d400eaf89f607.tar.xz qemu-a32086de4919b9affb2ab2d0112d400eaf89f607.zip | |
contrib/vhost-user-blk: Clean up deallocation of VuVirtqElement
We allocate VuVirtqElement with g_malloc() in
virtqueue_alloc_element(), but free it with free() in
vhost-user-blk.c. Harmless, but use g_free() anyway.
One of the calls is guarded by a "not null" condition. Useless,
because it cannot be null (it's dereferenced right before), and even
it it could be, free() and g_free() do the right thing. Drop the
conditional.
Fixes: Coverity CID 1490290
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Raphael Norwitz <raphael.norwitz@nutanix.com>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Message-Id: <20220630085219.1305519-1-armbru@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
Diffstat (limited to 'hw/display/xlnx_dp.c')
0 files changed, 0 insertions, 0 deletions