Bug #29
Apply old memory leak patch
Start date:
Due date:
% Done:
0%
Estimated time:
Resolution:
fixed
Description
The Fedora folks found and a fixed a memory leak in Kowari 1.0.5. It
<br/>
looks like this patch is still needed. See also
<br/>
[<a href="http://prototypo.blogspot.com/2005/09/kowari-memory-leak-found-and-fixed.html">http://prototypo.blogspot.com/2005/09/kowari-memory-leak-found-and-fixed.html</a>]
<br/>
Here is the one-line patch against Mulgara svn head:
<br/>
<br/>
-----------------------------------------------------------------
<br/>
Index: src/jar/util-xa/java/org/mulgara/store/xa/FreeList.java
<br/>
===================================================================
<br/>
--- src/jar/util-xa/java/org/mulgara/store/xa/FreeList.java (revision <a href="http://mulgara.org/trac/changeset/106">106</a>)
<br/>
+++ src/jar/util-xa/java/org/mulgara/store/xa/FreeList.java (working copy)
<br/>
@@ -1070,6 +1070,7 @@
<br/>
public Phase() throws IOException {
<br/>
synchronized (FreeList.this) {
<br/>
if (currentPhase != null) {
<br/>
+ removeClosedPhases();
<br/>
sequenceNumber = currentPhase.sequenceNumber + 1;
<br/>
head = currentPhase.head;
<br/>
tail = currentPhase.tail;
<br/>
-----------------------------------------------------------------
<br/>
Updated by Paula Gearon over 17 years ago
Good catch.
<br/>
<br/>
I hadn't looked at this fix before, but this is indeed missing. Running tests now.