View Issue Details [ Jump to Notes ]

[ Print ]

Project

Category

View Status

Date Submitted

Last Update

0011360

ITK

public

2010-10-25 15:33

2010-10-25 20:08

Reporter

kentwilliams

Assigned To

alexandre gouaillard

Priority

normal

Severity

major

Reproducibility

always

Status

assigned

Resolution

open

Platform

OS Version

Product Version

ITK-4-A3

Target Version

Fixed in Version

Summary

0011360: itk::AutoPointer members in mesh cells leaking

Description

Discovered while checking this bug report:

http://public.kitware.com/Bug/view.php?id=1033 [^]

valgrind is reporting memory leakes in the itkSimplexMeshToTriangleMeshFilterTest

The issue seems to be AutoPointers in the mesh cell classes somehow not being destroyed properly;

Additional Information

If you apply the patch below, which, by assigning 0 to the itk::SmartPointer instances that were leaking, the valgrind leak report goes away. This doesn't make any sense to me.

diff --git a/Testing/Code/BasicFilters/itkSimplexMeshToTriangleMeshFilterTest.cxx b/Testing/Code/BasicFilters/itkSimplexMeshToTriangleMeshFilterTest.cxx
index e74bc90..5d2da0b 100644
--- a/Testing/Code/BasicFilters/itkSimplexMeshToTriangleMeshFilterTest.cxx
+++ b/Testing/Code/BasicFilters/itkSimplexMeshToTriangleMeshFilterTest.cxx
@@ -74,6 +74,8 @@ int itkSimplexMeshToTriangleMeshFilterTest( int , char * [] )
   std::cout << "Back filtered Triangle Mesh: " << triangleMesh << std::endl;

   std::cout << "[TEST DONE]" << std::endl;
+ originalTriangleMesh = 0;
+ simplexFilter = 0;
   return EXIT_SUCCESS;

}

Tags

No tags attached.

Resolution Date

Sprint

Sprint Status

backlog

Attached Files

Relationships

Notes
(0022684) Bradley Lowekamp (developer) 2010-10-25 15:45	Is it possible there was a circular dependency with the smart pointers?

Issue History
Date Modified	Username	Field	Change
2010-10-25 15:33	kentwilliams	New Issue
2010-10-25 15:33	kentwilliams	Sprint Status	=> backlog
2010-10-25 15:45	Bradley Lowekamp	Note Added: 0022684
2010-10-25 20:08	Luis Ibanez	Status	new => assigned
2010-10-25 20:08	Luis Ibanez	Assigned To	=> alexandre gouaillard