As SERVER-4740 explains, the server shouldn't use gettimeofday() for timing. However, the code currently does, and it doesn't check whether the call succeeded or not. If gettimeofday() returns -1 (not 0), then the call has failed and the values in the timeval struct shouldn't be used (they may not have even been initialised).