diff --git a/Kernel/Process.h b/Kernel/Process.h index 68dc13539d..31aba2b744 100644 --- a/Kernel/Process.h +++ b/Kernel/Process.h @@ -253,11 +253,11 @@ public: KResultOr sys$waitid(Userspace); KResultOr sys$mmap(Userspace); KResultOr sys$mremap(Userspace); - KResultOr sys$munmap(void*, size_t size); + KResultOr sys$munmap(Userspace, size_t); KResultOr sys$set_mmap_name(Userspace); - KResultOr sys$mprotect(void*, size_t, int prot); - KResultOr sys$madvise(void*, size_t, int advice); - KResultOr sys$msyscall(void*); + KResultOr sys$mprotect(Userspace, size_t, int prot); + KResultOr sys$madvise(Userspace, size_t, int advice); + KResultOr sys$msyscall(Userspace); KResultOr sys$purge(int mode); KResultOr sys$select(Userspace); KResultOr sys$poll(Userspace); diff --git a/Kernel/Syscalls/mmap.cpp b/Kernel/Syscalls/mmap.cpp index 8802bd3f72..7f25a6fa08 100644 --- a/Kernel/Syscalls/mmap.cpp +++ b/Kernel/Syscalls/mmap.cpp @@ -145,7 +145,7 @@ KResultOr Process::sys$mmap(Userspace u if (!copy_from_user(¶ms, user_params)) return EFAULT; - void* addr = (void*)params.addr; + FlatPtr addr = params.addr; size_t size = params.size; size_t alignment = params.alignment; int prot = params.prot; @@ -288,7 +288,7 @@ static KResultOr expand_range_to_page_boundaries(FlatPtr address, size_t return Range { base, end - base.get() }; } -KResultOr Process::sys$mprotect(void* addr, size_t size, int prot) +KResultOr Process::sys$mprotect(Userspace addr, size_t size, int prot) { REQUIRE_PROMISE(stdio); @@ -296,7 +296,7 @@ KResultOr Process::sys$mprotect(void* addr, size_t size, int prot) REQUIRE_PROMISE(prot_exec); } - auto range_or_error = expand_range_to_page_boundaries((FlatPtr)addr, size); + auto range_or_error = expand_range_to_page_boundaries(addr, size); if (range_or_error.is_error()) return range_or_error.error(); @@ -366,11 +366,11 @@ KResultOr Process::sys$mprotect(void* addr, size_t size, int prot) return EINVAL; } -KResultOr Process::sys$madvise(void* address, size_t size, int advice) +KResultOr Process::sys$madvise(Userspace address, size_t size, int advice) { REQUIRE_PROMISE(stdio); - auto range_or_error = expand_range_to_page_boundaries((FlatPtr)address, size); + auto range_or_error = expand_range_to_page_boundaries(address, size); if (range_or_error.is_error()) return range_or_error.error(); @@ -445,14 +445,14 @@ KResultOr Process::sys$set_mmap_name(Userspace Process::sys$munmap(void* addr, size_t size) +KResultOr Process::sys$munmap(Userspace addr, size_t size) { REQUIRE_PROMISE(stdio); if (!size) return EINVAL; - auto range_or_error = expand_range_to_page_boundaries((FlatPtr)addr, size); + auto range_or_error = expand_range_to_page_boundaries(addr, size); if (range_or_error.is_error()) return range_or_error.error(); @@ -582,7 +582,7 @@ KResultOr Process::sys$allocate_tls(size_t size) return m_master_tls_region.unsafe_ptr()->vaddr().get(); } -KResultOr Process::sys$msyscall(void* address) +KResultOr Process::sys$msyscall(Userspace address) { if (space().enforces_syscall_regions()) return EPERM;