Uploaded image for project: 'Commons Dbcp'
  1. Commons Dbcp
  2. DBCP-341

LocalXAConnectionFactory does not properly check if Xid is equal to currentXid when resuming

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.4
    • Fix Version/s: 1.5.1, 2.0
    • Labels:
      None

      Description

      The current implementation checks if Xid is not equal to currentXid using the != . That doesn't work in all cases resulting in XAException.
      I think that this check should be left to the Xid implementation.

      1. DBPC-341-patch.txt
        0.9 kB
        Ioannis Canellos

        Activity

        Hide
        markt Mark Thomas added a comment -

        Thanks for the report and the fix. Sorry it has taken so long to address this.

        I've fixed trunk for the 2.x series the 1.5.x branch. It will be included in the next release of each.

        If there are further 1.3.x and 1.4.x releases (currently being discussed on the dev mailing list) my expectation is that they will be generated from the 1.5.x branch so will pick up this fix too.

        Show
        markt Mark Thomas added a comment - Thanks for the report and the fix. Sorry it has taken so long to address this. I've fixed trunk for the 2.x series the 1.5.x branch. It will be included in the next release of each. If there are further 1.3.x and 1.4.x releases (currently being discussed on the dev mailing list) my expectation is that they will be generated from the 1.5.x branch so will pick up this fix too.
        Hide
        ggregory@seagullsw.com ggregory@seagullsw.com added a comment -

        typo

        Show
        ggregory@seagullsw.com ggregory@seagullsw.com added a comment - typo
        Hide
        iocanel Ioannis Canellos added a comment -

        I am attaching a patch that replaces the != operation with a call the Xid equals method.

        Show
        iocanel Ioannis Canellos added a comment - I am attaching a patch that replaces the != operation with a call the Xid equals method.

          People

          • Assignee:
            Unassigned
            Reporter:
            iocanel Ioannis Canellos
          • Votes:
            1 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development