Uploaded image for project: 'camunda BPM'
  1. camunda BPM
  2. CAM-14622

In LDAP plugin, #checkPassword leaves an orphaned LDAP context which causes a memory leak

XMLWordPrintable

    • Icon: Bug Report Bug Report
    • Resolution: Fixed
    • Icon: L3 - Default L3 - Default
    • 7.18.0, 7.18.0-alpha1
    • None
    • engine
    • None

      Environment (Required on creation):

      Any distribution, LDAP identity plugin enabled.

      Description (Required on creation; please attach any relevant screenshots, stacktraces, log files, etc. to the ticket):

      The checkPassword method leaves an orphaned LDAP context. If the resource is not closed properly, it can lead to memory leaks.

      Steps to reproduce (Required on creation):

      1. Enable the LDAP identity plugin in any distribution.
      2. Stress test user login (the checkPassword method), e.g.:
        1. Enable authentication for the REST API.
        2. Repeatedly call any method that requires authentication from a stress test tool.

      Observed Behavior (Required on creation):

      A memory leak.

      Expected behavior (Required on creation):

      No memory leak.

      Root Cause (Required on prioritization):

      Solution Ideas (Optional):

      Close the context in a finally block, see this PR

      Hints (optional):

        This is the controller panel for Smart Panels app

            [CAM-14622] In LDAP plugin, #checkPassword leaves an orphaned LDAP context which causes a memory leak

            Daniel Ewing created issue -
            Daniel Ewing made changes -
            Link New: This issue is related to SUPPORT-13556 [ SUPPORT-13556 ]
            Tassilo Weidner made changes -
            Fix Version/s New: 7.18.0 [ 17394 ]
            Tassilo Weidner made changes -
            Status Original: Open [ 1 ] New: In Progress [ 3 ]
            Tassilo Weidner made changes -
            Resolution New: Fixed [ 1 ]
            Status Original: In Progress [ 3 ] New: Resolved [ 5 ]
            Remaining Estimate New: 0 minutes [ 0 ]
            Original Estimate New: 0 minutes [ 0 ]
            Tassilo Weidner made changes -
            Summary Original: Failure to close LDAP context might cause a memory leak New: In LDAP plugin, #checkPassword leaves an orphaned LDAP context which causes a memory leak
            Tassilo Weidner made changes -
            DRI New: Tassilo Weidner [ tassilo.weidner ]
            Reviewer New: Tassilo Weidner [ tassilo.weidner ]
            Daniel Ewing made changes -
            Labels Original: SUPPORT
            Daniel Ewing made changes -
            Link Original: This issue is related to SUPPORT-13556 [ SUPPORT-13556 ]
            Tassilo Weidner made changes -
            Status Original: Resolved [ 5 ] New: Closed [ 6 ]

              tassilo.weidner Tassilo Weidner
              daniel.ewing Daniel Ewing
              Tassilo Weidner Tassilo Weidner
              Tassilo Weidner Tassilo Weidner
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

                Created:
                Updated:
                Resolved: