Commit acb59a4b authored by Zilin Guan's avatar Zilin Guan Committed by Alex Williamson
Browse files

vfio/pds: replace bitmap_free with vfree 



host_seq_bmp is allocated with vzalloc but is currently freed with
bitmap_free, which uses kfree internally. This mismach prevents the
resource from being released properly and may result in memory leaks
or other issues.

Fix this by freeing host_seq_bmp with vfree to match the vzalloc
allocation.

Fixes: f232836a ("vfio/pds: Add support for dirty page tracking")
Signed-off-by: default avatarZilin Guan <zilin@seu.edu.cn>
Reviewed-by: default avatarBrett Creeley <brett.creeley@amd.com>
Link: https://lore.kernel.org/r/20250913153154.1028835-1-zilin@seu.edu.cn


Signed-off-by: default avatarAlex Williamson <alex.williamson@redhat.com>
parent 16df67f2

drivers/vfio/pci/pds/dirty.c

+1 −1
Original line number Diff line number Diff line
@@ -82,7 +82,7 @@ static int pds_vfio_dirty_alloc_bitmaps(struct pds_vfio_region *region, 


	host_ack_bmp = vzalloc(bytes);

	if (!host_ack_bmp) {

		bitmap_free(host_seq_bmp);

		vfree(host_seq_bmp);

		return -ENOMEM;

	}