[OPENAM-12645] Non-threadsafe fields are missing volatile keyword Created: 18/Mar/18  Updated: 26/Apr/18  Resolved: 23/Mar/18

Status: Resolved
Project: OpenAM
Component/s: sms
Affects Version/s: 6.0.0
Fix Version/s: 6.0.0, 14.1.2, 5.5.2

Type: Bug Priority: Major
Reporter: Peter Major [X] (Inactive) Assignee: Peter Major [X] (Inactive)
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Target Version/s:
Needs backport:
Needs QA verification:
Functional tests:
Are the reproduction steps defined?:
Yes and I used the same an in the description


Bug description

As part of functional test fixes I've found several fields in SMSEmbeddedLdapObject, SMSLdapObject and IdRepoPluginsCache missing the volatile keyword, potentially allowing caching issues.

How to reproduce the issue

Don't really have reproduction steps for this one.

Expected behaviour

When a field can be modified/read by multiple threads the volatile keyword is set on them.

Current behaviour

volatile keyword is missing on several fields.

Generated at Tue Sep 22 12:16:19 UTC 2020 using Jira 7.13.12#713012-sha1:6e07c38070d5191bbf7353952ed38f111754533a.