diff options
author | Lu Baolu <baolu.lu@linux.intel.com> | 2018-02-24 00:42:27 -0500 |
---|---|---|
committer | Joerg Roedel <jroedel@suse.de> | 2018-03-15 10:15:22 -0400 |
commit | bbe4b3af9d9e3172fb9aa1f8dcdfaedcb381fc64 (patch) | |
tree | 64909a7dbabe5a93bc19678017ac27913b9d1652 | |
parent | b1d03c1d12abbfa7de127772f281b309cf1650c3 (diff) |
iommu/vt-d: Fix a potential memory leak
A memory block was allocated in intel_svm_bind_mm() but never freed
in a failure path. This patch fixes this by free it to avoid memory
leakage.
Cc: Ashok Raj <ashok.raj@intel.com>
Cc: Jacob Pan <jacob.jun.pan@linux.intel.com>
Cc: <stable@vger.kernel.org> # v4.4+
Signed-off-by: Lu Baolu <baolu.lu@linux.intel.com>
Fixes: 2f26e0a9c9860 ('iommu/vt-d: Add basic SVM PASID support')
Signed-off-by: Joerg Roedel <jroedel@suse.de>
-rw-r--r-- | drivers/iommu/intel-svm.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/iommu/intel-svm.c b/drivers/iommu/intel-svm.c index 35a408d0ae4f..3d4b9245542d 100644 --- a/drivers/iommu/intel-svm.c +++ b/drivers/iommu/intel-svm.c | |||
@@ -396,6 +396,7 @@ int intel_svm_bind_mm(struct device *dev, int *pasid, int flags, struct svm_dev_ | |||
396 | pasid_max - 1, GFP_KERNEL); | 396 | pasid_max - 1, GFP_KERNEL); |
397 | if (ret < 0) { | 397 | if (ret < 0) { |
398 | kfree(svm); | 398 | kfree(svm); |
399 | kfree(sdev); | ||
399 | goto out; | 400 | goto out; |
400 | } | 401 | } |
401 | svm->pasid = ret; | 402 | svm->pasid = ret; |