diff options
author | Oleksandr Tyshchenko <oleksandr_tyshchenko@epam.com> | 2017-02-27 07:30:25 -0500 |
---|---|---|
committer | Will Deacon <will.deacon@arm.com> | 2017-03-10 13:23:34 -0500 |
commit | ed46e66cc1b3d684042f92dfa2ab15ee917b4cac (patch) | |
tree | a0307607cdfb450cabc1494fd2bc9945d967bdb3 | |
parent | c1ae3cfa0e89fa1a7ecc4c99031f5e9ae99d9201 (diff) |
iommu/io-pgtable-arm: Check for leaf entry before dereferencing it
Do a check for already installed leaf entry at the current level before
dereferencing it in order to avoid walking the page table down with
wrong pointer to the next level.
Signed-off-by: Oleksandr Tyshchenko <oleksandr_tyshchenko@epam.com>
CC: Will Deacon <will.deacon@arm.com>
CC: Robin Murphy <robin.murphy@arm.com>
Signed-off-by: Will Deacon <will.deacon@arm.com>
-rw-r--r-- | drivers/iommu/io-pgtable-arm.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/drivers/iommu/io-pgtable-arm.c b/drivers/iommu/io-pgtable-arm.c index feacc54bec68..f9bc6ebb8140 100644 --- a/drivers/iommu/io-pgtable-arm.c +++ b/drivers/iommu/io-pgtable-arm.c | |||
@@ -335,8 +335,12 @@ static int __arm_lpae_map(struct arm_lpae_io_pgtable *data, unsigned long iova, | |||
335 | if (cfg->quirks & IO_PGTABLE_QUIRK_ARM_NS) | 335 | if (cfg->quirks & IO_PGTABLE_QUIRK_ARM_NS) |
336 | pte |= ARM_LPAE_PTE_NSTABLE; | 336 | pte |= ARM_LPAE_PTE_NSTABLE; |
337 | __arm_lpae_set_pte(ptep, pte, cfg); | 337 | __arm_lpae_set_pte(ptep, pte, cfg); |
338 | } else { | 338 | } else if (!iopte_leaf(pte, lvl)) { |
339 | cptep = iopte_deref(pte, data); | 339 | cptep = iopte_deref(pte, data); |
340 | } else { | ||
341 | /* We require an unmap first */ | ||
342 | WARN_ON(!selftest_running); | ||
343 | return -EEXIST; | ||
340 | } | 344 | } |
341 | 345 | ||
342 | /* Rinse, repeat */ | 346 | /* Rinse, repeat */ |