[CrystalSpace] #984: memory leak in occluvis.h (fix included)

#984: memory leak in occluvis.h (fix included)
---------------------------+------------------------------------------------
 Reporter:  ralphcampbell  |       Owner:  jorrit
     Type:  defect         |      Status:  new   
 Priority:  major          |   Milestone:        
Component:  renderer       |     Version:  V2.1  
 Keywords:                 |  
---------------------------+------------------------------------------------
 I was running valgrind with PlaneShift and found this.
 Other locations in occluvis.cpp are careful to delete the rmeshes array
 before reassigning the pointer but the destructor doesn't delete the
 array.
 This fixes the problem.

 {{{
 Index: include/csplugincommon/rendermanager/occluvis.h
 ===================================================================
 --- include/csplugincommon/rendermanager/occluvis.h     (revision 39444)
 +++ include/csplugincommon/rendermanager/occluvis.h     (working copy)
 @@ -193,6 +193,10 @@

          ~NodeMeshList ()
          {
 +          for (int m = 0; m < numMeshes; ++m)
 +          {
 +            delete[] meshList[m].rmeshes;
 +          }
            delete[] meshList;
          }

 }}}

-- 
Ticket URL: <http://www.crystalspace3d.org/trac/CS/ticket/984>
CrystalSpace <http://www.crystalspace3d.org/>
Free open-source 3D SDK.
------------------------------------------------------------------------------
This SF.net email is sponsored by Windows:

Build for Windows Store.

http://p.sf.net/sfu/windows-dev2dev