tlwg / libthai

GNU Lesser General Public License v2.1
71 stars 18 forks source link

brk_pool_node_new: realloc() in brk_shot_reuse() can fail #24

Open marcmutz opened 1 year ago

marcmutz commented 1 year ago

Handle failure of realloc() in brk_shot_reuse(), which requires adding a return value to brk_shot_reuse() and handling it in brk_pool_new_node().

If brk_shot_reuse() fails, try allocating a new node. Yes, if realloc() failed, a following malloc() will likely fail, too, but this way the error at least propagates up the call chain.

To keep the logic clean, factor the old if (env->free_list) code into a separate function brk_pool_node_new_from_free_list().

Fixes #23.