SERVER-39303 Fix several useless Client* checks

The checks performed in functions that are never passed a nullptr and
would crash later if null was ever passed.  Removing the checks makes
coverity happy.

1 files changed, 2 insertions, 2 deletions

diff --git a/src/mongo/db/service_context.cpp b/src/mongo/db/service_context.cpp
index 4d9d218a200..ee26756a642 100644
--- a/src/mongo/db/service_context.cpp
+++ b/src/mongo/db/service_context.cpp
@@ -236,7 +236,7 @@ void ServiceContext::ClientDeleter::operator()(Client* client) const {
 
 ServiceContext::UniqueOperationContext ServiceContext::makeOperationContext(Client* client) {
     auto opCtx = std::make_unique<OperationContext>(client, _nextOpId.fetchAndAdd(1));
-    if (client && client->session()) {
+    if (client->session()) {
         _numCurrentOps.addAndFetch(1);
     }
 
@@ -264,7 +264,7 @@ void ServiceContext::OperationContextDeleter::operator()(OperationContext* opCtx
     opCtx->getBaton()->detach();
 
     auto client = opCtx->getClient();
-    if (client && client->session()) {
+    if (client->session()) {
         _numCurrentOps.subtractAndFetch(1);
     }
     auto service = client->getServiceContext();