[FELIX-5051] Memory leak in async delivery

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: eventadmin-1.4.2, eventadmin-1.4.4
• Fix Version/s: eventadmin-1.4.6
• Assignee: Carsten Ziegeler
• Reporter: Guillaume Nodet
• Created: 23/Sep/15 14:21
• Updated: 19/Dec/19 17:36
• Resolved: 08/Oct/15 14:59

Description

From debugging through the code, it seems that entries get added to the org.apache.felix.eventadmin.impl.tasks.AsyncDeliverTasks#m_running_threads (java.util.concurrent.ConcurrentHashMap) but not removed.

In org.apache.felix.eventadmin.impl.tasks.AsyncDeliverTasks.TaskExecuter#run() the call to remove is trying to remove the "object" rather than the key. This does not remove anything from the map.

  public void run()
        {
          ...
                        this.m_running_threads.remove(this);
           ...
            } while ( running );
        }