diff options
author | Julia Lawall <julia@diku.dk> | 2008-05-29 13:04:38 (GMT) |
---|---|---|
committer | Jesse Barnes <jbarnes@virtuousgeek.org> | 2008-06-10 17:59:54 (GMT) |
commit | ece6763419f44ed72f4fc78752e5f5364df1794b (patch) | |
tree | a0c53e4a499263ed26e8a2c273f3f906aec740ae /drivers/pci/intel-iommu.c | |
parent | 6a3f084971bad985722afe25b16a5c0a990cea75 (diff) | |
download | linux-fsl-qoriq-ece6763419f44ed72f4fc78752e5f5364df1794b.tar.xz |
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.c | 1 |
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; } |