MantisBT - Community
View Issue Details
0025128Community[OCCT] OCCT:Modeling Algorithmspublic2014-08-06 08:592014-11-11 12:57
emv 
bugmaster 
normalminor 
closedfixed 
[OCCT] 6.7.1 
[OCCT] 6.8.0[OCCT] 6.8.0 
Not needed
0025128: Memory leak in BOPDS_DS::Paves()
Memory leak has been detected in BOPDS_DS::Paves().

The issue comes from OCC Forum: http://www.opencascade.org/org/forum/thread_26382/?forum=3 [^]
class BOPDS_DS
  method
    void BOPDS_DS::Paves(const Standard_Integer theEdge,
                         BOPDS_ListOfPave& theLP)
class BOPDS_PaveBlock
  method
    void BOPDS_PaveBlock::Update(BOPDS_ListOfPaveBlock& theLPB,
                                 const Standard_Boolean theFlag)

 Using NCollection_Array1 class to ensure proper allocation and deallocation of memory.
No tags attached.
Issue History
2014-08-06 08:59emvNew Issue
2014-08-06 08:59emvAssigned To => emv
2014-08-06 09:08gitNote Added: 0030592
2014-08-06 09:10emvNote Added: 0030593
2014-08-06 09:10emvAssigned Toemv => abv
2014-08-06 09:10emvStatusnew => resolved
2014-08-06 09:10emvAdditional Information Updatedbug_revision_view_page.php?rev_id=7807#r7807
2014-08-06 09:18emvDescription Updatedbug_revision_view_page.php?rev_id=7809#r7809
2014-08-06 09:46abvNote Added: 0030596
2014-08-06 09:46abvAssigned Toabv => emv
2014-08-06 09:46abvStatusresolved => assigned
2014-08-07 10:09gitNote Added: 0030612
2014-08-07 10:12emvNote Added: 0030613
2014-08-07 10:12emvAssigned Toemv => abv
2014-08-07 10:12emvStatusassigned => resolved
2014-08-07 10:12emvAdditional Information Updatedbug_revision_view_page.php?rev_id=7820#r7820
2014-08-07 11:36gitNote Added: 0030615
2014-08-07 11:46abvNote Added: 0030616
2014-08-07 11:46abvAssigned Toabv => bugmaster
2014-08-07 11:46abvStatusresolved => reviewed
2014-08-08 15:56apvAssigned Tobugmaster => apv
2014-08-08 16:10gitNote Added: 0030635
2014-08-12 13:40apvTest case number => Not needed
2014-08-12 13:40apvTarget Version => 6.8.0
2014-08-12 13:44apvNote Added: 0030677
2014-08-12 13:44apvAssigned Toapv => bugmaster
2014-08-12 13:44apvStatusreviewed => tested
2014-08-15 11:25bugmasterChangeset attached => occt master 7a76337e
2014-08-15 11:25bugmasterStatustested => verified
2014-08-15 11:25bugmasterResolutionopen => fixed
2014-08-18 11:07gitNote Added: 0030815
2014-11-11 12:45aivFixed in Version => 6.8.0
2014-11-11 12:57aivStatusverified => closed

Notes
(0030592)
git   
2014-08-06 09:08   
Branch CR25128 has been created by emv.

SHA-1: d7d13cb8462eb52ebdcec467a89a6756e05960c7


This branch includes the following new commits:

       new d7d13cb 0025128: Memory leak in BOPDS_DS::Paves()


Detailed log of new commits:

commit d7d13cb8462eb52ebdcec467a89a6756e05960c7
Author: emv
Date: Wed Aug 6 09:08:16 2014 +0400

    0025128: Memory leak in BOPDS_DS::Paves()
    
    class BOPDS_DS
    method
      void BOPDS_DS::Paves(const Standard_Integer theEdge,
                           BOPDS_ListOfPave& theLP)
    
    Free allocated memory.
(0030593)
emv   
2014-08-06 09:10   
Git branch is ready to be reviewed.
(0030596)
abv   
2014-08-06 09:46   
Please use NCollection_Array or other class to ensure that allocation / deallocation is organized properly and not done manually in C-style.
(0030612)
git   
2014-08-07 10:09   
Branch CR25128 has been updated forcibly by emv.

SHA-1: 1014b44a71bb9652ad62772bb508d55032961d57
(0030613)
emv   
2014-08-07 10:12   
Git branch is ready to be reviewed.
(0030615)
git   
2014-08-07 11:36   
Branch CR25128 has been updated forcibly by emv.

SHA-1: d373c569dd885c5cc3492f7293328b169a65723d
(0030616)
abv   
2014-08-07 11:46   
No remarks, please test
(0030635)
git   
2014-08-08 16:10   
Branch CR25128 has been updated forcibly by apv.

SHA-1: 23e974c34c35767eb7355e7606064be4fb897770
(0030677)
apv   
2014-08-12 13:44   
Dear BugMaster,

Branch CR25128 (and products from GIT master) was compiled on Linux and Windows platforms and tested.
SHA-1: 23e974c34c35767eb7355e7606064be4fb897770

Number of compiler warnings:
occt component:
   Linux: 15 (15 on master)
   Windows: 0 (0 on master)
products component :
   Linux: 11 (11 on master)
   Windows: 1 (1 on master)

Regressions/Differences:
Not detected

Testing cases:
Not needed

Testing on Linux:
Total MEMORY difference: 348727836 / 348628408
Total CPU difference: 43993.71000000004 / 43340.71000000007

Testing on Windows:
Total MEMORY difference: 239015780 / 239709728
Total CPU difference: 30140.171875 / 28459.046875
(0030815)
git   
2014-08-18 11:07   
Branch CR25128 has been deleted by inv.

SHA-1: 23e974c34c35767eb7355e7606064be4fb897770