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

        Jukka Zitting made changes -
        Fix Version/s 2.5 [ 12319280 ]
        Fix Version/s 2.6 [ 12319480 ]
        Alex Parvulescu made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        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.
        Alex Parvulescu made changes -
        Fix Version/s 2.6 [ 12319480 ]
        Alex Parvulescu made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 2.4.1 [ 12319842 ]
        Resolution Fixed [ 1 ]
        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.
        Alex Parvulescu made changes -
        Field Original Value New Value
        Attachment JCR-3227.patch [ 12513590 ]
        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
        Jukka Zitting created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development