diff options
| author | Guoqing Jiang <guoqing.jiang@linux.dev> | 2022-01-28 11:10:02 +0800 |
|---|---|---|
| committer | Joerg Roedel <jroedel@suse.de> | 2022-01-31 16:53:09 +0100 |
| commit | 99e675d473eb8cf2deac1376a0f840222fc1adcf (patch) | |
| tree | 3accecfccfa863ca9b4e029b10e7d43d8b9ddb7e /tools/perf/scripts/python/export-to-postgresql.py | |
| parent | 30209b93177a75843673de92771716c941c20ef5 (diff) | |
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>
Diffstat (limited to 'tools/perf/scripts/python/export-to-postgresql.py')
0 files changed, 0 insertions, 0 deletions