Fixes bitcoin/bitcoin#19341.
Prevent data race for `pathHandlers` #25983
pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:220902-httpmutex changing 1 files +5 −1-
hebasto commented at 11:51 AM on September 2, 2022: member
-
Prevent data race for `pathHandlers` 4296dde287
-
jonatack commented at 12:37 PM on September 2, 2022: contributor
Concept and initial-look approach ACK
- ryanofsky approved
-
ryanofsky commented at 7:48 PM on September 6, 2022: contributor
Code review ACK 4296dde28757d88a7076847484669fb202b47bc8. This should protect the vector. It also seems to make the http_request_cb callback single threaded, but that seems ok, since it is just adding work queue items not actually processing requests.
- MarcoFalke added the label Needs backport (22.x) on Sep 7, 2022
- MarcoFalke added the label Needs backport (23.x) on Sep 7, 2022
- MarcoFalke added the label RPC/REST/ZMQ on Sep 7, 2022
- MarcoFalke merged this on Sep 7, 2022
- MarcoFalke closed this on Sep 7, 2022
- hebasto deleted the branch on Sep 7, 2022
- hebasto referenced this in commit 38d4601fe8 on Sep 7, 2022
-
- hebasto removed the label Needs backport (23.x) on Sep 7, 2022
- hebasto referenced this in commit 2c6c628ab9 on Sep 7, 2022
-
- hebasto removed the label Needs backport (22.x) on Sep 7, 2022
- sidhujag referenced this in commit d1f05de3c6 on Sep 7, 2022
- MarcoFalke referenced this in commit 6164618964 on Oct 24, 2022
- MarcoFalke referenced this in commit 40b69fb06e on Oct 24, 2022
- vertiond referenced this in commit 773a5ca963 on Nov 3, 2022
- psgreco referenced this in commit 3ef34c1e7c on Dec 7, 2022
- psgreco referenced this in commit 0837c1499e on Dec 7, 2022
- psgreco referenced this in commit 77c94f82d2 on Dec 7, 2022
- bitcoin locked this on Sep 7, 2023
