[ODFTOOLKIT-300] Memory Leak in ODF Simple API

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.5-incubating
• Fix Version/s: 0.6.2-incubating
• Component/s: simple api
• Environment: odfdom-java-0.8.7.jar; simple-odf-0.6.6.jar
• Assignee: Svante Schubert
• Reporter: Mathias Silbermann
• Created: 13/Jan/12 09:52
• Updated: 19/Feb/18 13:57
• Resolved: 05/Mar/17 12:07

Description

There is a memory leak in the ODF Simple API. I tried both, versions 0.6.6 and 0.6.5. It appears when running code like the examples on cookbook page

http://incubator.apache.org/odftoolkit/simple/document/cookbook/Manipulate%20TextSearch.html

In short, the call TextNavigation.nextSelection() leads to the leak. When you look down the method's call stack, you will find that items are added to the static variable "repository" of the static inner class "Selection.SelectionManager". The added items are never removed from the repository. One indication is that the method Selection.SelectionManager.unregisterItem() is never called.

The code works fine if text navigation is done with few documents. But when its run on a server thousands of times, it will fill the JVMs memory.