Kernel: Process::validate_pointer_access now maps the whole range

This fixes a bug where userspace provided address is not fully mapped
and the kernel tries to read/write it while using PageTable fast page.

In the future userspace input should be copied on syscall entry, so
userspace could not modify the input during syscall. Currently there
is change that userspace input passes kernel syscall validation and
after that userspace could modify the input before the value is
actually used.

kernel/include/kernel/Process.h

@@ -198,6 +198,7 @@ namespace Kernel
 		BAN::ErrorOr<int> block_until_exit(pid_t pid);
 
 		BAN::ErrorOr<void> validate_string_access(const char*);
+		BAN::ErrorOr<void> validate_pointer_access_check(const void*, size_t);
 		BAN::ErrorOr<void> validate_pointer_access(const void*, size_t);
 
 		uint64_t signal_pending_mask() const

kernel/kernel/Process.cpp

@@ -1857,7 +1857,7 @@ namespace Kernel
 		return validate_pointer_access(str, strlen(str) + 1);
 	}
 
-	BAN::ErrorOr<void> Process::validate_pointer_access(const void* ptr, size_t size)
+	BAN::ErrorOr<void> Process::validate_pointer_access_check(const void* ptr, size_t size)
 	{
 		ASSERT(&Process::current() == this);
 		auto& thread = Thread::current();
@@ -1894,4 +1894,27 @@ unauthorized_access:
 		return BAN::Error::from_errno(EINTR);
 	}
 
+	BAN::ErrorOr<void> Process::validate_pointer_access(const void* ptr, size_t size)
+	{
+		// TODO: This seems very slow as we loop over the range twice
+
+		TRY(validate_pointer_access_check(ptr, size));
+
+		const vaddr_t vaddr = reinterpret_cast<vaddr_t>(ptr);
+
+		// Make sure all of the pages are mapped here, so demand paging does not happen
+		// while processing syscall.
+		const vaddr_t page_start = vaddr & PAGE_ADDR_MASK;
+		const size_t page_count = range_page_count(vaddr, size);
+		for (size_t i = 0; i < page_count; i++)
+		{
+			const vaddr_t current = page_start + i * PAGE_SIZE;
+			if (page_table().get_page_flags(current) & PageTable::Flags::Present)
+				continue;
+			TRY(Process::allocate_page_for_demand_paging(current));
+		}
+
+		return {};
+	}
+
 }