Changeset 560b81c in mainline for kernel/generic/src/mm/as.c

Timestamp:

2016-09-17T15:09:40Z (8 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

ae66564

Parents:

97b8ca9

Message:

Make sure to test the present bit of the found PTE

By design, page_mapping_find() can return true and a copy of a PTE
which is not present. It is therefore necessary to test the found PTE
by PTE_PRESENT() macro.

File:

• kernel/generic/src/mm/as.c (modified) (1 diff)

kernel/generic/src/mm/as.c

@@ -1457 +1457 @@
         pte_t pte;
         bool found = page_mapping_find(AS, page, false, &pte);
-        if (found) {
-                if (PTE_PRESENT(&pte)) {
-                        if (((access == PF_ACCESS_READ) && PTE_READABLE(&pte)) ||
-                            (access == PF_ACCESS_WRITE && PTE_WRITABLE(&pte)) ||
-                            (access == PF_ACCESS_EXEC && PTE_EXECUTABLE(&pte))) {
-                                page_table_unlock(AS, false);
-                                mutex_unlock(&area->lock);
-                                mutex_unlock(&AS->lock);
-                                return AS_PF_OK;
-                        }
+        if (found && PTE_PRESENT(&pte)) {
+                if (((access == PF_ACCESS_READ) && PTE_READABLE(&pte)) ||
+                    (access == PF_ACCESS_WRITE && PTE_WRITABLE(&pte)) ||
+                    (access == PF_ACCESS_EXEC && PTE_EXECUTABLE(&pte))) {
+                        page_table_unlock(AS, false);
+                        mutex_unlock(&area->lock);
+                        mutex_unlock(&AS->lock);
+                        return AS_PF_OK;
                 }
         }