hyung-hwan/hcl
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fixed a bug in hcl_allocheap()

Browse Source
This commit is contained in:
hyung-hwan 2021-02-11 09:35:38 +00:00
parent 43322c6901
commit 6789e4b3a4
1 changed files with 29 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
lib/heap.c
View File

@ -46,8 +46,22 @@ hcl_heap_t* hcl_makeheap (hcl_t* hcl, hcl_oow_t size)
hcl_heap_t* heap; hcl_heap_t* heap;
hcl_oow_t alloc_size; hcl_oow_t alloc_size;
alloc_size = HCL_SIZEOF(*heap) + size; if (size <= HCL_SIZEOF(*heap)) /* 0 or smaller than the heap header */
{
/* make a zero-sized heap using the default memory manager.
* this zero-sized heap contains only the heap header */
size = 0;
alloc_size = HCL_SIZEOF(*heap);
heap = (hcl_heap_t*)hcl_allocmem(hcl, alloc_size);
}
else
{
/* if a non-zero heap size is given, create the heap with
* the dedicated heap allocator which is allowed to create
* a bigger heap than requested */
alloc_size = size;
heap = (hcl_heap_t*)hcl->vmprim.alloc_heap(hcl, &alloc_size); heap = (hcl_heap_t*)hcl->vmprim.alloc_heap(hcl, &alloc_size);
}
if (HCL_UNLIKELY(!heap)) if (HCL_UNLIKELY(!heap))
{ {
const hcl_ooch_t* oldmsg = hcl_backuperrmsg(hcl); const hcl_ooch_t* oldmsg = hcl_backuperrmsg(hcl);
@ -57,15 +71,17 @@ hcl_heap_t* hcl_makeheap (hcl_t* hcl, hcl_oow_t size)
/* the vmprim.alloc_heap() function is allowed to create a bigger heap than the requested size. /* the vmprim.alloc_heap() function is allowed to create a bigger heap than the requested size.
* if the created heap is bigger than requested, the heap will be utilized in full. */ * if the created heap is bigger than requested, the heap will be utilized in full. */
HCL_ASSERT (hcl, alloc_size >= HCL_SIZEOF(*heap) + size); HCL_ASSERT (hcl, alloc_size >= HCL_SIZEOF(*heap));
HCL_MEMSET (heap, 0, alloc_size); HCL_MEMSET (heap, 0, alloc_size);
alloc_size -= HCL_SIZEOF(*heap); /* exclude the header size */
heap->base = (hcl_uint8_t*)(heap + 1); heap->base = (hcl_uint8_t*)(heap + 1);
heap->size = alloc_size; heap->size = alloc_size;
if (size <= 0) if (size == 0)
{ {
/* use the existing memory allocator */ /* use the existing memory allocator */
HCL_ASSERT (hcl, alloc_size == 0);
heap->xmmgr = *hcl_getmmgr(hcl); heap->xmmgr = *hcl_getmmgr(hcl);
} }
else else
@ -89,10 +105,17 @@ hcl_heap_t* hcl_makeheap (hcl_t* hcl, hcl_oow_t size)
} }
void hcl_killheap (hcl_t* hcl, hcl_heap_t* heap) void hcl_killheap (hcl_t* hcl, hcl_heap_t* heap)
{
if (heap->size == 0)
{
hcl_freemem (hcl, heap);
}
else
{ {
if (heap->xma) hcl_xma_close (heap->xma); if (heap->xma) hcl_xma_close (heap->xma);
hcl->vmprim.free_heap (hcl, heap); hcl->vmprim.free_heap (hcl, heap);
} }
}
void* hcl_callocheapmem (hcl_t* hcl, hcl_heap_t* heap, hcl_oow_t size) void* hcl_callocheapmem (hcl_t* hcl, hcl_heap_t* heap, hcl_oow_t size)
{ {