Automated Code Analysis

cppcheck analysis of slicer_3.4.0~svn10438-7.dsc

• ./Applications/CLI/ExtractSkeleton/SkelGraph.cxx:48 [error] - instance of "SkelGraph" object destroyed immediately
• ./Applications/CLI/UtahBSpline/Register.cxx:401 [error] - Memory leak: fixedPyramid
• ./Applications/CLI/UtahBSpline/Register.cxx:401 [error] - Memory leak: maskPyramid
• ./Applications/CLI/UtahBSpline/Register.cxx:401 [error] - Memory leak: movingPyramid
• ./Base/GUI/vtkSlicerBoxRepresentation.cxx:272 [error] - Uninitialized variable: focalPoint
• ./Base/GUI/vtkSlicerDiffusionTensorGlyphDisplayWidget.cxx:139 [error] - Possible null pointer dereference: node - otherwise it is redundant to check if node is null at line 146
• ./Base/GUI/vtkSlicerParameterWidget.cxx:943 [error] - Returning pointer to auto variable
• ./Base/GUI/vtkSlicerSliceControllerWidget.cxx:1442 [error] - Possible null pointer dereference: ssgui - otherwise it is redundant to check if ssgui is null at line 1445
• ./Base/GUI/vtkSlicerSliceControllerWidget.cxx:1587 [error] - Possible null pointer dereference: ssgui - otherwise it is redundant to check if ssgui is null at line 1590
• ./Base/GUI/vtkSlicerSliceControllerWidget.cxx:1721 [error] - Possible null pointer dereference: ssgui - otherwise it is redundant to check if ssgui is null at line 1723
• ./Base/GUI/vtkSlicerSliceControllerWidget.cxx:3026 [error] - Possible null pointer dereference: ssgui - otherwise it is redundant to check if ssgui is null at line 3028
• ./Base/GUI/vtkSlicerSlicesControlGUI.cxx:1570 [error] - Invalid number of character ({) when these macros are defined: 'FOV_ENTRIES_DEBUG'.
• ./Base/GUI/vtkSlicerViewerWidget.cxx:1192 [error] - Possible null pointer dereference: modelDisplayNode - otherwise it is redundant to check if modelDisplayNode is null at line 1194
• ./Base/GUI/vtkSlicerViewerWidget.cxx:1637 [error] - Possible null pointer dereference: mdnode - otherwise it is redundant to check if mdnode is null at line 1725
• ./Base/Logic/vtkDataIOManagerLogic.cxx:634 [error] - Possible null pointer dereference: task - otherwise it is redundant to check if task is null at line 638
• ./Libs/FreeSurfer/vtkFSSurfaceAnnotationReader.cxx:258 [error] - Memory leak: rgbs
• ./Libs/FreeSurfer/vtkFSSurfaceAnnotationReader.cxx:258 [error] - Memory leak: labels
• ./Libs/FreeSurfer/vtkFSSurfaceAnnotationReader.cxx:877 [error] - Resource leak: file
• ./Libs/FreeSurfer/vtkFSSurfaceLabelReader.cxx:100 [error] - Resource leak: labelFile
• ./Libs/FreeSurfer/vtkFSSurfaceLabelReader.cxx:143 [error] - Memory leak: scalars
• ./Libs/FreeSurfer/vtkFSSurfaceScalarReader.cxx:118 [error] - Memory leak: FSscalars
• ./Libs/FreeSurfer/vtkFSSurfaceWFileReader.cxx:130 [error] - Memory leak: FSscalars
• ./Libs/ModuleDescriptionParser/ModuleFactory.cxx:683 [error] - Possible null pointer dereference: logoWidth
• ./Libs/ModuleDescriptionParser/ModuleFactory.cxx:683 [error] - Possible null pointer dereference: logoHeight
• ./Libs/ModuleDescriptionParser/ModuleFactory.cxx:684 [error] - Possible null pointer dereference: logoPixelSize
• ./Libs/ModuleDescriptionParser/ModuleFactory.cxx:684 [error] - Possible null pointer dereference: logoLength
• ./Libs/ModuleDescriptionParser/ModuleFactory.cxx:1493 [error] - Possible null pointer dereference: logoWidth
• ./Libs/ModuleDescriptionParser/ModuleFactory.cxx:1493 [error] - Possible null pointer dereference: logoHeight
• ./Libs/ModuleDescriptionParser/ModuleFactory.cxx:1494 [error] - Possible null pointer dereference: logoPixelSize
• ./Libs/ModuleDescriptionParser/ModuleFactory.cxx:1494 [error] - Possible null pointer dereference: logoLength
• ./Libs/ModuleDescriptionParser/ModuleFactory.cxx:1242 [error] - Possible null pointer dereference: logoWidth
• ./Libs/ModuleDescriptionParser/ModuleFactory.cxx:1242 [error] - Possible null pointer dereference: logoHeight
• ./Libs/ModuleDescriptionParser/ModuleFactory.cxx:1243 [error] - Possible null pointer dereference: logoPixelSize
• ./Libs/ModuleDescriptionParser/ModuleFactory.cxx:1243 [error] - Possible null pointer dereference: logoLength
• ./Libs/Qdec/QdecDataTable.cpp:73 [error] - Memory leak: tmpstr
• ./Libs/Qdec/QdecDataTable.cpp:360 [error] - Allocation with strdup, printf doesn't release it.
• ./Libs/vtkTeem/vtkDiffusionTensorGlyph.cxx:229 [error] - Memory leak: pts
• ./Libs/vtkTeem/vtkPreciseHyperStreamline.cxx:364 [error] - Possible null pointer dereference: inTensors - otherwise it is redundant to check if inTensors is null at line 367
• ./Libs/vtkTeem/vtkPreciseHyperStreamline.cxx:349 [error] - Uninitialized variable: v0
• ./Libs/vtkTeem/vtkPreciseHyperStreamline.cxx:350 [error] - Uninitialized variable: m0
• ./Libs/vtkTeem/vtkTensorImplicitFunctionToFunctionSet.cxx:91 [error] - Uninitialized variable: f
• ./Modules/ChangeTracker/DeformableMetric/applyTransformationITK.cxx:114 [error] - Resource leak: fid
• ./Modules/EMSegment/Algorithm/vtkFileOps.cxx:234 [error] - Memory leak: tmp
• ./Modules/EMSegment/Algorithm/vtkImageEMGeneral.cxx:554 [error] - Undefined behaviour: OpenFileName is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./Modules/EMSegment/Algorithm/vtkImageEMGeneral.cxx:555 [error] - Undefined behaviour: OpenFileName is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./Modules/EMSegment/Algorithm/vtkImageEMGeneral.cxx:556 [error] - Undefined behaviour: OpenFileName is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./Modules/EMSegment/Algorithm/vtkImageEMGeneral.cxx:557 [error] - Undefined behaviour: OpenFileName is used wrong in call to sprintf or snprintf. Quote: If copying takes place between objects that overlap as a result of a call to sprintf() or snprintf(), the results are undefined.
• ./Modules/EMSegment/Algorithm/vtkImageEMLocalSegmenter.cxx:471 [error] - Memory leak: NewLevelName
• ./Modules/EMSegment/Wizard/vtkEMSegmentIntensityImagesStep.cxx:309 [error] - Possible null pointer dereference: volumeNode
• ./Modules/FetchMI/vtkFetchMIWriterXND.cxx:181 [error] - Memory leak: dtdBuffer
• ./Modules/FetchMI/vtkFetchMIWriterXND.cxx:343 [error] - Resource leak: fptr
• ./Modules/FetchMI/vtkFetchMIWriterXND.cxx:367 [error] - Memory leak: dtdBuffer
• ./Modules/Meshing/Adapter/vtkUnstructuredGridToitkMesh.cxx:125 [error] - Memory leak: vtkCellTypes
• ./Modules/Meshing/BuildingBlock/IaFeMeshR21.cxx:123 [error] - Uninitialized variable: ret
• ./Modules/Meshing/BuildingBlock/vtkKWMimxAssignBoundaryConditionsGroup.cxx:1137 [error] - Possible null pointer dereference: strarray
• ./Modules/Meshing/Common/vtkMimxActorBase.cxx:49 [error] - Mismatching allocation and deallocation: vtkMimxActorBase::FilePath
• ./Modules/Meshing/Common/vtkMimxActorBase.cxx:48 [error] - Mismatching allocation and deallocation: vtkMimxActorBase::FileName
• ./Modules/Meshing/Common/vtkMimxActorBase.cxx:50 [error] - Mismatching allocation and deallocation: vtkMimxActorBase::UniqueId
• ./Modules/Meshing/Common/vtkMimxActorBase.cxx:51 [error] - Mismatching allocation and deallocation: vtkMimxActorBase::FoundationName
• ./Modules/Meshing/Common/vtkMimxMeshActor.cxx:2363 [error] - Memory leak: range
• ./Modules/Meshing/Widgets/vtkMimxBoundingBoxWidget.cxx:874 [error] - Uninitialized variable: x
• ./Modules/Meshing/Widgets/vtkMimxPolyDataWidget.cxx:758 [error] - Uninitialized variable: x
• ./Modules/Meshing/Widgets/vtkMimxUnstructuredGridWidget.cxx:1218 [error] - Uninitialized variable: x
• ./Modules/Meshing/Widgets/vtkMimxViewMergePointsWidget.cxx:1217 [error] - Uninitialized variable: x
• ./Modules/OpenIGTLinkIF/vtkOpenIGTLinkIFLogic.cxx:480 [error] - Dangerous iterator usage. After erase the iterator is invalid so dereferencing it or comparing it with another iterator is invalid.
• ./Modules/ProstateNav/Wizard/vtkProstateNavCalibrationStep.cxx:463 [error] - Returning pointer to auto variable
• ./Modules/ProstateNav/Wizard/vtkProstateNavCalibrationStep.cxx:489 [error] - Returning pointer to auto variable
• ./Modules/ProstateNav/vtkMRMLBrpRobotCommandNode.cxx:35 [error] - Uninitialized variable: ret
• ./Modules/QdecModule/vtkGDFReader.cxx:163 [error] - Resource leak: fp
• ./Modules/QueryAtlas/vtkQueryAtlasGUI.cxx:3246 [error] - Possible null pointer dereference: l - otherwise it is redundant to check if l is null at line 3248
• ./Modules/QueryAtlas/vtkQueryAtlasGUI.cxx:3279 [error] - Possible null pointer dereference: l - otherwise it is redundant to check if l is null at line 3281
• ./Modules/SlicerDaemon/vtkTclHelper.cxx:151 [error] - Possible null pointer dereference: tensorArray - otherwise it is redundant to check if tensorArray is null at line 153
• ./Modules/SlicerDaemon/vtkTclHelper.cxx:283 [error] - Possible null pointer dereference: tensorArray - otherwise it is redundant to check if tensorArray is null at line 285
• ./Modules/SlicerDaemon/vtkTclHelper.cxx:437 [error] - Memory leak: serializedTensorsP
• ./Modules/SlicerDaemon/vtkTclHelper.cxx:527 [error] - Memory leak: serializedTensorsP
• ./Modules/VolumeRendering/VolumeRenderingReplacements/vtkSlicerBoxRepresentation.cxx:272 [error] - Uninitialized variable: focalPoint
• ./Modules/VolumeRendering/VolumeRenderingReplacements/vtkSlicerFixedPointVolumeRayCastMapper.cxx:1044 [error] - Memory leak: minNonZeroScalarIndex
• ./Modules/VolumeRendering/VolumeRenderingReplacements/vtkSlicerFixedPointVolumeRayCastMapper.cxx:1044 [error] - Memory leak: minNonZeroGradientMagnitudeIndex
• ./Modules/Volumes/vtkSlicerDiffusionTestingWidget.cxx:345 [error] - Possible null pointer dereference: displayNode

---