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

Notification listeners are stored in sets potentially allowing loss of listeners

    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

      SMSNotificationManager temporarily stores the changeListeners in Set objects before iterating through them, however this means that depending on the hashCode/equals implementations it is possible that certain listeners get lost during this process.

      How to reproduce the issue

      Don't really have reproduction steps for this one.

      Expected behaviour

      Listeners are stored in collections that allow storage of non-unique elements.

      Current behaviour

      Listeners can be lost.

        Attachments

          Issue Links

            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: