beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-25 17:10:23 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 7

LibC: Don't leak memory for realloc(p, 0)

Browse source 
Previously we'd leak memory when the user called realloc(p, 0). Instead
this call should behave as if the user had called free(p).
This commit is contained in:
Gunnar Beutner 2021-05-29 14:46:10 +02:00 committed by Andreas Kling
parent 3ece67d62d
commit ea33e9647b
Notes: sideshowbarker 2024-07-18 17:12:13 +09:00 
Author: https://github.com/gunnarbeutner
Commit: https://github.com/SerenityOS/serenity/commit/ea33e9647be
Pull-request: https://github.com/SerenityOS/serenity/pull/7555
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
Userland/Libraries/LibC/malloc.cpp
View file

@ -437,8 +437,10 @@ void* realloc(void* ptr, size_t size)
{ {
if (!ptr) if (!ptr)
return malloc(size); return malloc(size);
if (!size) if (!size) {
free(ptr);
return nullptr; return nullptr;
}
Threading::Locker locker(malloc_lock()); Threading::Locker locker(malloc_lock());
auto existing_allocation_size = malloc_size(ptr); auto existing_allocation_size = malloc_size(ptr);