]> www.infradead.org Git - users/dwmw2/qemu.git/commit
hw/i386/amd_iommu: Don't leak memory in amdvi_update_iotlb()
authorPeter Maydell <peter.maydell@linaro.org>
Wed, 31 Jul 2024 17:00:19 +0000 (18:00 +0100)
committerMichael S. Tsirkin <mst@redhat.com>
Thu, 1 Aug 2024 08:32:00 +0000 (04:32 -0400)
commit9a45b0761628cc59267b3283a85d15294464ac31
tree7e7064ecf84fb73a51406422e77b1695943d3843
parent93829009a685687e2fb48158cb5df18c4eb49d07
hw/i386/amd_iommu: Don't leak memory in amdvi_update_iotlb()

In amdvi_update_iotlb() we will only put a new entry in the hash
table if to_cache.perm is not IOMMU_NONE.  However we allocate the
memory for the new AMDVIIOTLBEntry and for the hash table key
regardless.  This means that in the IOMMU_NONE case we will leak the
memory we alloacted.

Move the allocations into the if() to the point where we know we're
going to add the item to the hash table.

Cc: qemu-stable@nongnu.org
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/2452
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Message-Id: <20240731170019.3590563-1-peter.maydell@linaro.org>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
hw/i386/amd_iommu.c