-
-
Notifications
You must be signed in to change notification settings - Fork 179
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
Fix FileStream memory leak #2160
Conversation
Automated fixes for code style.
@alberk8 there are issues with the code style on the source files. Make sure to follow the project code style. Check the details here on how it works and the tools required to help you with that. |
…80a-97f6-4576-b3b9-057d7d053917 Code style fixes for nanoframework/nf-interpreter PR#2160
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice job chasing this down!
I've added a missing check for succesfulll malloc and comments stressing the need to free memory by the caller.
It's not perfect, but will get the job done.
Automated fixes for code style.
@alberk8 there are issues with the code style on the source files. Make sure to follow the project code style. Check the details here on how it works and the tools required to help you with that. |
…a90-7a74-4eed-80f1-3a642abf1fa1 Code style fixes for nanoframework/nf-interpreter PR#2160
Head branch was pushed to by a user without write access
Description
Motivation and Context
How Has This Been Tested?
loop 100 times.
https://github.com/alberk8/nanoFramework-SDCard-Test/tree/main/nanoFrameworkSDCard
Screenshots
Types of changes
Checklist: