Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.4
    • Fix Version/s: 2.4.1, 2.5
    • Component/s: jackrabbit-core
    • Labels:
      None

      Description

      The MultiIndex.resetVolatileIndex() method doesn't properly close the existing VolatileIndex instance before creating a new one. This can confuse the DynamicPooledExecutor reference count added in JCR-2836, leading to a background thread leak.

      1. JCR-3227.patch
        0.8 kB
        Alex Parvulescu

        Activity

        Hide
        Alex Parvulescu added a comment -

        ported to 2.4 in rev 1304334.

        Show
        Alex Parvulescu added a comment - ported to 2.4 in rev 1304334.
        Hide
        Alex Parvulescu added a comment -

        > The null assignment can be skipped since the next statement just overrides it.
        agreed

        fixed in rev: 1241461

        Show
        Alex Parvulescu added a comment - > The null assignment can be skipped since the next statement just overrides it. agreed fixed in rev: 1241461
        Hide
        Jukka Zitting added a comment -

        +1 looks good. The null assignment can be skipped since the next statement just overrides it.

        Show
        Jukka Zitting added a comment - +1 looks good. The null assignment can be skipped since the next statement just overrides it.
        Hide
        Alex Parvulescu added a comment -

        yup, that is indeed a bug

        the fix looks simple enough.
        attached proposed patch

        Show
        Alex Parvulescu added a comment - yup, that is indeed a bug the fix looks simple enough. attached proposed patch

          People

          • Assignee:
            Unassigned
            Reporter:
            Jukka Zitting
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development