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

Coverity issues (resource leaks) in Framework/MDEvents #247

Merged
merged 3 commits into from Feb 19, 2015

Conversation

FedeMPouzols
Copy link
Contributor

Fixes trac issue #11060 - a few memory leaks of supposed "high impact" but not that terrible in the end. See the ticket for a list of issues with short description.

To test:

  • review code
  • check that tests pass (these classes are in principle well tested).

@FedeMPouzols FedeMPouzols added this to the Release 3.4 milestone Feb 19, 2015
@FedeMPouzols FedeMPouzols added Maintenance Unassigned issues to be addressed in the next maintenance period. Coverity Framework Issues and pull requests related to components in the Framework labels Feb 19, 2015
@DanNixon DanNixon self-assigned this Feb 19, 2015
DanNixon added a commit that referenced this pull request Feb 19, 2015
…aks_in_MDEvents

Coverity issues (resource leaks) in Framework/MDEvents
@DanNixon DanNixon merged commit 077007e into master Feb 19, 2015
@DanNixon DanNixon deleted the 11060_coverity_resource_leaks_in_MDEvents branch February 19, 2015 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Framework Issues and pull requests related to components in the Framework Maintenance Unassigned issues to be addressed in the next maintenance period.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants