Fixed
Pinned fields
Click on the next to a field label to start pinning.
Details
Assignee
Gregory Newman-SmithGregory Newman-SmithReporter
Tor ColvinTor ColvinStory Points
1Components
Sprint
NoneFix versions
Priority
MajorInstabug
Open Instabug
Details
Details
Assignee
Gregory Newman-Smith
Gregory Newman-SmithReporter
Tor Colvin
Tor ColvinStory Points
1
Components
Sprint
None
Fix versions
Priority
Instabug
Open Instabug
PagerDuty
PagerDuty
PagerDuty
Sentry
Sentry
Sentry
Zendesk Support
Zendesk Support
Zendesk Support
Created September 18, 2023 at 12:32 PM
Updated August 31, 2024 at 11:01 AM
Resolved September 19, 2023 at 11:35 AM
We don't have sufficient handling for a nil doc body retruned by shallow copy inside the function Mutable1xBody.
When we take a shallow copy of the document here https://github.com/couchbase/sync_gateway/blob/fe16b931b3923052ae6a8d4e6b400300ec86b6ab/db/revision_cache_interface.go#L160
If we get a nil body returned here we can get a panic here https://github.com/couchbase/sync_gateway/blob/fe16b931b3923052ae6a8d4e6b400300ec86b6ab/db/revision_cache_interface.go#L165 by trying to assign to a nil map. We should have a check to see if b in this code is nil and if so we should initialise an empty body so we can assign the rev and doc id without panicing.