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

applications: Resource leak #1884

Closed
maddin200 opened this issue Mar 3, 2021 · 1 comment · Fixed by #1914
Closed

applications: Resource leak #1884

maddin200 opened this issue Mar 3, 2021 · 1 comment · Fixed by #1914
Labels
issue: bug (minor) Bug affecting only some users or with no major impact on the framework

Comments

@maddin200
Copy link
Contributor

applications/plugins/SofaDistanceGrid/extlibs/miniFlowVR/src/librender/mesh_io_mesh.cpp line 130
Code:
if (fread(&magic, sizeof(int), 1, fp)<=0) return false; // <-- missing fclose
@fredroy
Copy link
Contributor

fredroy commented Mar 12, 2021

I took the liberty to fix this issue in the same PR as the previous issue #1841

@fredroy fredroy mentioned this issue Mar 12, 2021
Merged
@fredroy fredroy added the issue: bug (minor) Bug affecting only some users or with no major impact on the framework label Mar 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
issue: bug (minor) Bug affecting only some users or with no major impact on the framework
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[SofaGuiQt] Fix compilation when SOFA_DUMP_VISITOR is enabled fredroy/sofa
2 participants
@maddin200 @fredroy