iommu/vt-d: Fix potential memory leak in intel_setup_irq_remapping()
authorGuoqing Jiang <guoqing.jiang@linux.dev>
Fri, 28 Jan 2022 03:10:02 +0000 (11:10 +0800)
committerJoerg Roedel <jroedel@suse.de>
Mon, 31 Jan 2022 15:53:09 +0000 (16:53 +0100)
commit99e675d473eb8cf2deac1376a0f840222fc1adcf
tree3accecfccfa863ca9b4e029b10e7d43d8b9ddb7e
parent30209b93177a75843673de92771716c941c20ef5
iommu/vt-d: Fix potential memory leak in intel_setup_irq_remapping()

After commit e3beca48a45b ("irqdomain/treewide: Keep firmware node
unconditionally allocated"). For tear down scenario, fn is only freed
after fail to allocate ir_domain, though it also should be freed in case
dmar_enable_qi returns error.

Besides free fn, irq_domain and ir_msi_domain need to be removed as well
if intel_setup_irq_remapping fails to enable queued invalidation.

Improve the rewinding path by add out_free_ir_domain and out_free_fwnode
lables per Baolu's suggestion.

Fixes: e3beca48a45b ("irqdomain/treewide: Keep firmware node unconditionally allocated")
Suggested-by: Lu Baolu <baolu.lu@linux.intel.com>
Signed-off-by: Guoqing Jiang <guoqing.jiang@linux.dev>
Link: https://lore.kernel.org/r/20220119063640.16864-1-guoqing.jiang@linux.dev
Signed-off-by: Lu Baolu <baolu.lu@linux.intel.com>
Link: https://lore.kernel.org/r/20220128031002.2219155-3-baolu.lu@linux.intel.com
Signed-off-by: Joerg Roedel <jroedel@suse.de>
drivers/iommu/intel/irq_remapping.c