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.

Sign up for GitHub

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

Jump to bottom

Memory leak if libzt events aren't consumed by user app #244

Closed
joseph-henry opened this issue Aug 4, 2023 · 0 comments
Closed

Memory leak if libzt events aren't consumed by user app #244

joseph-henry opened this issue Aug 4, 2023 · 0 comments
Assignees
@bostick @joseph-henry
Labels
Bug

Comments

@joseph-henry
Copy link
Contributor

I noticed a memory leak in the C examples if zts_init_set_event_handler wasn't set and events weren't consumed. This is a note to self but @bostick feel free to investigate this if you have a chance.
bostick added a commit that referenced this issue Aug 21, 2023
@bostick
Fix #244:
ee5047b 
Memory leak if libzt events aren't consumed by user app
If arg is not enqueued by Events, then treat as if ownership has NOT
been transferred and caller of Events->enqueue is responsible for freeing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bostick bostick

@joseph-henry joseph-henry

Labels
Bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bostick @joseph-henry