[SERVER-56768] Race in atomic compareAndSwap can make FailPoint::enableFailPoint() to spin forever Created: 07/May/21 Updated: 06/Dec/22 |
|
| Status: | Open |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | 4.0.24 |
| Fix Version/s: | 4.0 Required |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Andrew Shuvalov (Inactive) | Assignee: | Backlog - Service Architecture |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Assigned Teams: |
Service Arch
|
| Operating System: | ALL |
| Participants: |
| Description |
|
This is not a production bug so threat it accordingly. The race: Even though this is not production failure, someone may copy-paste this pattern into production code. Atomics are subtle and must be treated with care. Fix: load _fpInfo each time inside loop. Same for enableFailPoint(). |