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: Muon and NeXus load issues #797
Conversation
@@ -77,16 +77,16 @@ class DLLExport AppendGeometryToSNSNexus : public API::Algorithm { | |||
bool m_makeNexusCopy; | |||
|
|||
/// Algorithm progress keeper | |||
API::Progress *progress; | |||
API::Progress *m_progress; | |||
|
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.
With this m_progress I think we have the same issue as I've seen in other algorithms. Somewhere in the implementation it is allocated with new but then never deleted. This is just a small leak but it could be fixed here, simply adding the missing delete or, most likely better, by using a smart pointer. Oh, and also progress/m_progess does not need to be a member of the class, it can live just inside the exec method().
We got failures in the |
Coverity issues: Muon and NeXus load issues
Fixes #11825
For tester: code review, check that all the issues reported in the trac ticket have been solved.