[#JRA-12600] CachedGenericConfigManager is not thread safe

JIRA

Created: 19/Apr/07 08:23 PM Updated: 09/Jan/08 09:22 PM

Component/s:

Backend / Domain Model

Affects Version/s:

None

Fix Version/s:

3.10.2

Time Tracking:

Original Estimate:

6 hours

Remaining Estimate:

6 hours

Time Spent:

Not Specified

Issue Links:

Duplicate

This issue is duplicated by:

~~JRA-14248~~ Null Pointer Exception while creating new issue under LM2Impact

Participants:	Dylan Etkin [Atlassian], Jed Wesley-Smith [Atlassian] and Nick Menere [Atlassian]
Since last comment:	1 year, 5 weeks, 3 days ago
Resolution Date:	30/Aug/07 09:51 PM
To be done by:	Single developer
Labels:

Description

« Hide

The CachedGenericConfigManager contains a non thread safe map.

It contains the code:

public void refreshCache()
    {
        cache = MultiKeyMap.decorate(new LRUMap(250));
        Collections.synchronizedMap(cache);
    }

But should be:

public void refreshCache()
    {
        cache = MultiKeyMap.decorate(new LRUMap(250));
        cache = Collections.synchronizedMap(cache);
    }

There could be a few more little issues as well, such as when refreshCache is called at the same time as retrieve...

All

Comments

Work Log

Change History

Sort Order:

[ Permlink | « Hide ]

Upgrading to a later version of common-collection may work as well.

[ Permlink | « Hide ]

This was fixed back on 5/25/07 the issue was never closed.