[Lldb-commits] [PATCH] D127016: [lldb] Prevent crash due to reading memory from page zero.

Chelsea Cassanova via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Jun 3 15:46:00 PDT 2022

cassanova created this revision.
cassanova added reviewers: JDevlieghere, mib.
cassanova added a project: LLDB.
Herald added a project: All.
cassanova requested review of this revision.
Herald added a subscriber: lldb-commits.

Adds a check to ensure that a process exists before attempting to get its ABI to prevent lldb from crash due to trying to read from page zero.

  rG LLVM Github Monorepo



Index: lldb/test/Shell/Driver/TestPageZeroRead.test
--- /dev/null
+++ lldb/test/Shell/Driver/TestPageZeroRead.test
@@ -0,0 +1,3 @@
+# Ensure that the read from memory command doesn't try and read from page zero.
+# RUN: %clang_host %p/Inputs/hello.c -g -o a.out
+# RUN: %lldb -b a.out -o 'x 0'
Index: lldb/source/Commands/CommandObjectMemory.cpp
--- lldb/source/Commands/CommandObjectMemory.cpp
+++ lldb/source/Commands/CommandObjectMemory.cpp
@@ -593,7 +593,10 @@
       return false;
-    ABISP abi = m_exe_ctx.GetProcessPtr()->GetABI();
+    ABISP abi;
+    if (Process * proc = m_exe_ctx.GetProcessPtr())
+      abi = proc->GetABI();
     if (abi)
       addr = abi->FixDataAddress(addr);

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D127016.434172.patch
Type: text/x-patch
Size: 842 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20220603/a9575f0e/attachment.bin>

More information about the lldb-commits mailing list