[Lldb-commits] [lldb] [lldb-dap] Check the process state before thread or frame validation. (PR #179202)

[Jonas Devlieghere via lldb-commits]

================
@@ -189,6 +190,10 @@ static bool FillStackFrames(DAP &dap, lldb::SBThread &thread,
 
 llvm::Expected<protocol::StackTraceResponseBody>
 StackTraceRequestHandler::Run(const protocol::StackTraceArguments &args) const {
+  const lldb::StateType process_state = dap.target.GetProcess().GetState();
+  if (!lldb::SBDebugger::StateIsStoppedState(process_state))
+    return llvm::make_error<NotStoppedError>();
----------------
JDevlieghere wrote:

Would it be worthwhile to have a `DAP::ProcessIsStopped` to avoid the repetition?

https://github.com/llvm/llvm-project/pull/179202