Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-43028

memOverheadChangedCallback heap-use-after-free during ~VBucket

    XMLWordPrintable

    Details

    • Triage:
      Triaged
    • Story Points:
      1
    • Is this a Regression?:
      Yes

      Description

      The memOverheadChangedCallback accesses the VBucket state.
      During VBucket destruction the callback is triggered when destroying the CheckpointManager as this in turn destroys Checkpoint s, reducing overhead. The state is no longer valid at this point.

      First seen in CV from merging patches introducing the memOverheadChangedCallback forward to master. Not previously identified on mad-hatter, possibly as a result of mad-hatter CV using an older clang.

      http://cv.jenkins.couchbase.com/job/kv_engine.threadsanitizer/job/master/7760/consoleFull#-130641249961882284-c5b1-40af-8076-4f8cb2d12fb1

        Attachments

          Issue Links

          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

            Activity

            Hide
            build-team Couchbase Build Team added a comment -

            Build couchbase-server-6.6.1-9210 contains kv_engine commit d169730 with commit message:
            MB-43028: [2/2] Make overhead tracking safe at VBucket destruction

            Show
            build-team Couchbase Build Team added a comment - Build couchbase-server-6.6.1-9210 contains kv_engine commit d169730 with commit message: MB-43028 : [2/2] Make overhead tracking safe at VBucket destruction
            Hide
            build-team Couchbase Build Team added a comment -

            Build couchbase-server-6.6.1-9210 contains kv_engine commit 6badce2 with commit message:
            MB-43028: [1/2] Make overhead tracking safe at VBucket destruction

            Show
            build-team Couchbase Build Team added a comment - Build couchbase-server-6.6.1-9210 contains kv_engine commit 6badce2 with commit message: MB-43028 : [1/2] Make overhead tracking safe at VBucket destruction
            Hide
            build-team Couchbase Build Team added a comment -

            Build couchbase-server-7.0.0-3943 contains kv_engine commit e96a9f5 with commit message:
            MB-43028: [1/2] Make overhead tracking safe at VBucket destruction

            Show
            build-team Couchbase Build Team added a comment - Build couchbase-server-7.0.0-3943 contains kv_engine commit e96a9f5 with commit message: MB-43028 : [1/2] Make overhead tracking safe at VBucket destruction
            Hide
            build-team Couchbase Build Team added a comment -

            Build couchbase-server-7.0.0-4084 contains kv_engine commit d169730 with commit message:
            MB-43028: [2/2] Make overhead tracking safe at VBucket destruction

            Show
            build-team Couchbase Build Team added a comment - Build couchbase-server-7.0.0-4084 contains kv_engine commit d169730 with commit message: MB-43028 : [2/2] Make overhead tracking safe at VBucket destruction
            Hide
            build-team Couchbase Build Team added a comment -

            Build couchbase-server-7.0.0-4084 contains kv_engine commit 6badce2 with commit message:
            MB-43028: [1/2] Make overhead tracking safe at VBucket destruction

            Show
            build-team Couchbase Build Team added a comment - Build couchbase-server-7.0.0-4084 contains kv_engine commit 6badce2 with commit message: MB-43028 : [1/2] Make overhead tracking safe at VBucket destruction

              People

              Assignee:
              owend Daniel Owen
              Reporter:
              james.harrison James Harrison
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  PagerDuty