#15 Memory leak in deindexer conditioner

closed-fixed
Conditioner (8)
5
2008-04-18
2008-04-17
Anonymous
No

There is a memory leak in the deindexer conditioner.
The main for loop of execute() has a local variable which is an STL set of pointers to VertexIndexes objects which are dynamically allocated but never released in the case where skininfo == NULL

This is easily fixed by replacing the following chunk of code around l.560 of deindexer_core.cpp :

if (skininfo==NULL) continue;

by:

if (skininfo==NULL)
{
for(sortedsetiter = sortedvertexset.begin();
sortedsetiter != sortedvertexset.end();
sortedsetiter++)
{
delete *sortedsetiter;
}
continue;
}

Discussion

  • Alexandre de Pereyra

    Logged In: YES
    user_id=818574
    Originator: NO

    Another one: MergeVertex doesn't delete the "losing" vertex.

     
  • Alexandre de Pereyra

    Logged In: YES
    user_id=818574
    Originator: NO

    Both leaks fixed in revision 80.

     
  • Alexandre de Pereyra

    • status: open --> open-fixed
     
  • Alexandre de Pereyra

    Logged In: YES
    user_id=818574
    Originator: NO

    Both leaks fixed in revision 80.

     
  • Alexandre de Pereyra

    • status: open-fixed --> closed-fixed
     
  • Alexandre de Pereyra

    Logged In: YES
    user_id=818574
    Originator: NO

    Closing...

     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks