[jboss-jira] [JBoss JIRA] (ELY-956) Coverity static analysis: Synchronization on java.util.concurrent object in ElytronPolicyConfigurationFactory

[Ilia Vassilev (JIRA)]

    [ https://issues.jboss.org/browse/ELY-956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13374754#comment-13374754 ] 

Ilia Vassilev commented on ELY-956:
-----------------------------------

[~mchoma] ConcurrentHashMap is synchronized, but not locked for exclusive access. We need this code because the list may change. Can we close this issue? 

> Coverity static analysis: Synchronization on java.util.concurrent object in ElytronPolicyConfigurationFactory
> -------------------------------------------------------------------------------------------------------------
>
>                 Key: ELY-956
>                 URL: https://issues.jboss.org/browse/ELY-956
>             Project: WildFly Elytron
>          Issue Type: Bug
>    Affects Versions: 1.1.0.Beta24
>            Reporter: Martin Choma
>            Assignee: Ilia Vassilev
>            Priority: Minor
>
> Coverity static-analysis scan found a synchronization on java.util.concurrent.ConcurrentHashMap object in:
> *  {{ElytronPolicyConfigurationFactory.getPolicyConfiguration}}
> *  {{ElytronPolicyConfigurationFactory.inService}}
> If policyConfiguration synchronization is concern, it should be enough to synchronize on policyConfiguration object.
>  
> https://scan7.coverity.com/reports.htm#v23632/p11778/fileInstanceId=8490244&defectInstanceId=2123208&mergedDefectId=1377483
> https://scan7.coverity.com/reports.htm#v23632/p11778/fileInstanceId=8490244&defectInstanceId=2123209&mergedDefectId=1377485



--
This message was sent by Atlassian JIRA
(v7.2.3#72005)