You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directmemory.apache.org by bp...@apache.org on 2012/01/06 13:44:17 UTC

svn commit: r1228159 - /incubator/directmemory/trunk/directmemory-cache/src/main/java/org/apache/directmemory/memory/OffHeapMemoryBufferImpl.java

Author: bperroud
Date: Fri Jan  6 12:44:17 2012
New Revision: 1228159

URL: http://svn.apache.org/viewvc?rev=1228159&view=rev
Log:
DIRECTMEMORY-54 : OffHeapMemoryBuffer.clear should set to free all pointers to avoid misusage

Modified:
    incubator/directmemory/trunk/directmemory-cache/src/main/java/org/apache/directmemory/memory/OffHeapMemoryBufferImpl.java

Modified: incubator/directmemory/trunk/directmemory-cache/src/main/java/org/apache/directmemory/memory/OffHeapMemoryBufferImpl.java
URL: http://svn.apache.org/viewvc/incubator/directmemory/trunk/directmemory-cache/src/main/java/org/apache/directmemory/memory/OffHeapMemoryBufferImpl.java?rev=1228159&r1=1228158&r2=1228159&view=diff
==============================================================================
--- incubator/directmemory/trunk/directmemory-cache/src/main/java/org/apache/directmemory/memory/OffHeapMemoryBufferImpl.java (original)
+++ incubator/directmemory/trunk/directmemory-cache/src/main/java/org/apache/directmemory/memory/OffHeapMemoryBufferImpl.java Fri Jan  6 12:44:17 2012
@@ -150,6 +150,10 @@ public class OffHeapMemoryBufferImpl
 
     public synchronized void clear()
     {
+        for (final Pointer pointer : pointers)
+        {
+            pointer.free = true;
+        }
         allocationErrors = 0;
         pointers.clear();
         createAndAddFirstPointer();