summaryrefslogtreecommitdiffstats
path: root/drivers/pci/intel-iommu.c
diff options
context:
space:
mode:
authorJulia Lawall <julia@diku.dk>2008-05-29 15:04:38 +0200
committerJesse Barnes <jbarnes@virtuousgeek.org>2008-06-10 10:59:54 -0700
commitece6763419f44ed72f4fc78752e5f5364df1794b (patch)
treea0c53e4a499263ed26e8a2c273f3f906aec740ae /drivers/pci/intel-iommu.c
parent6a3f084971bad985722afe25b16a5c0a990cea75 (diff)
downloadop-kernel-dev-ece6763419f44ed72f4fc78752e5f5364df1794b.zip
op-kernel-dev-ece6763419f44ed72f4fc78752e5f5364df1794b.tar.gz
PCI: eliminate double kfree in intel-iommu initialization
The destination of goto error also does a kfree(g_iommus), so it is not correct to do one here. This was found using Coccinelle (http://www.emn.fr/x-info/coccinelle/). Signed-off-by: Julia Lawall <julia@diku.dk> Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Diffstat (limited to 'drivers/pci/intel-iommu.c')
-rw-r--r--drivers/pci/intel-iommu.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/pci/intel-iommu.c b/drivers/pci/intel-iommu.c
index 66c0fd2..4f05d91 100644
--- a/drivers/pci/intel-iommu.c
+++ b/drivers/pci/intel-iommu.c
@@ -1725,7 +1725,6 @@ int __init init_dmars(void)
deferred_flush = kzalloc(g_num_of_iommus *
sizeof(struct deferred_flush_tables), GFP_KERNEL);
if (!deferred_flush) {
- kfree(g_iommus);
ret = -ENOMEM;
goto error;
}
OpenPOWER on IntegriCloud