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

ServiceConfigImpl causes memory leak.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 11.0.0, 12.0.0, 13.0.0, 13.5.0
    • Fix Version/s: 13.5.1, 14.5.0, 14.1.2
    • Component/s: None
    • Labels:
    • Sprint:
      Sustaining Sprint 11, Sustaining Sprint 12, AM Sustaining Sprint 13, AM Sustaining Sprint 14, AM Sustaining Sprint 15, AM Sustaining Sprint 34, AM Sustaining Sprint 35, AM Sustaining Sprint 36, AM Sustaining Sprint 37
    • Story Points:
      5
    • Support Ticket IDs:

      Description

      When OpenAM creates ServiceConfigImpl as global configuration, it overwrite old value in cache.
      Then, some reference are remained.

      such as:

      • ServiceConfigImpl.smsEntry
      • ServiceConfigImpl.ssmi

      So GC does not collect them.

      Reproduction step 1:

      1) Access the admin console
      2) Click tab or link again and again

      Reproduction step 2 (only OpenAM 11):

      1) Configure OpenAM as OAuth2 Provider that use the global configuration
      2) Get an access token
      3) Access the tokeninfo endpoint again and again

      Reproduction step 3:
      Create a chain as per forgerock authenticator chain in the admin guide
      put al2.jsp in your webapps/openam directory, altering to match your oath chain
      ab -c 50 -n 1000000 http://openam.example.com:18080/openam/al2.jsp
      monitor memory usage and performance figures.

      Expected result
      Memory usage to grow under load and shrink afterwards. Performance to generally stay at the same level after multiple tests

      Actual result
      Massive memory growth, performance degradation.

        Attachments

        1. 13.5.0.png
          13.5.0.png
          16 kB
        2. 14.1.2.png
          14.1.2.png
          18 kB
        3. al2.jsp
          0.7 kB
        4. openam-11.0.0-tokeninfo_memleak.patch
          3 kB

          Issue Links

          relates to

          Bug - A problem which impairs or prevents the functions of the product. OPENAM-6642 Repeated requests to OAuth2 tokeninfo endpoint can end up creating large numbers of ServiceConfigImpl instances

          • Major - Major loss of function.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. OPENAM-7579 High performance regression for userinfo and refresh_token requests (Compared to 12.0.2)

          • Critical - Crashes, loss of data, severe memory leak.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. OPENAM-8023 OAuth2 load: contention at com.sun.identity.sm.ServiceConfigImpl.getInstance level (perf regression compared to AM12.0.2)

          • Major - Major loss of function.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. OPENAM-11154 Memory leak in SMSEventListenerManager#subNodeChanges

          • Major - Major loss of function.
          • Closed

            Activity

              People

              • Assignee:
                markdr Mark de Reeper
                Reporter:
                tsujiguchi tsujiguchi
              • Votes:
                2 Vote for this issue
                Watchers:
                16 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - 0h
                  0h
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 27h 47m
                  27h 47m