Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-8340

Failpoint for recv errors in sock.cpp is injected too late in code path.

    • Type: Icon: Bug Bug
    • Resolution: Done
    • Priority: Icon: Minor - P4 Minor - P4
    • 2.4.0-rc1
    • Affects Version/s: 2.3.2
    • Component/s: Internal Code
    • Labels:
    • Fully Compatible
    • ALL

      The socket library failpoint for 'recv' does not share a conditional expression with the evaluation of the return value of 'recv': instead, it is currently placed in the error handling function _handleRecvError. This is too late: if the failpoint is enabled, it will only be triggered if 'recv' actually fails, rather than making it appear as if 'recv' failed. The failpoint must be moved earlier, so that it can influence the apparent success or failure of recv.

            andrew.morrow@mongodb.com Andrew Morrow (Inactive)
            andrew.morrow@mongodb.com Andrew Morrow (Inactive)
            0 Vote for this issue
            1 Start watching this issue