Differential D10872

[net processing] Guard m_continuation_block with m_block_inv_mutex
ClosedPublic
Actions

Authored by Fabien on Jan 24 2022, 16:39.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Maniphest Tasks

Restricted Maniphest Task

Commits

rABC9d5d1a6e7432: [net processing] Guard m_continuation_block with m_block_inv_mutex

Summary

Completes backport of core#19829:
https://github.com/bitcoin/bitcoin/pull/19829/commits/3002b4af2b4fde63026f8f7c575452a5c989c662

Depends on D10871.

Ref T1696.

Test Plan

With Clang and debug:

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Jan 24 2022, 16:39

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 24 2022, 16:39

Fabien requested review of this revision.Jan 24 2022, 16:39

teamcity edited the summary of this revision. (Show Details)Jan 24 2022, 16:39

@bot build-tsan

teamcity awarded a token.Jan 24 2022, 16:42

PiRK accepted this revision.Jan 24 2022, 17:03

This revision is now accepted and ready to land.Jan 24 2022, 17:03

Harbormaster completed remote builds in B17980: Diff 31917.Jan 24 2022, 17:06

Closed by commit rABC9d5d1a6e7432: [net processing] Guard m_continuation_block with m_block_inv_mutex (authored by John Newbery <john@johnnewbery.com>, committed by Fabien). · Explain WhyJan 24 2022, 19:46

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC9d5d1a6e7432: [net processing] Guard m_continuation_block with m_block_inv_mutex.

Revision Contents
Changeset List

Path

Size

src/

net_processing.h

7 lines

net_processing.cpp

30 lines

Diff 31920

View Options

src/net_processing.h