You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@felix.apache.org by "Thomas Watson (JIRA)" <ji...@apache.org> on 2016/01/12 21:09:39 UTC

[jira] [Created] (FELIX-5166) WrappedResource does not handle namespace correctly

Thomas Watson created FELIX-5166:
------------------------------------

             Summary: WrappedResource does not handle namespace correctly
                 Key: FELIX-5166
                 URL: https://issues.apache.org/jira/browse/FELIX-5166
             Project: Felix
          Issue Type: Bug
          Components: Resolver
    Affects Versions: resolver-1.8.0
            Reporter: Thomas Watson


This is similar to FELIX-4727.  There are several issues with how namespace is handled in getCapabilities and getRequirements for the org.apache.felix.resolver.WrappedResource class

In getCapabilities the namespace is correctly used to ask the host and fragment resources for the named capabilities.  But then that list is saved into the m_cachedCapabilities field.  If you first ask for the 'foo' namespace it will save all the 'foo' capabilities to m_cachedCapabilities, then if you ask for the 'bar' capabilities it will simply return the m_cachedCapabilities from before with 'foo' capabilities.

In getRequirements the namespace is completely ignored.  I think there are several reasons to stop lazily crating these caches.  It leads to strange bugs like the ones above, but also there is no synchronization here and the var is not volatile so I think there could be some issues with multi-threading.  Now that the resolver is multi-threaded I think we should stop creating these lists of capabilities and requirements lazily.  Instead I think we should make them final and create them in the constructor. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)