Skip to content

Commit d4650bc

Browse files
committed
Change pool-based allocation to heap-based allocation in jerry.c
There was an abuse of memory pools in jerry.c: `compiled_code_map_entry_t` is not an ECMA data model object but was still alocated/freed via mem pool functions. Changed the appropriate calls to heap-related functions. JerryScript-DCO-1.0-Signed-off-by: Akos Kiss [email protected]
1 parent a187e6d commit d4650bc

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

jerry-core/jerry.c

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1851,7 +1851,8 @@ snapshot_add_compiled_code (ecma_compiled_code_t *compiled_code_p, /**< compiled
18511851

18521852
snapshot_last_compiled_code_offset = snapshot_buffer_write_offset;
18531853

1854-
compiled_code_map_entry_t *new_entry = (compiled_code_map_entry_t *) mem_pools_alloc ();
1854+
compiled_code_map_entry_t *new_entry;
1855+
new_entry = (compiled_code_map_entry_t *) mem_heap_alloc_block (sizeof (compiled_code_map_entry_t));
18551856

18561857
if (new_entry == NULL)
18571858
{
@@ -2133,7 +2134,7 @@ jerry_parse_and_save_snapshot (const jerry_api_char_t *source_p, /**< script sou
21332134
{
21342135
compiled_code_map_entry_t *next_p = ECMA_GET_POINTER (compiled_code_map_entry_t,
21352136
current_p->next_cp);
2136-
mem_pools_free ((uint8_t *) current_p);
2137+
mem_heap_free_block (current_p, sizeof (compiled_code_map_entry_t));
21372138
current_p = next_p;
21382139
}
21392140

0 commit comments

Comments
 (0)