Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jlindgren90
Copy link

  • 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.

- Optionally set errno to ENOMEM when out of memory
- Optionally use memset() to implement memclear()
- Optionally use memcpy() to implement memcopy()
@jlindgren90
Copy link
Author

Updated with some more fixes:

@postspectacular
Copy link
Member

@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...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants