summaryrefslogtreecommitdiff
path: root/Kernel/Memory/AddressSpace.cpp
diff options
context:
space:
mode:
authorAndreas Kling <kling@serenityos.org>2021-11-10 11:55:37 +0100
committerAndreas Kling <kling@serenityos.org>2021-11-10 21:58:58 +0100
commit88b6428c25ea046a4bb19bb6f3f68dd4f1439539 (patch)
tree86eca67f4ffc83d5387590b0d502ecce7cc07b91 /Kernel/Memory/AddressSpace.cpp
parentcd49f30bea734feb9ac46d637e2ed3439e47e3c3 (diff)
downloadserenity-88b6428c25ea046a4bb19bb6f3f68dd4f1439539.zip
AK: Make Vector::try_* functions return ErrorOr<void>
Instead of signalling allocation failure with a bool return value (false), we now use ErrorOr<void> and return ENOMEM as appropriate. This allows us to use TRY() and MUST() with Vector. :^)
Diffstat (limited to 'Kernel/Memory/AddressSpace.cpp')
-rw-r--r--Kernel/Memory/AddressSpace.cpp3
1 files changed, 1 insertions, 2 deletions
diff --git a/Kernel/Memory/AddressSpace.cpp b/Kernel/Memory/AddressSpace.cpp
index f0ce03dc24..6f3e54d313 100644
--- a/Kernel/Memory/AddressSpace.cpp
+++ b/Kernel/Memory/AddressSpace.cpp
@@ -110,8 +110,7 @@ ErrorOr<void> AddressSpace::unmap_mmap_range(VirtualAddress addr, size_t size)
// Otherwise, split the regions and collect them for future mapping.
auto split_regions = TRY(try_split_region_around_range(*region, range_to_unmap));
- if (new_regions.try_extend(split_regions))
- return ENOMEM;
+ TRY(new_regions.try_extend(split_regions));
}
// Give back any unwanted VM to the range allocator.