View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0012418 | ParaView | (No Category) | public | 2011-07-28 12:40 | 2011-08-25 17:11 | ||||
Reporter | Kyle Lutz | ||||||||
Assigned To | Kyle Lutz | ||||||||
Priority | normal | Severity | minor | Reproducibility | have not tried | ||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | |||||||||
Target Version | Fixed in Version | 3.12 | |||||||
Summary | 0012418: Segfault in vtkQuadricClustering::AddEdges() | ||||||||
Description | On Wed, Jun 29, 2011 at 11:21 AM, Dennis McWherter <dmcwhe2@illinois.edu> wrote: > Hello, > > I have generated a Legacy VTK file using VTK's Python wrapper. This file > crashes ParaView 3.10.1 (Windows 7) when attempting to rotate it in > ParaView's viewport, however, it seems that I can appropriately apply > filters and such. However, this data did work properly on ParaView 3.8.1. > > If anyone can help in inspecting the file for potential bugs, it can be > found here (~55MB): > https://netfiles.uiuc.edu/xythoswfs/webview/_xy-22178502_2 [^] > > ParaView crashes in vtkQuadricClustering::AddEdges at this line: > > points->GetPoint(ptIds[0], pt0); > > ptIds is NULL. > > The data itself is a PolyData object with PolyLines and arrays. I presume > there may be a bad/missing data structure within the data, can any ParaView > expert shed some light on the issue? > > Regards, > Dennis McWherter | ||||||||
Tags | No tags attached. | ||||||||
Project | TBD | ||||||||
Topic Name | 12418-fix-quadric-clustering-segfault | ||||||||
Type | crash | ||||||||
Attached Files | |||||||||
Relationships | |
Relationships |
Notes | |
(0027214) Kyle Lutz (viewer) 2011-08-03 13:17 |
Bisected: commit b0391d81669b40f31edd4cf447c413f9d75d290a Author: François Bertel <francois.bertel@kitware.com> Date: Wed Jul 14 10:31:30 2010 -0400 BUG:Fixed vtkCellArray::GetNextCell when traversal is done. npts and pts were not initialized to 0. vtkVRMLImporter is using GetNextCell directly without checking for the return value and then has a loop involving npts. This also fixes a warning about the fact that npts may be used uninitialized in vtkVRMLImporter. diff --git a/Filtering/vtkCellArray.h b/Filtering/vtkCellArray.h index 7593639..283aadf 100644 --- a/Filtering/vtkCellArray.h +++ b/Filtering/vtkCellArray.h @@ -313,6 +313,8 @@ inline int vtkCellArray::GetNextCell(vtkIdType& npts, vtkIdType* &pts) this->TraversalLocation += npts; return 1; } + npts=0; + pts=0; return 0; } |
(0027219) Kyle Lutz (viewer) 2011-08-03 14:35 |
Merge topic '12418-fix-quadric-clustering-segfault-next' into next c750580 Merge topic '11580_fix_extract_surface_subdivision-next' into 12418-fix-quadric-clustering-segfault 8954727 Update VTK to bring in '12418-fix-quadric-clustering-segfault' Merge topic '12418-fix-quadric-clustering-segfault' into pv-next 58414a3 Check for a non-empty cell before using its points |
(0027232) Utkarsh Ayachit (administrator) 2011-08-04 18:08 |
Developer note: Issue slated for gatekeeper review on Aug 7, 2 PM EDT. |
(0027270) Utkarsh Ayachit (administrator) 2011-08-05 11:32 |
verified. |
(0027356) Utkarsh Ayachit (administrator) 2011-08-12 14:23 |
Merged into git-master |
Notes |
Issue History | |||
Date Modified | Username | Field | Change |
2011-07-28 12:40 | Kyle Lutz | New Issue | |
2011-07-28 12:51 | Kyle Lutz | Assigned To | => Kyle Lutz |
2011-07-28 12:51 | Kyle Lutz | Status | backlog => todo |
2011-08-03 13:17 | Kyle Lutz | Note Added: 0027214 | |
2011-08-03 14:21 | Kyle Lutz | Status | todo => active development |
2011-08-03 14:35 | Kyle Lutz | Topic Name | => 12418-fix-quadric-clustering-segfault |
2011-08-03 14:35 | Kyle Lutz | Note Added: 0027219 | |
2011-08-03 14:35 | Kyle Lutz | Status | active development => gatekeeper review |
2011-08-03 14:35 | Kyle Lutz | Fixed in Version | => 3.12 |
2011-08-03 14:35 | Kyle Lutz | Resolution | open => fixed |
2011-08-04 18:08 | Utkarsh Ayachit | Note Added: 0027232 | |
2011-08-05 11:32 | Utkarsh Ayachit | Note Added: 0027270 | |
2011-08-12 14:23 | Utkarsh Ayachit | Status | gatekeeper review => customer review |
2011-08-12 14:23 | Utkarsh Ayachit | Note Added: 0027356 | |
2011-08-25 17:11 | Kyle Lutz | Status | customer review => closed |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |