View Issue Details [ Jump to Notes ]

[ Print ]

Project

Category

View Status

Date Submitted

Last Update

0012348

VTK

(No Category)

public

2011-07-09 18:56

2011-08-02 14:06

Reporter

Ivan Manuylovich

Assigned To

Kyle Lutz

Priority

high

Severity

minor

Reproducibility

have not tried

Status

closed

Resolution

fixed

Platform

OS Version

Product Version

Target Version

Fixed in Version

Summary

0012348: A bug in vtkDataSetSurfaceFilter.cxx - memory leak

Description

This issue is related with my previous one with ID 0012288
(memory leaks in vtkDLG when loading PentaHexa.vtk).

The memory leak is concerned with function vtkDataSetSurfaceFilter::InsertPolygonInHash

At the line 1595 of vtkDataSetSurfaceFilter.cxx after comment "That is all we need to do. Hide any tri shared by two or more cells." just only "return" is written and the "tab" memory isn't deallocated.

The following replace is needed at the line #1595.
Was:
return;

Should be:
delete [] tab; // MUST deallocate memory before return
return;

Tags

No tags attached.

Project

TBD

Type

incorrect functionality

Attached Files

Relationships

related to

0012288

closed

Dave DeMarle

Memory leaks in testing vtkDLG application if loaded PentaHexa.vtk file

Notes
(0026989) Kyle Lutz (viewer) 2011-07-11 13:58	Merge topic '12348-fix-surface-filter-memory-leak' into next 7f3948b Merge topic 'better-support-generic-dataset' into 12348-fix-surface-filter-memory-leak b5f3147 Merge topic '12023-read-block-names' into 12348-fix-surface-filter-memory-leak c77d81e Update VTK to bring in topic '12348-fix-surface-filter-memory-leak' Merge topic '12348-fix-surface-filter-memory-leak' into pv-next 87705c2 Free memory after match in InsertPolygonInHash().

(0027178) Ivan Manuylovich (reporter) 2011-07-31 16:41	If you are waiting for my note to close the issue then I confirm the thesis "87705c2 Free memory after match in InsertPolygonInHash()."

Issue History
Date Modified	Username	Field	Change
2011-07-09 18:56	Ivan Manuylovich	New Issue
2011-07-11 12:56	Kyle Lutz	Relationship added	related to 0012288
2011-07-11 12:56	Kyle Lutz	Assigned To	=> Kyle Lutz
2011-07-11 12:56	Kyle Lutz	Status	backlog => todo
2011-07-11 12:57	Kyle Lutz	Status	todo => active development
2011-07-11 13:58	Kyle Lutz	Note Added: 0026989
2011-07-11 13:58	Kyle Lutz	Status	active development => gatekeeper review
2011-07-31 16:41	Ivan Manuylovich	Note Added: 0027178
2011-08-01 10:05	Utkarsh Ayachit	Status	gatekeeper review => customer review
2011-08-01 10:12	Utkarsh Ayachit	Status	customer review => closed
2011-08-01 10:12	Utkarsh Ayachit	Resolution	open => fixed
2011-08-02 14:06	Kyle Lutz	Source_changeset_attached	=> VTK master 87705c2f