Page MenuHomePhabricator

Merge #11618: rpc: Lock cs_main in blockToJSON/blockheaderToJSON

Authored by jasonbcox on Mar 12 2019, 22:50.



a9b6ba0b7 Add missing cs_main locks when calling blockToJSON/blockheaderToJSON (practicalswift)

Pull request description:

`blockToJSON(...)` and `blockheaderToJSON(...)` read the variable `chainActive` which requires holding the mutex `cs_main`. So does `GetDifficulty(...)`.

Tree-SHA512: bfb94f5e3238accbf6a4daddde49d53f1891c38ae9b07e25b3098c485747159258f64bb66a50e147b32beac601de89d9d04ff717b6c4f1460d329c90a53d3333

Backport of Core PR 11618

Test Plan

make check
bitcoin-cli getblock 000000000933ea01ad0ee984209779baaec3ced90fa3f408719526f8d77f4943

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

jasonbcox created this revision.Mar 12 2019, 22:50
Herald added a reviewer: Restricted Project. · View Herald TranscriptMar 12 2019, 22:50
Herald added a subscriber: schancel. · View Herald Transcript
Fabien accepted this revision.Mar 13 2019, 10:35
This revision is now accepted and ready to land.Mar 13 2019, 10:35
This revision was automatically updated to reflect the committed changes.