Details
-
Bug
-
Resolution: Won't Fix
-
Trivial - P5
-
None
-
None
-
ALL
Description
This seems extremely minor, but I believe the checkForInterruptFail fail point (introduced in SERVER-11695) can incorrectly fail when chance is set to 0 and the random number also happens to be 0:
double next = static_cast<double>(std::abs(checkForInterruptPRNG.nextInt64()));
|
double upperBound =
|
std::numeric_limits<int64_t>::max() * failPointInfo["chance"].numberDouble();
|
if (next > upperBound) {
|
return false;
|
}
|
return true;
|
I would expect chance: 0 to never reach the return true statement.
Version:
419be507ff3ccafd1a5dcda9a40144ed6fb4901e
Attachments
Issue Links
- is related to
-
SERVER-11695 Add checkForInterruptFail fail point
-
- Closed
-