View Issue Details

IDProjectCategoryView StatusLast Update
0023489CommunityOCCT:Application Frameworkpublic2013-10-02 15:52
ReporterRoman Lygin Assigned ToRoman Lygin  
PrioritynormalSeveritymajor 
Status closedResolutionfixed 
PlatformAOSL 
Product Version6.5.3 
Target Version6.5.4Fixed in Version6.5.4 
Summary0023489: Memory leak in TNaming_NamedShape
DescriptionUpon TNaming_NamedShape destruction, internal shape collections do not get freed leading to massive leaks.

The fix will be to add a destructor that will call TNaming_NamedShape::Clear().
Steps To ReproduceSee the enclosed test case.
With the enforced clean up by calling TNaming_NamedShape::Clear(), the real destruction takes place.
TagsNo tags attached.
Test case numberbugs caf bug23489

Attached Files

  • tnaming_namedshape_memoryleak.cxx (1,242 bytes)
  • tnaming_namedshape_memoryleak-v2.cxx (1,038 bytes)

Relationships

related to 0023554 closedPawel Community Crash when closing BinXCAF document 
related to 0023901 closedomy Open CASCADE Memory leaks in tests 
related to 0024164 closedbugmaster Open CASCADE Optimization of OCAF document closing 

Activities

Roman Lygin

2012-10-24 23:57

developer  

tnaming_namedshape_memoryleak.cxx (1,242 bytes)

Roman Lygin

2012-10-25 00:55

developer   ~0021934

Fixed pushed in branch CR23489.
The slightly simplified test case (without warmup) enclosed.

Roman Lygin

2012-10-25 00:55

developer  

tnaming_namedshape_memoryleak-v2.cxx (1,038 bytes)

szy

2012-10-25 11:38

manager   ~0021935

Reviewed. It's Ok.

abv

2012-10-31 16:14

manager   ~0022034

I have committed test case for memory leak (bugs ocaf bug23489), please review and check on Linux

apn

2012-11-01 11:00

administrator   ~0022043

Dear BugMaster,
Branch CR23489(and products from GIT master) was compiled on Linux and Windows platforms and tested.

Regression:
Not detected

Improvements:
Not detected

Testing case:
bugs caf bug23489 - OK

szy

2012-11-01 11:22

manager   ~0022044

Additional Draw command <ClearNS Doc Entry> is integrated in CR23489. This command allows to clean NamedShape attribute's data structure if found at the label with the specified Entry.

abv

2012-11-02 09:40

manager   ~0022064

I have pushed modified version of the test case and procedure for evaluating memory leaks to branch CR23489, please check on Linux how many iterations are needed now to confirm absence of the leak

Related Changesets

occt: master deb26df7

2012-11-09 12:55:48

Roman Lygin

Details Diff
0023489: Memory leak in TNaming_NamedShape

Added test for memory leak (bugs/ocaf/bug23489)
Missing return added in QANewBRepNaming_BooleanOperationFeat::IsWRCase2()
Adding test case and general procedure for trend checking for detection of memory leaks
Affected Issues
0023489
mod - src/QANewBRepNaming/QANewBRepNaming_BooleanOperationFeat.cxx Diff File
mod - src/TNaming/TNaming_NamedShape.cdl Diff File
mod - tests/bugs/begin Diff File
add - tests/bugs/caf/bug23489 Diff File
mod - tests/bugs/grids.list Diff File

Issue History

Date Modified Username Field Change
2012-10-24 23:57 Roman Lygin New Issue
2012-10-24 23:57 Roman Lygin Assigned To => szy
2012-10-24 23:57 Roman Lygin File Added: tnaming_namedshape_memoryleak.cxx
2012-10-25 00:55 Roman Lygin Note Added: 0021934
2012-10-25 00:55 Roman Lygin Status new => resolved
2012-10-25 00:55 Roman Lygin File Added: tnaming_namedshape_memoryleak-v2.cxx
2012-10-25 09:02 abv Target Version => 6.5.4
2012-10-25 11:38 szy Note Added: 0021935
2012-10-25 11:38 szy Assigned To szy => mkv
2012-10-25 11:38 szy Status resolved => reviewed
2012-10-31 16:14 abv Note Added: 0022034
2012-11-01 11:00 apn Note Added: 0022043
2012-11-01 11:00 apn Test case number => bugs caf bug23489
2012-11-01 11:00 apn Assigned To mkv => bugmaster
2012-11-01 11:00 apn Status reviewed => tested
2012-11-01 11:22 szy Note Added: 0022044
2012-11-02 09:40 abv Note Added: 0022064
2012-11-02 09:40 abv Assigned To bugmaster => apn
2012-11-02 09:40 abv Status tested => feedback
2012-11-02 18:28 apn Assigned To apn => bugmaster
2012-11-02 18:28 apn Status feedback => tested
2012-11-12 11:33 Roman Lygin Changeset attached => occt master deb26df7
2012-11-12 11:33 Roman Lygin Assigned To bugmaster => Roman Lygin
2012-11-12 11:33 Roman Lygin Status tested => verified
2012-11-12 11:33 Roman Lygin Resolution open => fixed
2012-11-14 18:24 abv Relationship added related to 0023554
2012-11-16 13:13 bugmaster Fixed in Version => 6.5.4
2012-11-16 13:18 bugmaster Status verified => closed
2013-04-16 14:11 omy Relationship added related to 0023901
2013-10-02 15:52 abv Relationship added related to 0024164