|Anonymous | Login||2017-07-25 18:47 EDT|
|My View | View Issues | Change Log | Roadmap|
|View Issue Details|
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0000721||VTK||(No Category)||public||2004-03-26 21:54||2016-08-12 09:54|
|Assigned To||David Partyka|
|Target Version||Fixed in Version|
|Summary||0000721: Picking a vtkPolyData with degenerate cells|
|Description||If a degenerate triangle with two identical vertices is traversed by vtkCellPicker, a crash is caused.|
"denom" on line 272 (vtkLine.cxx) will evaluate to zero and will cause a division by zero error.
Perhaps, a check here to return -1 if denom == 0, then a check in vtkTriangle::EvaluatePosition to return -1 (ie numerical error) if vtkLine::DistanceToLine returns -1
vtkTriangle::IntersectWithLine should also check for a error from vtkTriangle::EvaluatePosition and vtkLine::DistanceToLine and return 0;
|Tags||No tags attached.|
|Attached Files|| vtkLine.cxx.patch [^] (578 bytes) 1969-12-31 19:00 [Show Content]
vtkTriangle.cxx.patch [^] (597 bytes) 1969-12-31 19:00 [Show Content]
Goodwin Lawlor (reporter)
Rather than vtkTriangle::IntersectWithLine checking return values from vtkTriangle::EvaluatePosition and vtkLine::DistanceToLine, maybe just a check at the beginning of the method to see if the triangle is degenerate.
If fact perhaps every cell type should have a method to check itself for degeneracy, and each cell method should use this check before it executes.
Mathieu Malaterre (developer)
Waht does :
vtkTriangle::ComputeNormal (pt1, pt2, pt3, n);
return in your case. Maybe we can check the result to determine if the triangle is degenerate.
Goodwin Lawlor (reporter)
Uploaded two patches that fix a crash after cell picking a band-contoured polydata, ie, a polydata that could have some degenerate cells.
Dave DeMarle (administrator)
|Dave P no longer works on the project. If these old issues still exist in 6.0.0, reopen them and assign to Dave DeMarle|
Kitware Robot (administrator)
Resolving issue as `moved`.
This issue tracker is no longer used. Further discussion of this issue may take place in the current VTK Issues page linked in the banner at the top of this page.
|2008-11-30 09:17||Mathieu Malaterre||Assigned To||Mathieu Malaterre => David Cole|
|2011-01-19 08:52||Utkarsh Ayachit||Assigned To||David Cole => David Partyka|
|2011-06-16 13:11||Zack Galbreath||Category||=> (No Category)|
|2013-07-22 20:33||Dave DeMarle||Status||backlog => expired|
|2013-07-22 20:33||Dave DeMarle||Note Added: 0031303|
|2016-08-12 09:54||Kitware Robot||Note Added: 0036721|
|2016-08-12 09:54||Kitware Robot||Status||expired => closed|
|2016-08-12 09:54||Kitware Robot||Resolution||open => moved|
|Copyright © 2000 - 2017 MantisBT Team|