Ticket #6157 (closed: fixed)
SANS: Memory Leak on Diagnostics
| Reported by: | Gesner Passos | Owned by: | Gesner Passos |
|---|---|---|---|
| Priority: | major | Milestone: | Release 2.4 |
| Component: | Mantid | Keywords: | |
| Cc: | anders.markvardsen@… | Blocked By: | |
| Blocking: | Tester: | Martyn Gigg |
Description
This commit introduced a memory leak, as long the memory allocated for map is not released.
http://trac.mantidproject.org/mantid/changeset/e47d661bb1f18889046242cb8c8a65e0373ae922/
This commit was related to this ticket: #2470
Attachments
Change History
Changed 8 years ago by Gesner Passos
- Attachment patch_2470 added
comment:2 Changed 8 years ago by Gesner Passos
Solving leak memory for SANSDiagnostics.cpp
Release the memory allocated for listing the spectra related to the detectors IDs. RE #6157
Changeset: 0da74222554ca2a37edfff7d0c4630910c215237
comment:3 Changed 8 years ago by Anders Markvardsen
- Status changed from accepted to assigned
- Priority changed from critical to major
comment:4 Changed 8 years ago by Gesner Passos
Solving leak memory for SANSDiagnostics.cpp
Release the memory allocated for listing the spectra related to the detectors IDs. RE #6157
Changeset: 0da74222554ca2a37edfff7d0c4630910c215237
comment:5 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.4 to Release 2.5
Moved at the code freeze for release 2.4
comment:8 Changed 8 years ago by Gesner Passos
- Status changed from accepted to verify
- Resolution set to fixed
comment:9 Changed 8 years ago by Martyn Gigg
- Status changed from verify to verifying
- Tester set to Martyn Gigg
comment:10 Changed 8 years ago by Martyn Gigg
- Status changed from verifying to closed
Confirmed that the change resolves the memory leak.
comment:11 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 7003
This patch should solve the memory leak.