View Issue Details [ Jump to Notes ]

[ Print ]

Project

Category

View Status

Date Submitted

Last Update

0015888

VTK

(No Category)

public

2015-12-16 22:34

2016-07-06 11:30

Reporter

Carlos Azevedo

Assigned To

Berk Geveci

Priority

normal

Severity

minor

Reproducibility

have not tried

Status

closed

Resolution

unable to reproduce

Platform

OS Version

Product Version

Target Version

Fixed in Version

Summary

0015888: FIX FOR: Paraview crashes loading VTU files

Description

I've been unable to load several VTU files into Paraview 5.0.0-RC1. Compiled it from source (cloned from Git master branch) on Ubuntu 14.04 LTS and found the problem on the InsertPolygonInHash method of vtkDataSetSurfaceFilter.
If the numPts parameter passed in is zero (which my data files apparently cause) the code will allocate a zero sized array of vtkIdTypes, which will cause undefined behaviour (a crash in my case) when dereferencing it later.
I've added the following lines to prevent the crash, not sure if something else should be fixed also:

  vtkFastGeomQuad *quad, **end;

+ // Bail out if 'numPts' == 0, otherwise a crash will insue due to 'end = this->QuadHash + tab[0];'
+ if (numPts == 0)
+ {
+ return;
+ }

  // find the index to the smallest id
  vtkIdType offset = 0;

This issue still exists in VTK's master repository as of today. Sorry for not providing a proper patch...

Tags

No tags attached.

Project

TBD

Type

crash

Attached Files

Relationships

Notes
(0036180) Berk Geveci (administrator) 2016-07-06 11:29	Can you please provide an example dataset? It's unclear how a dataset would cause this to have 0 points. It may be an issue with the dataset.

Issue History
Date Modified	Username	Field	Change
2015-12-16 22:34	Carlos Azevedo	New Issue
2016-07-06 11:29	Berk Geveci	Note Added: 0036180
2016-07-06 11:30	Berk Geveci	Assigned To	=> Berk Geveci
2016-07-06 11:30	Berk Geveci	Status	backlog => closed
2016-07-06 11:30	Berk Geveci	Resolution	open => unable to reproduce