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

found a resource leak in file wxWidgets-2.8/src/expat/xmlwf/readfilemap.c #11432

Closed
wxtrac opened this issue Nov 10, 2009 · 3 comments
Closed

found a resource leak in file wxWidgets-2.8/src/expat/xmlwf/readfilemap.c #11432

wxtrac opened this issue Nov 10, 2009 · 3 comments
Labels

Comments

@wxtrac
Copy link
Collaborator

@wxtrac wxtrac commented Nov 10, 2009

Issue migrated from trac ticket # 11432

component: GUI-all | priority: normal | resolution: wontfix

2009-11-10 23:07:39: ettl.martin (Ettl Martin) created the issue


during a check with the static code analysis tool a resource leak appeared. See the attached patch, that closes the filepointers correctly.

@wxtrac
Copy link
Collaborator Author

@wxtrac wxtrac commented Nov 10, 2009

2009-11-10 23:08:10: ettl.martin (Ettl Martin) uploaded file res_leak.patch (0.6 KiB)

should fix the resource leaks

@wxtrac
Copy link
Collaborator Author

@wxtrac wxtrac commented Nov 10, 2009

2009-11-10 23:39:10: ettl.martin (Ettl Martin) commented


i reported it to the expat developers:

https://sourceforge.net/tracker/?func=detail&aid=2895533&group_id=10127&atid=110127

@wxtrac
Copy link
Collaborator Author

@wxtrac wxtrac commented Nov 11, 2009

2009-11-11 02:12:53: @vadz commented


Thanks, we won't be changing this in wx as this code is never used in it but we'll pick up their fix when we upgrade to a next expat version.

2009-11-11 02:12:53: @vadz changed status from new to closed

2009-11-11 02:12:53: @vadz set resolution to wontfix

@wxtrac wxtrac closed this Nov 11, 2009
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant