diff options
| author | Gunnar Beutner <gbeutner@serenityos.org> | 2021-06-28 20:59:35 +0200 |
|---|---|---|
| committer | Andreas Kling <kling@serenityos.org> | 2021-06-28 22:29:28 +0200 |
| commit | 2a78bf85968ebff58b4d23be43fecdbe6178adb2 (patch) | |
| tree | 07daafb91a2e20fe77843c7aeb5726469cd29081 /Kernel/Syscalls/mount.cpp | |
| parent | b6435372cc91a9ce2c3f70a6d12c7f7869dfb2f0 (diff) | |
| download | serenity-2a78bf85968ebff58b4d23be43fecdbe6178adb2.zip | |
Kernel: Fix the return type for syscalls
The Process::Handler type has KResultOr<FlatPtr> as its return type.
Using a different return type with an equally-sized template parameter
sort of works but breaks once that condition is no longer true, e.g.
for KResultOr<int> on x86_64.
Ideally the syscall handlers would also take FlatPtrs as their args
so we can get rid of the reinterpret_cast for the function pointer
but I didn't quite feel like cleaning that up as well.
Diffstat (limited to 'Kernel/Syscalls/mount.cpp')
| -rw-r--r-- | Kernel/Syscalls/mount.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Kernel/Syscalls/mount.cpp b/Kernel/Syscalls/mount.cpp index 237bb48a01..c6b9f7c5b8 100644 --- a/Kernel/Syscalls/mount.cpp +++ b/Kernel/Syscalls/mount.cpp @@ -16,7 +16,7 @@ namespace Kernel { -KResultOr<int> Process::sys$mount(Userspace<const Syscall::SC_mount_params*> user_params) +KResultOr<FlatPtr> Process::sys$mount(Userspace<const Syscall::SC_mount_params*> user_params) { if (!is_superuser()) return EPERM; @@ -111,7 +111,7 @@ KResultOr<int> Process::sys$mount(Userspace<const Syscall::SC_mount_params*> use return result; } -KResultOr<int> Process::sys$umount(Userspace<const char*> user_mountpoint, size_t mountpoint_length) +KResultOr<FlatPtr> Process::sys$umount(Userspace<const char*> user_mountpoint, size_t mountpoint_length) { if (!is_superuser()) return EPERM; |