diff options
| author | Julia Lawall <julia@diku.dk> | 2008-05-29 15:04:38 +0200 |
|---|---|---|
| committer | Jesse Barnes <jbarnes@virtuousgeek.org> | 2008-06-10 10:59:54 -0700 |
| commit | ece6763419f44ed72f4fc78752e5f5364df1794b (patch) | |
| tree | a0c53e4a499263ed26e8a2c273f3f906aec740ae /drivers | |
| parent | 6a3f084971bad985722afe25b16a5c0a990cea75 (diff) | |
| download | op-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')
| -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; } |
