[Lldb-commits] [lldb] [lldb] Tighten ABI assert in `StopInfoMachException::DeterminePtrauthFailure` (NFC) (PR #95015)

Greg Clayton via lldb-commits lldb-commits at lists.llvm.org
Mon Jun 10 22:29:35 PDT 2024


================
@@ -110,6 +108,9 @@ bool StopInfoMachException::DeterminePtrauthFailure(ExecutionContext &exe_ctx) {
     strm.Printf("Note: Possible pointer authentication failure detected.\n");
   };
 
+  ABISP abi_sp = process.GetABI();
+  assert(abi_sp && "Missing ABI info");
----------------
clayborg wrote:

Do we really want to crash the debugger if there is no abi plug-in? I would like us to not crash by reporting an error to the debug console and being able to let the debugger live and just check `abi_sp` before we use it.

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


More information about the lldb-commits mailing list