View Issue Details [ Jump to Notes ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0014725VTK(No Category)public2014-05-12 19:042014-05-14 18:05
ReporterAndras Lasso 2 
Assigned ToDavid Gobbi 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version6.2.0 
Summary0014725: Incorrect null-pointer check in vtkBSplineTransform causes incorrect ForwardTransformPoint result
DescriptionThere is a typo in
vtkBSplineTransform::ForwardTransformPoint(const double inPoint[3], double outPoint[3]) that prevents proper computation of the transformed point position.

The first line of the method should be

  if (!this->GridPointer || !this->CalculateSpline)

instead it is

  if (this->GridPointer || !this->CalculateSpline)

See:
https://github.com/Kitware/VTK/blob/master/Filters/Hybrid/vtkBSplineTransform.cxx#L461 [^]
TagsNo tags attached.
ProjectTBD
Typeincorrect functionality
Attached Files

 Relationships

  Notes
(0032625)
David Gobbi (developer)
2014-05-13 08:25

 Thanks for the report, I've pushed a fix to gerrit (http://review.source.kitware.com/15429 [^]) but still need to improve the test coverage. The fix will go into VTK 6.2.
(0032626)
Andras Lasso 2 (reporter)
2014-05-13 09:36

 Great, thanks!
(0032631)
David Gobbi (developer)
2014-05-14 18:05

 Merged along with more tests, commit ddf898ae

 Issue History
Date Modified Username Field Change
2014-05-12 19:04 Andras Lasso 2 New Issue
2014-05-13 08:25 David Gobbi Note Added: 0032625
2014-05-13 08:25 David Gobbi Assigned To => David Gobbi
2014-05-13 08:25 David Gobbi Status backlog => gerrit review
2014-05-13 09:36 Andras Lasso 2 Note Added: 0032626
2014-05-14 18:05 David Gobbi Note Added: 0032631
2014-05-14 18:05 David Gobbi Status gerrit review => closed
2014-05-14 18:05 David Gobbi Resolution open => fixed
2014-05-14 18:05 David Gobbi Fixed in Version => 6.2.0

Copyright © 2000 - 2018 MantisBT Team