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

fix memory leak has introduced a segmentation fault in stream while using coreml #904

Closed
dev-mush opened this issue May 10, 2023 · 2 comments
Closed

fix memory leak has introduced a segmentation fault in stream while using coreml #904

dev-mush opened this issue May 10, 2023 · 2 comments

Comments

@dev-mush
Copy link

The changes introduced in this pr #899 are causing segmentation fault crashes when executing ./stream.

I've inspected the code a little, and my safe bet is that the program is attempting to release possibily nil objects, I don't see any other big changes from the previous version that could justify the segfault... but am not touching c++/objc code since ages so might be wrong.
@multispecies-lab
Copy link

Same issue here...

@jhen0409 jhen0409 mentioned this issue May 12, 2023
Merged
@ggerganov
Copy link
Owner

Should be fixed now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@dev-mush @ggerganov @multispecies-lab