aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/pci/intel-iommu.c
diff options
context:
space:
mode:
authorJulia Lawall <julia@diku.dk>2008-05-29 09:04:38 -0400
committerJesse Barnes <jbarnes@virtuousgeek.org>2008-06-10 13:59:54 -0400
commitece6763419f44ed72f4fc78752e5f5364df1794b (patch)
treea0c53e4a499263ed26e8a2c273f3f906aec740ae /drivers/pci/intel-iommu.c
parent6a3f084971bad985722afe25b16a5c0a990cea75 (diff)
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 66c0fd21894b..4f05d91a0fd8 100644
--- a/drivers/pci/intel-iommu.c
+++ b/drivers/pci/intel-iommu.c
@@ -1725,7 +1725,6 @@ int __init init_dmars(void)
1725 deferred_flush = kzalloc(g_num_of_iommus * 1725 deferred_flush = kzalloc(g_num_of_iommus *
1726 sizeof(struct deferred_flush_tables), GFP_KERNEL); 1726 sizeof(struct deferred_flush_tables), GFP_KERNEL);
1727 if (!deferred_flush) { 1727 if (!deferred_flush) {
1728 kfree(g_iommus);
1729 ret = -ENOMEM; 1728 ret = -ENOMEM;
1730 goto error; 1729 goto error;
1731 } 1730 }