Fixed long standing double-free bug in the cache.

I guess we don't delete stuff very often.

1 files changed, 23 insertions, 1 deletions

diff --git a/src/tests/cachedel.cpp b/src/tests/cachedel.cpp
index 3fa3e86..f4cb1b5 100644
--- a/src/tests/cachedel.cpp
+++ b/src/tests/cachedel.cpp
@@ -21,6 +21,12 @@ public:
 
     virtual ~Something()
     {
+        //Bu::println("Deleting %1").arg( this->toString() );
+    }
+
+    void sayHi()
+    {
+        Bu::println("Hello %1").arg( toString() );
     }
 
     virtual Bu::Uuid getKey() const
@@ -39,7 +45,7 @@ public:
         changed();
     }
 
-    virtual Bu::String toString() const=0;
+    virtual Bu::String toString() const=0;// { return Bu::String("ERROR"); };
 
 private:
     Bu::Uuid uId;
@@ -60,6 +66,11 @@ public:
         iNumber( iNumber )
     {
     }
+
+    virtual ~SubSomethingA()
+    {
+        Bu::println("Deleting-A %1").arg( this->toString() );
+    }
     
     virtual Bu::String toString() const
     {
@@ -84,6 +95,10 @@ public:
         sString( sString )
     {
     }
+
+    virtual ~SubSomethingB()
+    {
+    }
     
     virtual Bu::String toString() const
     {
@@ -187,6 +202,11 @@ int main( int, char *[] )
     Bu::MemBuf mbStore;
     SomethingCache c( mbStore );
 
+    {
+        SubSomethingA a("Test", 1);
+        a.sayHi();
+    }
+
     SomethingPtr ptr;
     if( time(NULL)%2 )
         ptr = c.insert( new SubSomethingA("Hello", 55) ).cast<Something>();
@@ -211,6 +231,8 @@ int main( int, char *[] )
     c.erase( id );
     Bu::println("p2 %1: %2").arg( id ).arg( c.has( id ) );
 
+    Bu::println("Program listing over, leaving main scope.");
+
     return 0;
 }