Uploaded image for project: 'OpenAM'
  1. OpenAM
  2. OPENAM-12645

Non-threadsafe fields are missing volatile keyword

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 6.0.0
    • Fix Version/s: 6.0.0, 14.1.2, 5.5.2
    • Component/s: sms
    • Labels:
      None
    • Needs backport:
      Yes
    • Needs QA verification:
      No
    • Functional tests:
      No
    • Are the reproduction steps defined?:
      Yes and I used the same an in the description

      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.

        Attachments

          Activity

            People

            • Assignee:
              peter.major Peter Major [X] (Inactive)
              Reporter:
              peter.major Peter Major [X] (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: