diff options
author | Chris Fallin <cfallin@google.com> | 2015-05-14 18:05:39 -0700 |
---|---|---|
committer | Chris Fallin <cfallin@google.com> | 2015-05-15 11:04:07 -0700 |
commit | 710111cc7c58f9b51be949fb9a30487372af5dd9 (patch) | |
tree | 5158915774564340155bb9f7015225f36f9ed9b7 /upb | |
parent | eace8e32954eb6152e8df06f5a18905c235f0156 (diff) |
Bugfix: seeded_alloc() should not realloc() user-provided memory.
Diffstat (limited to 'upb')
-rw-r--r-- | upb/env.c | 11 |
1 files changed, 10 insertions, 1 deletions
@@ -211,6 +211,8 @@ static size_t align_up(size_t size) { UPB_FORCEINLINE static void *seeded_alloc(void *ud, void *ptr, size_t oldsize, size_t size) { + UPB_UNUSED(ptr); + upb_seededalloc *a = ud; size = align_up(size); @@ -224,7 +226,14 @@ UPB_FORCEINLINE static void *seeded_alloc(void *ud, void *ptr, size_t oldsize, } else { // Slow path: fallback to other allocator. a->need_cleanup = true; - return a->alloc(a->alloc_ud, ptr, oldsize, size); + // Is `ptr` part of the user-provided initial block? Don't pass it to the + // default allocator if so; otherwise, it may try to realloc() the block. + char *chptr = ptr; + if (chptr >= a->mem_base && chptr < a->mem_limit) { + return a->alloc(a->alloc_ud, NULL, 0, size); + } else { + return a->alloc(a->alloc_ud, ptr, oldsize, size); + } } } |