You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by ro...@apache.org on 2018/10/01 16:21:40 UTC

svn commit: r1842515 - /felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DependencyManager.java

Author: rotty3000
Date: Mon Oct  1 16:21:39 2018
New Revision: 1842515

URL: http://svn.apache.org/viewvc?rev=1842515&view=rev
Log:
FELIX-5947 ClassLoader memory leak

Signed-off-by: Raymond Auge <ro...@apache.org>

Modified:
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DependencyManager.java

Modified: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DependencyManager.java
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DependencyManager.java?rev=1842515&r1=1842514&r2=1842515&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DependencyManager.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/DependencyManager.java Mon Oct  1 16:21:39 2018
@@ -1657,7 +1657,9 @@ public class DependencyManager<S, T> imp
     {
         // only invoke the unbind method if there is an instance (might be null
         // in the delayed component situation) and the unbind method is declared.
-        boolean doUnbind = componentContext != null && m_dependencyMetadata.getUnbind() != null;
+        boolean doUnbind =
+            componentContext != null &&
+                (m_dependencyMetadata.getField() != null || m_dependencyMetadata.getUnbind() != null);
 
         AtomicInteger trackingCount = new AtomicInteger();
         Collection<RefPair<S, T>> refPairs;