Ticket #5399 (closed: fixed)
BoxController::m_numSplit is not initialized
Reported by: | Russell Taylor | Owned by: | Owen Arnold |
---|---|---|---|
Priority: | minor | Milestone: | Release 2.2 |
Component: | Mantid | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Russell Taylor |
Description
Presumably this has not caused any noticeable problem, but valgrind complains about it not being initialized:
==25377== Conditional jump or move depends on uninitialised value(s) ==25377== at 0x5CFAD65: Mantid::API::BoxController::resetMaxNumBoxes() (BoxController.h:451) ==25377== by 0x5CFAC2B: Mantid::API::BoxController::resetNumBoxes() (BoxController.h:337) ==25377== by 0x1AD32A49: Mantid::API::BoxController::BoxController(unsigned long) (BoxController.h:45) ==25377== by 0x1ADB2D4C: Mantid::MDEvents::MDEventWorkspace<Mantid::MDEvents::MDLeanEvent<3ul>, 3ul>::MDEventWorkspace() (MDEventWorkspace.cpp:40) ==25377== by 0x1AD31F95: Mantid::MDEvents::MDEventFactory::CreateMDWorkspace(unsigned long, std::string const&) (MDEventFactory.cpp:230) ==25377== by 0x1AC562A8: Mantid::MDEvents::ConvertToDiffractionMDWorkspace::exec() (ConvertToDiffractionMDWorkspace.cpp:446) ==25377== by 0x5CB707A: Mantid::API::Algorithm::execute() (Algorithm.cpp:541) ==25377== ==25377== Conditional jump or move depends on uninitialised value(s) ==25377== at 0x5CFAD65: Mantid::API::BoxController::resetMaxNumBoxes() (BoxController.h:451) ==25377== by 0x5CFAC2B: Mantid::API::BoxController::resetNumBoxes() (BoxController.h:337) ==25377== by 0x5CFAB8D: Mantid::API::BoxController::setMaxDepth(unsigned long) (BoxController.h:235) ==25377== by 0x1AC15E52: Mantid::MDEvents::BoxControllerSettingsAlgorithm::setBoxController(boost::shared_ptr<Mantid::API::BoxController>) (BoxControllerSettingsAlgorithm.cpp:91) ==25377== by 0x1AC56A23: Mantid::MDEvents::ConvertToDiffractionMDWorkspace::exec() (ConvertToDiffractionMDWorkspace.cpp:472)
I expect it should be initialized to 1, but I'm not certain...
Change History
comment:1 Changed 8 years ago by Nick Draper
- Status changed from new to assigned
- Owner set to Owen Arnold
comment:2 Changed 8 years ago by Owen Arnold
refs #5399. Fix issue identified by valgrind
Changeset: 24e311fc4cc7ad6837495b779951cc8b5d26a807
comment:4 Changed 8 years ago by Owen Arnold
refs #5399. Fix initialisation order warning
Changeset: d24362be7a8494064860014165c019732f6a562e
comment:5 Changed 8 years ago by Owen Arnold
- Status changed from accepted to verify
- Resolution set to fixed
Tester: This is a simple fix. It should be clear from the code changes.
comment:6 Changed 8 years ago by Owen Arnold
refs #5399. Fix issue identified by valgrind
Changeset: 24e311fc4cc7ad6837495b779951cc8b5d26a807
comment:7 Changed 8 years ago by Owen Arnold
refs #5399. Fix initialisation order warning
Changeset: d24362be7a8494064860014165c019732f6a562e
comment:8 Changed 8 years ago by Russell Taylor
- Status changed from verify to verifying
- Tester set to Russell Taylor
comment:10 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 6245