diff options
| author | Markus Armbruster <armbru@redhat.com> | 2022-06-30 10:52:19 +0200 |
|---|---|---|
| committer | Laurent Vivier <laurent@vivier.eu> | 2022-08-08 07:19:26 +0200 |
| commit | a32086de4919b9affb2ab2d0112d400eaf89f607 (patch) | |
| tree | 568c1d3c5aaf4192229518e840d715666068b977 /hw/display | |
| parent | c669f22f1a47897e8d1d595d6b8a59a572f9158c (diff) | |
| download | qemu-a32086de4919b9affb2ab2d0112d400eaf89f607.zip qemu-a32086de4919b9affb2ab2d0112d400eaf89f607.tar.gz qemu-a32086de4919b9affb2ab2d0112d400eaf89f607.tar.bz2 | |
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')
0 files changed, 0 insertions, 0 deletions