Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CBL-6169: DataFile::documentKeys() is not thread-safe #2132

Merged
merged 1 commit into from
Sep 10, 2024
Merged

Conversation

jianminzhao
Copy link
Contributor

_documentKeys is initialized lazily without protection. We fix it by call_once.

_documentKeys is initialized lazily without protection. We fix it by call_once.
@cbl-bot
Copy link

cbl-bot commented Sep 6, 2024

Code Coverage Results:

Type Percentage
branches 65.98
functions 78.7
instantiations 33.16
lines 77.63
regions 73.25

@jianminzhao jianminzhao merged commit 02a5393 into master Sep 10, 2024
9 checks passed
@jianminzhao jianminzhao deleted the cbl-6169 branch September 10, 2024 17:24
jianminzhao added a commit that referenced this pull request Sep 17, 2024
CBL-5616: Update SG version in docker-compose.yml (#2121)
CBL-6239: Fixed c4doc_getRevisionBody in a 2.x db (#2136)
92fc5cc Make sync work across VV upgrades (#1913)
abe9759 Merge in tag 3.2.0
8009302 Statically link to libstdc++ (#2134)
CBL-6169: DataFile::documentKeys() is not thread-safe (#2132)
CBL-6189: Annotation of thread-safety, filling in the missing ones after the first pass. (#2129)
471e5ec Allow a Timer to destruct itself during its callback (#2124)
b090363 Fixed C4CollectionSpec hash and equality (#2122)
cba409e LogEncoder cleanup (#2130)
cf53ccd Rewritten query translator (#1966)
9f75fab Updated Fleece now that its PR is merged
d6c2091 Update Android NDK even more
47cc9ad Update Jenkins configuration to support C++20
1652943 C++20: use std::ranges, contains() methods, std::size()
f27f8dd Build with C++20
callumbirks pushed a commit that referenced this pull request Sep 25, 2024
Cherry-pick 02a5393 from CBL-6169 (#2132).

_documentKeys is initialized lazily without protection. We fix it by call_once.
callumbirks added a commit that referenced this pull request Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants