diff options
author | Julia Lawall <julia@diku.dk> | 2008-05-29 09:04:38 -0400 |
---|---|---|
committer | Jesse Barnes <jbarnes@virtuousgeek.org> | 2008-06-10 13:59:54 -0400 |
commit | ece6763419f44ed72f4fc78752e5f5364df1794b (patch) | |
tree | a0c53e4a499263ed26e8a2c273f3f906aec740ae /drivers/pci | |
parent | 6a3f084971bad985722afe25b16a5c0a990cea75 (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')
-rw-r--r-- | drivers/pci/intel-iommu.c | 1 |
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 | } |