Commit 41464a46 authored by Yi Liu's avatar Yi Liu Committed by Jason Gunthorpe
Browse files

iommufd: Initialize the flags of vevent in iommufd_viommu_report_event() 

The vevent->header.flags is not initialized per allocation, hence the
vevent read path may treat the vevent as lost_events_header wrongly.
Use kzalloc() to alloc memory for new vevent.

Fixes: e8e1ef9b ("iommufd/viommu: Add iommufd_viommu_report_event helper")
Link: https://patch.msgid.link/r/20250324120034.5940-2-yi.l.liu@intel.com


Signed-off-by: default avatarYi Liu <yi.l.liu@intel.com>
Reviewed-by: default avatarNicolin Chen <nicolinc@nvidia.com>
Tested-by: default avatarNicolin Chen <nicolinc@nvidia.com>
Signed-off-by: default avatarJason Gunthorpe <jgg@nvidia.com>
parent 6d9500bb

drivers/iommu/iommufd/driver.c

+1 −1
Original line number Diff line number Diff line
@@ -102,7 +102,7 @@ int iommufd_viommu_report_event(struct iommufd_viommu *viommu, 
		goto out_set_header;

	}



	vevent = kmalloc(struct_size(vevent, event_data, data_len), GFP_ATOMIC);

	vevent = kzalloc(struct_size(vevent, event_data, data_len), GFP_ATOMIC);

	if (!vevent) {

		rc = -ENOMEM;

		vevent = &veventq->lost_events_header;