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

fix temp buffer usage in entry cache

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Critical
    • 5.5.1
    • 5.5.0
    • secondary-index
    • None
    • Untriaged
    • Unknown

    Description

      fix temp buffer usage in entry cache. compkeybuf usage needs to be safer.

      It can lead to random crashes during scan.

      panic: runtime error: slice bounds out of range [recovered]
              panic: runtime error: slice bounds out of range
       
      goroutine 21116 [running]:
      panic(0xe3c3e0, 0xc4200160d0)
              /home/couchbase/.cbdepscache/exploded/x86_64/go-1.7.6/go/src/runtime/panic.go:500 +0x1a1 fp=0xc53bcdb0f0 sp=0xc53bcdb060
      github.com/couchbase/indexing/secondary/indexer.(*IndexScanSource).Routine.func1(0xc4dbdf6a10)
              goproj/src/github.com/couchbase/indexing/secondary/indexer/scan_pipeline.go:133 +0xcf fp=0xc53bcdb148 sp=0xc53bcdb0f0
      runtime.call32(0x0, 0x10676b8, 0xc4898f5330, 0x800000008)
              /home/couchbase/.cbdepscache/exploded/x86_64/go-1.7.6/go/src/runtime/asm_amd64.s:479 +0x4c fp=0xc53bcdb178 sp=0xc53bcdb148
      panic(0xe3c3e0, 0xc4200160d0)
              /home/couchbase/.cbdepscache/exploded/x86_64/go-1.7.6/go/src/runtime/panic.go:458 +0x243 fp=0xc53bcdb208 sp=0xc53bcdb178
      runtime.panicslice()
              /home/couchbase/.cbdepscache/exploded/x86_64/go-1.7.6/go/src/runtime/panic.go:34 +0x6d fp=0xc53bcdb238 sp=0xc53bcdb208
      github.com/couchbase/indexing/secondary/indexer.(*entryCache).Update(0xc4ff8b1b00, 0x7fd6c2327a4c, 0x53, 0x53, 0xc4a2d622a0, 0x4, 0x4, 0xc44f457f00, 0x4, 0x4)
              goproj/src/github.com/couchbase/indexing/secondary/indexer/scan_pipeline.go:1333 +0x1dd fp=0xc53bcdb2d8 sp=0xc53bcdb238
      

      Attachments

        Issue Links

          For Gerrit Dashboard: MB-30823
          # Subject Branch Project Status CR V

          Activity

            People

              deepkaran.salooja Deepkaran Salooja
              deepkaran.salooja Deepkaran Salooja
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty