static FeeFilterRounder in net_processing needs a mutex (or documentation that concurrent access is not allowed) #19254

1. 
   MarcoFalke commented at 6:50 pm on June 11, 2020: member

   The class uses a fast random context. The context is documented to be not thread-safe.

   Currently, SendMessages is only called in a single thread, but in the future or in unit tests, this might change.

2. MarcoFalke added the label Bug on Jun 11, 2020
3. 
   sipa commented at 6:53 pm on June 11, 2020: member

   Thanks!
4. MarcoFalke added the label Brainstorming on Jun 11, 2020
5. MarcoFalke added the label P2P on Jun 11, 2020
6. 
   hebasto commented at 3:12 pm on June 13, 2020: member

   @sipa @MarcoFalke

   Mind looking into #19268?

7. MarcoFalke referenced this in commit 07c83ce039 on Jul 14, 2020
8. MarcoFalke closed this on Jul 14, 2020

   ---

9. sidhujag referenced this in commit 9081bb4469 on Jul 14, 2020
10. DrahtBot locked this on Feb 15, 2022

Contributors
MarcoFalke sipa hebasto

Labels
Brainstorming Bug P2P