| Message ID | 20150730165401.33962.81348.stgit@dwillia2-desk3.amr.corp.intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Thu, Jul 30, 2015 at 12:54:01PM -0400, Dan Williams wrote: > iommu_load_old_irte() appears to leak the old_irte mapping after use. > > Cc: Joerg Roedel <jroedel@suse.de> > Signed-off-by: Dan Williams <dan.j.williams@intel.com> > --- > drivers/iommu/intel_irq_remapping.c | 2 ++ > 1 file changed, 2 insertions(+) Applied, thanks.
diff --git a/drivers/iommu/intel_irq_remapping.c b/drivers/iommu/intel_irq_remapping.c index f15692a410c7..27cdfa84ec5b 100644 --- a/drivers/iommu/intel_irq_remapping.c +++ b/drivers/iommu/intel_irq_remapping.c @@ -426,6 +426,8 @@ static int iommu_load_old_irte(struct intel_iommu *iommu) bitmap_set(iommu->ir_table->bitmap, i, 1); } + iounmap(old_ir_table); + return 0; }
iommu_load_old_irte() appears to leak the old_irte mapping after use. Cc: Joerg Roedel <jroedel@suse.de> Signed-off-by: Dan Williams <dan.j.williams@intel.com> --- drivers/iommu/intel_irq_remapping.c | 2 ++ 1 file changed, 2 insertions(+)