felix-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Pierre De Rop (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (FELIX-4394) Race problems in DependencyManager Configuration Dependency
Date Sun, 01 Mar 2015 14:07:04 GMT

     [ https://issues.apache.org/jira/browse/FELIX-4394?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Pierre De Rop updated FELIX-4394:
---------------------------------
    Fix Version/s:     (was: dependencymanager-4.0.0)
                   org.apache.felix.dependencymanager-r1

> Race problems in DependencyManager Configuration Dependency
> -----------------------------------------------------------
>
>                 Key: FELIX-4394
>                 URL: https://issues.apache.org/jira/browse/FELIX-4394
>             Project: Felix
>          Issue Type: Bug
>          Components: Dependency Manager
>    Affects Versions: dependencymanager-3.1.0
>            Reporter: Pierre De Rop
>            Assignee: Pierre De Rop
>             Fix For: org.apache.felix.dependencymanager-r1
>
>
> There are some synchronization issues in the ConfigurationDependencyImpl:
> 1) in the stop(DependencyService service) method, if the current size of the "m_services"
list is greater than 1, then the 'service' method parameter to be removed is not removed from
the internal "m_services" list. 
> Also, if the list size is 1, then the service is removed from the list, but outside the
synchronized block.
> 2) in the updated(Dictionary settings) method: the service list (m_services) is iterated
outside a synchronized block.
> 3) the ensureActive method should be synchronized since it checks the m_services list
size.
> 4) I'm sometimes seeing the following WARN message logged from the ConfigurationDependencyImpl.updated
class:
>                     m_logger.log(Logger.LOG_ERROR, "Service " + ds + " with configuration
dependency " + this + " could not be instantiated.");



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

Mime
View raw message