nUndoPos by cs_main
ClosedPublic
Actions

Authored by PiRK on Jan 24 2023, 07:22.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABCdcac626d18b0: Guard CBlockIndex::nStatus/nFile/nDataPos/nUndoPos by cs_main

Summary

CBlockIndex::nStatus may be racy, i.e. potentially accessed by multiple threads, see core#17161. A solution is to guard it by cs_main, along with fellow data members nFile, nDataPos and nUndoPos.

Co-authored-by: Vasil Dimov <vd@FreeBSD.org>

This concludes backport of core#22932 and core#24197
https://github.com/bitcoin/bitcoin/pull/22932/commits/6ea56827842b9b2bd730edc38f3a7b1f46f6247b
https://github.com/bitcoin/bitcoin/pull/24197/commits/20276ca5d124285bdd1bda4cd777ca186b378555

Depends on D13037

Test Plan

With clang and DEBUG:

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Branch

pr22932

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 21833
Build 43301: Build Diff	build-diff · lint-circular-dependencies · build-without-wallet · build-clang-tidy · build-debug · build-clang
Build 43300: arc lint + arc unit

Event Timeline

PiRK created this revision.Jan 24 2023, 07:22

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 24 2023, 07:22

PiRK requested review of this revision.Jan 24 2023, 07:22

teamcity edited the summary of this revision. (Show Details)Jan 24 2023, 07:23

Harbormaster completed remote builds in B21831: Diff 37656.Jan 24 2023, 08:18

Fabien requested changes to this revision.Jan 24 2023, 09:03

Fabien added a subscriber: Fabien.

Fabien added inline comments.

src/blockindex.h
88 ↗	(On Diff #37656)
src/test/blockindex_tests.cpp
55 ↗	(On Diff #37656)	Just lock once before the loops
276 ↗	(On Diff #37656)	dito
src/txdb.cpp
344 ↗	(On Diff #37656)	See https://github.com/bitcoin/bitcoin/pull/24197