From 97d0ebba20edb6afeb015801bec4f7305edf39a0 Mon Sep 17 00:00:00 2001 From: Gunnar Beutner Date: Thu, 27 May 2021 21:07:56 +0200 Subject: LibJS: Make sure aligned_alloc() doesn't return a null pointer The previous VERIFY() call checked that aligned_alloc() didn't return MAP_FAILED. When out of memory aligned_alloc() returns a null pointer so let's check for that instead. --- Userland/Libraries/LibJS/Heap/BlockAllocator.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Userland/Libraries/LibJS/Heap/BlockAllocator.cpp b/Userland/Libraries/LibJS/Heap/BlockAllocator.cpp index 1e50414b88..d622623385 100644 --- a/Userland/Libraries/LibJS/Heap/BlockAllocator.cpp +++ b/Userland/Libraries/LibJS/Heap/BlockAllocator.cpp @@ -38,10 +38,11 @@ void* BlockAllocator::allocate_block([[maybe_unused]] char const* name) #ifdef __serenity__ auto* block = (HeapBlock*)serenity_mmap(nullptr, HeapBlock::block_size, PROT_READ | PROT_WRITE, MAP_ANONYMOUS | MAP_RANDOMIZED | MAP_PRIVATE, 0, 0, HeapBlock::block_size, name); + VERIFY(block != MAP_FAILED); #else auto* block = (HeapBlock*)aligned_alloc(HeapBlock::block_size, HeapBlock::block_size); + VERIFY(block); #endif - VERIFY(block != MAP_FAILED); return block; } -- cgit v1.2.3