Oh yea actually remove the thing

microsoft · Dec 18, 2020 · 5939636 · 5939636
1 parent f978a9c
commit 5939636
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/cascadia/Remoting/Monarch.cpp b/src/cascadia/Remoting/Monarch.cpp
@@ -106,14 +106,17 @@ namespace winrt::Microsoft::Terminal::Remoting::implementation
         {
             auto peasantSearch = _peasants.find(peasantID);
             auto maybeThePeasant = peasantSearch == _peasants.end() ? nullptr : peasantSearch->second;
-            maybeThePeasant.GetPID();
+            if (maybeThePeasant)
+            {
+                maybeThePeasant.GetPID();
+            }
             return maybeThePeasant;
         }
         catch (...)
         {
             LOG_CAUGHT_EXCEPTION();
             // TODO: Remove the peasant from the list of peasants
-
+            _peasants.erase(peasantID);
             return nullptr;
         }
     }