diff options
| author | Bora Guvendik <bora.guvendik@intel.com> | 2019-12-23 16:44:06 -0800 |
|---|---|---|
| committer | Aaron Durbin <adurbin@chromium.org> | 2020-01-02 18:22:53 +0000 |
| commit | a347ea3787284483158aaa77846ab0b34c017fdb (patch) | |
| tree | 82a8ac385958115c30871b43eb5be232bf8c8aed /src/include | |
| parent | 320d6e88afa56f2c11d9118faf6e47036fcc6540 (diff) | |
| download | coreboot-a347ea3787284483158aaa77846ab0b34c017fdb.tar.xz | |
lib/malloc: Implement a simple free() only for last malloc()
Implement a free() that supports only the last malloc(). Rewind
the heap to the last allocation point if the ptr to be freed is
matching the end of heap before last malloc(). With current situation,
since free() is no-op, every call to malloc() is a memory leak.
BUG=b:140124451
TEST=Wrote a test function to do malloc and free operations.
Change-Id: I6d43cf54b79e6897cf6882335730b2310e4eae45
Signed-off-by: Bora Guvendik <bora.guvendik@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/37919
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'src/include')
| -rw-r--r-- | src/include/stdlib.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/include/stdlib.h b/src/include/stdlib.h index a8297f82a5..f6369bf0f4 100644 --- a/src/include/stdlib.h +++ b/src/include/stdlib.h @@ -5,7 +5,6 @@ void *memalign(size_t boundary, size_t size); void *malloc(size_t size); -/* We never free memory */ -static inline void free(void *ptr) {} +void free(void *ptr); #endif /* STDLIB_H */ |