BTM
  1. BTM
  2. BTM-38

NPE in IncrementalRecoverer

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.3.2
    • Fix Version/s: 1.3.3
    • Labels:
      None
    • Number of attachments :
      1

      Description

      From line 59 in Incrementalrecoverer:

      TransactionLogRecord tlog = (TransactionLogRecord) danglingRecords.get(gtrid);
      if (tlog != null)

      { if (log.isDebugEnabled()) log.debug("committing " + xid); success &= RecoveryHelper.commit(xarhs, xid); updateJournal(tlog.getGtrid(), uniqueName, Status.STATUS_COMMITTED); commitCount++; }

      else

      { if (log.isDebugEnabled()) log.debug("rolling back " + xid); success &= RecoveryHelper.rollback(xarhs, xid); updateJournal(tlog.getGtrid(), uniqueName, Status.STATUS_ROLLEDBACK); rollbackCount++; }

      If variable 'tlog' is returned as null, it will NullPointerException at line 69: updateJournal(tlog.getGtrid(), uniqueName, Status.STATUS_ROLLEDBACK);

        Activity

        Hide
        Morten Haraldsen added a comment -

        BTM stacktrace

        Show
        Morten Haraldsen added a comment - BTM stacktrace
        Hide
        Ludovic Orban added a comment -

        Fixed in trunk, thanks.

        Show
        Ludovic Orban added a comment - Fixed in trunk, thanks.
        Hide
        Morten Haraldsen added a comment -

        Thanks for the quick fix. Haven't tested it yet, but looks good.

        Show
        Morten Haraldsen added a comment - Thanks for the quick fix. Haven't tested it yet, but looks good.

          People

          • Assignee:
            Ludovic Orban
            Reporter:
            Morten Haraldsen
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: