Skip to content

Commit 41464a4

Browse files
yiliu1765jgunthorpe
authored andcommitted
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: e8e1ef9 ("iommufd/viommu: Add iommufd_viommu_report_event helper") Link: https://patch.msgid.link/r/[email protected] Signed-off-by: Yi Liu <[email protected]> Reviewed-by: Nicolin Chen <[email protected]> Tested-by: Nicolin Chen <[email protected]> Signed-off-by: Jason Gunthorpe <[email protected]>
1 parent 6d9500b commit 41464a4

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

drivers/iommu/iommufd/driver.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ int iommufd_viommu_report_event(struct iommufd_viommu *viommu,
102102
goto out_set_header;
103103
}
104104

105-
vevent = kmalloc(struct_size(vevent, event_data, data_len), GFP_ATOMIC);
105+
vevent = kzalloc(struct_size(vevent, event_data, data_len), GFP_ATOMIC);
106106
if (!vevent) {
107107
rc = -ENOMEM;
108108
vevent = &veventq->lost_events_header;

0 commit comments

Comments
 (0)