summaryrefslogtreecommitdiff
path: root/Kernel/Syscalls/rmdir.cpp
diff options
context:
space:
mode:
authorBrian Gianforcaro <bgianf@serenityos.org>2021-12-29 00:10:17 -0800
committerAndreas Kling <kling@serenityos.org>2021-12-29 18:08:15 +0100
commitbad6d50b86ae1e0a46219baf149fa0a3574af9ce (patch)
tree83d0da5a800509416d70555b751cd01244cbe98c /Kernel/Syscalls/rmdir.cpp
parentc4f60844c5d2bd51e70a621753208fe0c9392339 (diff)
downloadserenity-bad6d50b86ae1e0a46219baf149fa0a3574af9ce.zip
Kernel: Use Process::require_promise() instead of REQUIRE_PROMISE()
This change lays the foundation for making the require_promise return an error hand handling the process abort outside of the syscall implementations, to avoid cases where we would leak resources. It also has the advantage that it makes removes a gs pointer read to look up the current thread, then process for every syscall. We can instead go through the Process this pointer in most cases.
Diffstat (limited to 'Kernel/Syscalls/rmdir.cpp')
-rw-r--r--Kernel/Syscalls/rmdir.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/Kernel/Syscalls/rmdir.cpp b/Kernel/Syscalls/rmdir.cpp
index dfa68b38b4..e570f562c5 100644
--- a/Kernel/Syscalls/rmdir.cpp
+++ b/Kernel/Syscalls/rmdir.cpp
@@ -13,7 +13,7 @@ namespace Kernel {
ErrorOr<FlatPtr> Process::sys$rmdir(Userspace<const char*> user_path, size_t path_length)
{
VERIFY_PROCESS_BIG_LOCK_ACQUIRED(this)
- REQUIRE_PROMISE(cpath);
+ require_promise(Pledge::cpath);
auto path = TRY(get_syscall_path_argument(user_path, path_length));
TRY(VirtualFileSystem::the().rmdir(path->view(), current_directory()));
return 0;