View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0004485Slicer4Core: Base Codepublic2017-12-06 12:002018-05-27 23:41
Reporterinorton Assigned Tojcfr  
PrioritynormalSeverityminorReproducibilityhave not tried
Status resolvedResolutionfixed 
Product Version 
Target VersionFixed in VersionSlicer 4.9.0 
Summary0004485: Unsafe call in vtkMRMLModelDisplayNode::GetOutputMesh
Description

These lines of vtkMRMLModelDisplayNode::GetOutputMesh are potentially unsafe:

https://github.com/Slicer/Slicer/blob/57b1022969917d39d73958983426b250af93e4d2/Libs/MRML/Core/vtkMRMLModelDisplayNode.cxx#L220-L221

 vtkAlgorithm* producer =  outputConnection? outputConnection->GetProducer() : 0;
  producer->Update();

If outputConnection->GetProducer is considered an invariant then we should probably just assert(producer) rather than using the ternary.

(I don't have a crash reproducer, simply noticed this while working through something else)

TagsNo tags attached.

Activities

lassoan

lassoan

2018-05-27 23:41

developer   ~0015751

The current trunk version does not have this issue anymore.

Issue History

Date Modified Username Field Change
2017-12-06 12:00 inorton New Issue
2017-12-06 12:00 inorton Status new => assigned
2017-12-06 12:00 inorton Assigned To => jcfr
2018-05-27 23:41 lassoan Status assigned => resolved
2018-05-27 23:41 lassoan Resolution open => fixed
2018-05-27 23:41 lassoan Fixed in Version => Slicer 4.9.0
2018-05-27 23:41 lassoan Note Added: 0015751