Correctness fixes and (optional) better C stdlib integration #13

jlindgren90 commented 2024年03月23日 20:41:43 +01:00 (Migrated from github.com)

• Add early NULL check to ta_free()
• Add overflow checks in alloc_block() and ta_calloc()
• Clear full block size to 0's in ta_calloc()
• Optionally set errno to ENOMEM when out of memory
• Optionally use memset() to implement memclear()
• Add ta_getsize() to return allocated block size
• Add ta_realloc() (optional, requires memcpy())

ta_getsize() and ta_realloc() are based on ideas from #11, but rewritten to fix various issues in that PR.

- Add early NULL check to ta_free() - Add overflow checks in alloc_block() and ta_calloc() - Clear full block size to 0's in ta_calloc() - Optionally set errno to ENOMEM when out of memory - Optionally use memset() to implement memclear() - Add ta_getsize() to return allocated block size - Add ta_realloc() (optional, requires memcpy()) ta_getsize() and ta_realloc() are based on ideas from #11, but rewritten to fix various issues in that PR.

jlindgren90 commented 2024年03月25日 05:04:27 +01:00 (Migrated from github.com)

Updated with some more fixes:

• Fix behavior of ta_alloc(0) (see #15)
• Fix additional arithmetic overflows
• Add fallback memcopy() when not using C stdlib
• Add early return to ta_realloc() when size is similar

Updated with some more fixes: - Fix behavior of ta_alloc(0) (see #15) - Fix additional arithmetic overflows - Add fallback memcopy() when not using C stdlib - Add early return to ta_realloc() when size is similar

postspectacular commented 2024年03月27日 08:26:14 +01:00 (Migrated from github.com)

@jlindgren90 Thank you very much for this and the other PR - I will take a look asap, but likely will take a little while for me to review & test this...

@jlindgren90 Thank you very much for this and the other PR - I will take a look asap, but likely will take a little while for me to review & test this...

This pull request can be merged automatically.

You are not authorized to merge this pull request.