SERVER-50891 alow opCtx's to be created without globalStorageEngines

o It's required that one be able to create opCtx's without
  globalStorageEngines. When the globalStorageEngine fails to get properly
  initialized, the process will shutdown. Shutdown tasks will then need to
  be able to create and use opCtx's

2 files changed, 10 insertions, 9 deletions

diff --git a/src/mongo/db/operation_context.h b/src/mongo/db/operation_context.h
index 57a1c12c584..cbd2d3e943b 100644
--- a/src/mongo/db/operation_context.h
+++ b/src/mongo/db/operation_context.h
@@ -94,6 +94,7 @@ public:
      * Interface for durability.  Caller DOES NOT own pointer.
      */
     RecoveryUnit* recoveryUnit() const {
+        invariant(_recoveryUnit);
         return _recoveryUnit.get();
     }
 
@@ -123,6 +124,7 @@ public:
      * Interface for locking.  Caller DOES NOT own pointer.
      */
     Locker* lockState() const {
+        invariant(_locker);
         return _locker.get();
     }
 
diff --git a/src/mongo/db/service_context_d.cpp b/src/mongo/db/service_context_d.cpp
index 7fbb189194d..8d70cff7667 100644
--- a/src/mongo/db/service_context_d.cpp
+++ b/src/mongo/db/service_context_d.cpp
@@ -268,17 +268,16 @@ std::unique_ptr<OperationContext> ServiceContextMongoD::_newOpCtx(Client* client
     invariant(&cc() == client);
     auto opCtx = stdx::make_unique<OperationContext>(client, opId);
 
-    auto globalStorageEngine = getGlobalStorageEngine();
-    invariant(globalStorageEngine);
+    if (auto globalStorageEngine = getGlobalStorageEngine()) {
+        if (globalStorageEngine->isMmapV1()) {
+            opCtx->setLockState(stdx::make_unique<MMAPV1LockerImpl>());
+        } else {
+            opCtx->setLockState(stdx::make_unique<DefaultLockerImpl>());
+        }
 
-    if (globalStorageEngine->isMmapV1()) {
-        opCtx->setLockState(stdx::make_unique<MMAPV1LockerImpl>());
-    } else {
-        opCtx->setLockState(stdx::make_unique<DefaultLockerImpl>());
+        opCtx->setRecoveryUnit(globalStorageEngine->newRecoveryUnit(),
+                               OperationContext::kNotInUnitOfWork);
     }
-
-    opCtx->setRecoveryUnit(globalStorageEngine->newRecoveryUnit(),
-                           OperationContext::kNotInUnitOfWork);
     return opCtx;
 }