diff options
author | Dan Carpenter <error27@gmail.com> | 2009-07-19 07:47:45 -0400 |
---|---|---|
committer | David Woodhouse <David.Woodhouse@intel.com> | 2009-07-20 00:01:20 -0400 |
commit | 86f4d0123b1fddb47d35b9a893f8c0b94bf89abe (patch) | |
tree | f96b34958b8a1724fe91040324ad2b65d953e7d8 /drivers/pci/intel-iommu.c | |
parent | 0db9b7aebb6a1c2bba2d0636ae0b1f9ef729c827 (diff) |
intel-iommu: double kfree()
g_iommus is freed after we "goto error;".
Found by smatch (http://repo.or.cz/w/smatch.git).
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
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 86a83946a8f5..097d5da2fae1 100644 --- a/drivers/pci/intel-iommu.c +++ b/drivers/pci/intel-iommu.c | |||
@@ -2224,7 +2224,6 @@ int __init init_dmars(void) | |||
2224 | deferred_flush = kzalloc(g_num_of_iommus * | 2224 | deferred_flush = kzalloc(g_num_of_iommus * |
2225 | sizeof(struct deferred_flush_tables), GFP_KERNEL); | 2225 | sizeof(struct deferred_flush_tables), GFP_KERNEL); |
2226 | if (!deferred_flush) { | 2226 | if (!deferred_flush) { |
2227 | kfree(g_iommus); | ||
2228 | ret = -ENOMEM; | 2227 | ret = -ENOMEM; |
2229 | goto error; | 2228 | goto error; |
2230 | } | 2229 | } |