diff options
| author | Will Deacon <will.deacon@arm.com> | 2013-11-08 16:08:06 +0100 |
|---|---|---|
| committer | Will Deacon <will.deacon@arm.com> | 2013-12-06 17:44:25 +0100 |
| commit | 5552ecdbf9fb4f7608c1d7933a8baafcfa1bb337 (patch) | |
| tree | f3d3a2c1858cc366ca123a3a7aaf49c4ac8778ad /drivers/iommu/arm-smmu.c | |
| parent | iommu/arm-smmu: use mutex instead of spinlock for locking page tables (diff) | |
| download | linux-5552ecdbf9fb4f7608c1d7933a8baafcfa1bb337.tar.xz linux-5552ecdbf9fb4f7608c1d7933a8baafcfa1bb337.zip | |
iommu/arm-smmu: remove potential NULL dereference on mapping path
When handling mapping requests, we dereference the SMMU domain before
checking that it is NULL. This patch fixes the issue by removing the check
altogether, since we don't actually use the leaf_smmu when creating
mappings.
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Will Deacon <will.deacon@arm.com>
Diffstat (limited to '')
| -rw-r--r-- | drivers/iommu/arm-smmu.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/iommu/arm-smmu.c b/drivers/iommu/arm-smmu.c index 6dbcaa4433cd..ef77e3dd6dd2 100644 --- a/drivers/iommu/arm-smmu.c +++ b/drivers/iommu/arm-smmu.c @@ -1423,9 +1423,8 @@ static int arm_smmu_map(struct iommu_domain *domain, unsigned long iova, phys_addr_t paddr, size_t size, int flags) { struct arm_smmu_domain *smmu_domain = domain->priv; - struct arm_smmu_device *smmu = smmu_domain->leaf_smmu; - if (!smmu_domain || !smmu) + if (!smmu_domain) return -ENODEV; /* Check for silent address truncation up the SMMU chain. */ |