https://github.com/chelcassanova created https://github.com/llvm/llvm-project/pull/96560
Reverting to unblock macOS buildbots which are currently failing on these tests. https://green.lab.llvm.org/job/llvm.org/view/LLDB/job/as-lldb-cmake/6377/ >From 22127178761968b01be207a1c83c7048dc3ec47d Mon Sep 17 00:00:00 2001 From: Chelsea Cassanova <chelsea_cassan...@apple.com> Date: Mon, 24 Jun 2024 14:17:09 -0700 Subject: [PATCH 1/2] Revert "[lldb] Fix failing TestFind(Ranges)InMemory.py tests. (#96511)" This reverts commit 33a9c57b89c3ea901a057c3fcc9c9160eaf5a625. Alongside 10bd5ad this is being reverted as these commits are blocking the macOS buildbots: https://green.lab.llvm.org/job/llvm.org/view/LLDB/job/as-lldb-cmake/6375/ --- lldb/test/API/python_api/find_in_memory/address_ranges_helper.py | 1 + 1 file changed, 1 insertion(+) diff --git a/lldb/test/API/python_api/find_in_memory/address_ranges_helper.py b/lldb/test/API/python_api/find_in_memory/address_ranges_helper.py index 2c11fba80766f..0544100f97b29 100644 --- a/lldb/test/API/python_api/find_in_memory/address_ranges_helper.py +++ b/lldb/test/API/python_api/find_in_memory/address_ranges_helper.py @@ -35,6 +35,7 @@ def GetRangeFromAddrValue(test_base, addr): ) test_base.assertTrue(region.IsReadable()) + test_base.assertFalse(region.IsExecutable()) address_start = lldb.SBAddress(region.GetRegionBase(), test_base.target) stack_size = region.GetRegionEnd() - region.GetRegionBase() >From 10b926cb193292252bc8df6d0ed8345dc73c30e5 Mon Sep 17 00:00:00 2001 From: Chelsea Cassanova <chelsea_cassan...@apple.com> Date: Mon, 24 Jun 2024 14:18:51 -0700 Subject: [PATCH 2/2] Revert "[lldb][API] Add Find(Ranges)InMemory() to Process SB API (#95007)" This reverts commit 10bd5ad0a133fe73ffc1b05e63bc3fb2d56ba79c. Alongside 33a9c57 this commit is being reverted to unblock the macOS buildbots. --- lldb/bindings/python/python-typemaps.swig | 3 +- lldb/include/lldb/API/SBProcess.h | 10 - lldb/include/lldb/Core/AddressRangeListImpl.h | 4 - lldb/include/lldb/Target/Process.h | 14 -- lldb/source/API/SBProcess.cpp | 58 +---- lldb/source/Target/Process.cpp | 123 ---------- .../API/python_api/find_in_memory/Makefile | 3 - .../find_in_memory/TestFindInMemory.py | 131 ----------- .../find_in_memory/TestFindRangesInMemory.py | 221 ------------------ .../find_in_memory/address_ranges_helper.py | 73 ------ .../API/python_api/find_in_memory/main.cpp | 27 --- 11 files changed, 6 insertions(+), 661 deletions(-) delete mode 100644 lldb/test/API/python_api/find_in_memory/Makefile delete mode 100644 lldb/test/API/python_api/find_in_memory/TestFindInMemory.py delete mode 100644 lldb/test/API/python_api/find_in_memory/TestFindRangesInMemory.py delete mode 100644 lldb/test/API/python_api/find_in_memory/address_ranges_helper.py delete mode 100644 lldb/test/API/python_api/find_in_memory/main.cpp diff --git a/lldb/bindings/python/python-typemaps.swig b/lldb/bindings/python/python-typemaps.swig index f8c33e15c03e6..c39594c7df041 100644 --- a/lldb/bindings/python/python-typemaps.swig +++ b/lldb/bindings/python/python-typemaps.swig @@ -257,8 +257,7 @@ AND call SWIG_fail at the same time, because it will result in a double free. } // For SBProcess::WriteMemory, SBTarget::GetInstructions and SBDebugger::DispatchInput. %typemap(in) (const void *buf, size_t size), - (const void *data, size_t data_len), - (const void *buf, uint64_t size) { + (const void *data, size_t data_len) { if (PythonString::Check($input)) { PythonString str(PyRefType::Borrowed, $input); $1 = (void *)str.GetString().data(); diff --git a/lldb/include/lldb/API/SBProcess.h b/lldb/include/lldb/API/SBProcess.h index a6ab7ae759918..f1b5d1fb92ce2 100644 --- a/lldb/include/lldb/API/SBProcess.h +++ b/lldb/include/lldb/API/SBProcess.h @@ -209,16 +209,6 @@ class LLDB_API SBProcess { lldb::addr_t ReadPointerFromMemory(addr_t addr, lldb::SBError &error); - lldb::SBAddressRangeList FindRangesInMemory(const void *buf, uint64_t size, - const SBAddressRangeList &ranges, - uint32_t alignment, - uint32_t max_matches, - SBError &error); - - lldb::addr_t FindInMemory(const void *buf, uint64_t size, - const SBAddressRange &range, uint32_t alignment, - SBError &error); - // Events static lldb::StateType GetStateFromEvent(const lldb::SBEvent &event); diff --git a/lldb/include/lldb/Core/AddressRangeListImpl.h b/lldb/include/lldb/Core/AddressRangeListImpl.h index 6742e6ead87de..46ebfe73d4d92 100644 --- a/lldb/include/lldb/Core/AddressRangeListImpl.h +++ b/lldb/include/lldb/Core/AddressRangeListImpl.h @@ -13,9 +13,7 @@ #include <cstddef> namespace lldb { -class SBAddressRangeList; class SBBlock; -class SBProcess; } namespace lldb_private { @@ -41,9 +39,7 @@ class AddressRangeListImpl { lldb_private::AddressRange GetAddressRangeAtIndex(size_t index); private: - friend class lldb::SBAddressRangeList; friend class lldb::SBBlock; - friend class lldb::SBProcess; AddressRanges &ref(); diff --git a/lldb/include/lldb/Target/Process.h b/lldb/include/lldb/Target/Process.h index ceaf547ebddaf..eec337c15f7ed 100644 --- a/lldb/include/lldb/Target/Process.h +++ b/lldb/include/lldb/Target/Process.h @@ -2685,15 +2685,6 @@ void PruneThreadPlans(); lldb::addr_t FindInMemory(lldb::addr_t low, lldb::addr_t high, const uint8_t *buf, size_t size); - AddressRanges FindRangesInMemory(const uint8_t *buf, uint64_t size, - const AddressRanges &ranges, - size_t alignment, size_t max_matches, - Status &error); - - lldb::addr_t FindInMemory(const uint8_t *buf, uint64_t size, - const AddressRange &range, size_t alignment, - Status &error); - protected: friend class Trace; @@ -2809,11 +2800,6 @@ void PruneThreadPlans(); virtual size_t DoReadMemory(lldb::addr_t vm_addr, void *buf, size_t size, Status &error) = 0; - virtual void DoFindInMemory(lldb::addr_t start_addr, lldb::addr_t end_addr, - const uint8_t *buf, size_t size, - AddressRanges &matches, size_t alignment, - size_t max_matches); - /// DoGetMemoryRegionInfo is called by GetMemoryRegionInfo after it has /// removed non address bits from load_addr. Override this method in /// subclasses of Process. diff --git a/lldb/source/API/SBProcess.cpp b/lldb/source/API/SBProcess.cpp index efb3c8def2796..c37c111c5a58e 100644 --- a/lldb/source/API/SBProcess.cpp +++ b/lldb/source/API/SBProcess.cpp @@ -14,7 +14,6 @@ #include "lldb/lldb-defines.h" #include "lldb/lldb-types.h" -#include "lldb/Core/AddressRangeListImpl.h" #include "lldb/Core/Debugger.h" #include "lldb/Core/Module.h" #include "lldb/Core/PluginManager.h" @@ -27,7 +26,6 @@ #include "lldb/Target/Target.h" #include "lldb/Target/Thread.h" #include "lldb/Utility/Args.h" -#include "lldb/Utility/LLDBLog.h" #include "lldb/Utility/ProcessInfo.h" #include "lldb/Utility/State.h" #include "lldb/Utility/Stream.h" @@ -322,8 +320,8 @@ void SBProcess::ReportEventState(const SBEvent &event, FileSP out) const { if (process_sp) { StreamFile stream(out); const StateType event_state = SBProcess::GetStateFromEvent(event); - stream.Printf("Process %" PRIu64 " %s\n", process_sp->GetID(), - SBDebugger::StateAsCString(event_state)); + stream.Printf("Process %" PRIu64 " %s\n", + process_sp->GetID(), SBDebugger::StateAsCString(event_state)); } } @@ -380,6 +378,7 @@ bool SBProcess::SetSelectedThreadByIndexID(uint32_t index_id) { ret_val = process_sp->GetThreadList().SetSelectedThreadByIndexID(index_id); } + return ret_val; } @@ -547,6 +546,7 @@ ByteOrder SBProcess::GetByteOrder() const { if (process_sp) byteOrder = process_sp->GetTarget().GetArchitecture().GetByteOrder(); + return byteOrder; } @@ -558,6 +558,7 @@ uint32_t SBProcess::GetAddressByteSize() const { if (process_sp) size = process_sp->GetTarget().GetArchitecture().GetAddressByteSize(); + return size; } @@ -809,55 +810,6 @@ const char *SBProcess::GetBroadcasterClass() { return ConstString(Process::GetStaticBroadcasterClass()).AsCString(); } -lldb::SBAddressRangeList SBProcess::FindRangesInMemory( - const void *buf, uint64_t size, const SBAddressRangeList &ranges, - uint32_t alignment, uint32_t max_matches, SBError &error) { - LLDB_INSTRUMENT_VA(this, buf, size, ranges, alignment, max_matches, error); - - lldb::SBAddressRangeList matches; - - ProcessSP process_sp(GetSP()); - if (!process_sp) { - error.SetErrorString("SBProcess is invalid"); - return matches; - } - Process::StopLocker stop_locker; - if (!stop_locker.TryLock(&process_sp->GetRunLock())) { - error.SetErrorString("process is running"); - return matches; - } - std::lock_guard<std::recursive_mutex> guard( - process_sp->GetTarget().GetAPIMutex()); - matches.m_opaque_up->ref() = process_sp->FindRangesInMemory( - reinterpret_cast<const uint8_t *>(buf), size, ranges.ref().ref(), - alignment, max_matches, error.ref()); - return matches; -} - -lldb::addr_t SBProcess::FindInMemory(const void *buf, uint64_t size, - const SBAddressRange &range, - uint32_t alignment, SBError &error) { - LLDB_INSTRUMENT_VA(this, buf, size, range, alignment, error); - - ProcessSP process_sp(GetSP()); - - if (!process_sp) { - error.SetErrorString("SBProcess is invalid"); - return LLDB_INVALID_ADDRESS; - } - - Process::StopLocker stop_locker; - if (!stop_locker.TryLock(&process_sp->GetRunLock())) { - error.SetErrorString("process is running"); - return LLDB_INVALID_ADDRESS; - } - - std::lock_guard<std::recursive_mutex> guard( - process_sp->GetTarget().GetAPIMutex()); - return process_sp->FindInMemory(reinterpret_cast<const uint8_t *>(buf), size, - range.ref(), alignment, error.ref()); -} - size_t SBProcess::ReadMemory(addr_t addr, void *dst, size_t dst_len, SBError &sb_error) { LLDB_INSTRUMENT_VA(this, addr, dst, dst_len, sb_error); diff --git a/lldb/source/Target/Process.cpp b/lldb/source/Target/Process.cpp index 6fac0df1d7a66..9b905663a2c32 100644 --- a/lldb/source/Target/Process.cpp +++ b/lldb/source/Target/Process.cpp @@ -2007,129 +2007,6 @@ size_t Process::ReadMemory(addr_t addr, void *buf, size_t size, Status &error) { } } -void Process::DoFindInMemory(lldb::addr_t start_addr, lldb::addr_t end_addr, - const uint8_t *buf, size_t size, - AddressRanges &matches, size_t alignment, - size_t max_matches) { - // Inputs are already validated in FindInMemory() functions. - assert(buf != nullptr); - assert(size > 0); - assert(alignment > 0); - assert(max_matches > 0); - assert(start_addr != LLDB_INVALID_ADDRESS); - assert(end_addr != LLDB_INVALID_ADDRESS); - assert(start_addr < end_addr); - - lldb::addr_t start = llvm::alignTo(start_addr, alignment); - while (matches.size() < max_matches && (start + size) < end_addr) { - const lldb::addr_t found_addr = FindInMemory(start, end_addr, buf, size); - if (found_addr == LLDB_INVALID_ADDRESS) - break; - - if (found_addr % alignment) { - // We need to check the alignment because the FindInMemory uses a special - // algorithm to efficiently search mememory but doesn't support alignment. - start = llvm::alignTo(start + 1, alignment); - continue; - } - - matches.emplace_back(found_addr, size); - start = found_addr + alignment; - } -} - -AddressRanges Process::FindRangesInMemory(const uint8_t *buf, uint64_t size, - const AddressRanges &ranges, - size_t alignment, size_t max_matches, - Status &error) { - AddressRanges matches; - if (buf == nullptr) { - error.SetErrorString("buffer is null"); - return matches; - } - if (size == 0) { - error.SetErrorString("buffer size is zero"); - return matches; - } - if (ranges.empty()) { - error.SetErrorString("empty ranges"); - return matches; - } - if (alignment == 0) { - error.SetErrorString("alignment must be greater than zero"); - return matches; - } - if (max_matches == 0) { - error.SetErrorString("max_matches must be greater than zero"); - return matches; - } - - int resolved_ranges = 0; - Target &target = GetTarget(); - for (size_t i = 0; i < ranges.size(); ++i) { - if (matches.size() >= max_matches) - break; - const AddressRange &range = ranges[i]; - if (range.IsValid() == false) - continue; - - const lldb::addr_t start_addr = - range.GetBaseAddress().GetLoadAddress(&target); - if (start_addr == LLDB_INVALID_ADDRESS) - continue; - - ++resolved_ranges; - const lldb::addr_t end_addr = start_addr + range.GetByteSize(); - DoFindInMemory(start_addr, end_addr, buf, size, matches, alignment, - max_matches); - } - - if (resolved_ranges > 0) - error.Clear(); - else - error.SetErrorString("unable to resolve any ranges"); - - return matches; -} - -lldb::addr_t Process::FindInMemory(const uint8_t *buf, uint64_t size, - const AddressRange &range, size_t alignment, - Status &error) { - if (buf == nullptr) { - error.SetErrorString("buffer is null"); - return LLDB_INVALID_ADDRESS; - } - if (size == 0) { - error.SetErrorString("buffer size is zero"); - return LLDB_INVALID_ADDRESS; - } - if (!range.IsValid()) { - error.SetErrorString("range is invalid"); - return LLDB_INVALID_ADDRESS; - } - if (alignment == 0) { - error.SetErrorString("alignment must be greater than zero"); - return LLDB_INVALID_ADDRESS; - } - - Target &target = GetTarget(); - const lldb::addr_t start_addr = - range.GetBaseAddress().GetLoadAddress(&target); - if (start_addr == LLDB_INVALID_ADDRESS) { - error.SetErrorString("range load address is invalid"); - return LLDB_INVALID_ADDRESS; - } - const lldb::addr_t end_addr = start_addr + range.GetByteSize(); - - AddressRanges matches; - DoFindInMemory(start_addr, end_addr, buf, size, matches, alignment, 1); - if (matches.empty()) - return LLDB_INVALID_ADDRESS; - - error.Clear(); - return matches[0].GetBaseAddress().GetLoadAddress(&target); -} - size_t Process::ReadCStringFromMemory(addr_t addr, std::string &out_str, Status &error) { char buf[256]; diff --git a/lldb/test/API/python_api/find_in_memory/Makefile b/lldb/test/API/python_api/find_in_memory/Makefile deleted file mode 100644 index 99998b20bcb05..0000000000000 --- a/lldb/test/API/python_api/find_in_memory/Makefile +++ /dev/null @@ -1,3 +0,0 @@ -CXX_SOURCES := main.cpp - -include Makefile.rules diff --git a/lldb/test/API/python_api/find_in_memory/TestFindInMemory.py b/lldb/test/API/python_api/find_in_memory/TestFindInMemory.py deleted file mode 100644 index 4a459c47bcc02..0000000000000 --- a/lldb/test/API/python_api/find_in_memory/TestFindInMemory.py +++ /dev/null @@ -1,131 +0,0 @@ -""" -Test Process::FindInMemory. -""" - -import lldb -from lldbsuite.test.lldbtest import * -from lldbsuite.test import lldbutil -from address_ranges_helper import * - - -class FindInMemoryTestCase(TestBase): - NO_DEBUG_INFO_TESTCASE = True - - def setUp(self): - TestBase.setUp(self) - - self.build() - ( - self.target, - self.process, - self.thread, - self.bp, - ) = lldbutil.run_to_source_breakpoint( - self, "break here", lldb.SBFileSpec("main.cpp") - ) - self.assertTrue(self.bp.IsValid()) - - def test_find_in_memory_ok(self): - """Make sure a match exists in the heap memory and the right address ranges are provided""" - self.assertTrue(self.process, PROCESS_IS_VALID) - self.assertState(self.process.GetState(), lldb.eStateStopped, PROCESS_STOPPED) - error = lldb.SBError() - addr = self.process.FindInMemory( - SINGLE_INSTANCE_PATTERN_STACK, - GetStackRange(self), - 1, - error, - ) - - self.assertSuccess(error) - self.assertNotEqual(addr, lldb.LLDB_INVALID_ADDRESS) - - def test_find_in_memory_double_instance_ok(self): - """Make sure a match exists in the heap memory and the right address ranges are provided""" - self.assertTrue(self.process, PROCESS_IS_VALID) - self.assertState(self.process.GetState(), lldb.eStateStopped, PROCESS_STOPPED) - error = lldb.SBError() - addr = self.process.FindInMemory( - DOUBLE_INSTANCE_PATTERN_HEAP, - GetHeapRanges(self)[0], - 1, - error, - ) - - self.assertSuccess(error) - self.assertNotEqual(addr, lldb.LLDB_INVALID_ADDRESS) - - def test_find_in_memory_invalid_alignment(self): - """Make sure the alignment 0 is failing""" - self.assertTrue(self.process, PROCESS_IS_VALID) - self.assertState(self.process.GetState(), lldb.eStateStopped, PROCESS_STOPPED) - - error = lldb.SBError() - addr = self.process.FindInMemory( - SINGLE_INSTANCE_PATTERN_STACK, - GetStackRange(self), - 0, - error, - ) - - self.assertFailure(error) - self.assertEqual(addr, lldb.LLDB_INVALID_ADDRESS) - - def test_find_in_memory_invalid_address_range(self): - """Make sure invalid address range is failing""" - self.assertTrue(self.process, PROCESS_IS_VALID) - self.assertState(self.process.GetState(), lldb.eStateStopped, PROCESS_STOPPED) - - error = lldb.SBError() - addr = self.process.FindInMemory( - SINGLE_INSTANCE_PATTERN_STACK, - lldb.SBAddressRange(), - 1, - error, - ) - - self.assertFailure(error) - self.assertEqual(addr, lldb.LLDB_INVALID_ADDRESS) - - def test_find_in_memory_invalid_buffer(self): - """Make sure the empty buffer is failing""" - self.assertTrue(self.process, PROCESS_IS_VALID) - self.assertState(self.process.GetState(), lldb.eStateStopped, PROCESS_STOPPED) - - error = lldb.SBError() - addr = self.process.FindInMemory( - "", - GetStackRange(self), - 1, - error, - ) - - self.assertFailure(error) - self.assertEqual(addr, lldb.LLDB_INVALID_ADDRESS) - - def test_find_in_memory_unaligned(self): - """Make sure the unaligned match exists in the heap memory and is not found with alignment 8""" - self.assertTrue(self.process, PROCESS_IS_VALID) - self.assertState(self.process.GetState(), lldb.eStateStopped, PROCESS_STOPPED) - error = lldb.SBError() - range = GetAlignedRange(self) - - # First we make sure the pattern is found with alignment 1 - addr = self.process.FindInMemory( - UNALIGNED_INSTANCE_PATTERN_HEAP, - range, - 1, - error, - ) - self.assertSuccess(error) - self.assertNotEqual(addr, lldb.LLDB_INVALID_ADDRESS) - - # With alignment 8 the pattern should not be found - addr = self.process.FindInMemory( - UNALIGNED_INSTANCE_PATTERN_HEAP, - range, - 8, - error, - ) - self.assertSuccess(error) - self.assertEqual(addr, lldb.LLDB_INVALID_ADDRESS) diff --git a/lldb/test/API/python_api/find_in_memory/TestFindRangesInMemory.py b/lldb/test/API/python_api/find_in_memory/TestFindRangesInMemory.py deleted file mode 100644 index 31bc0e99f4914..0000000000000 --- a/lldb/test/API/python_api/find_in_memory/TestFindRangesInMemory.py +++ /dev/null @@ -1,221 +0,0 @@ -""" -Test Process::FindRangesInMemory. -""" - -import lldb -from lldbsuite.test.lldbtest import * -from lldbsuite.test import lldbutil -from address_ranges_helper import * - - -class FindRangesInMemoryTestCase(TestBase): - NO_DEBUG_INFO_TESTCASE = True - - def setUp(self): - TestBase.setUp(self) - - self.build() - ( - self.target, - self.process, - self.thread, - self.bp, - ) = lldbutil.run_to_source_breakpoint( - self, "break here", lldb.SBFileSpec("main.cpp") - ) - self.assertTrue(self.bp.IsValid()) - - def test_find_ranges_in_memory_two_matches(self): - """Make sure two matches exist in the heap memory and the right address ranges are provided""" - self.assertTrue(self.process, PROCESS_IS_VALID) - self.assertState(self.process.GetState(), lldb.eStateStopped, PROCESS_STOPPED) - - addr_ranges = GetHeapRanges(self) - error = lldb.SBError() - matches = self.process.FindRangesInMemory( - DOUBLE_INSTANCE_PATTERN_HEAP, - addr_ranges, - 1, - 10, - error, - ) - - self.assertSuccess(error) - self.assertEqual(matches.GetSize(), 2) - - def test_find_ranges_in_memory_one_match(self): - """Make sure exactly one match exists in the heap memory and the right address ranges are provided""" - self.assertTrue(self.process, PROCESS_IS_VALID) - self.assertState(self.process.GetState(), lldb.eStateStopped, PROCESS_STOPPED) - - addr_ranges = GetStackRanges(self) - error = lldb.SBError() - matches = self.process.FindRangesInMemory( - SINGLE_INSTANCE_PATTERN_STACK, - addr_ranges, - 1, - 10, - error, - ) - - self.assertSuccess(error) - self.assertEqual(matches.GetSize(), 1) - - def test_find_ranges_in_memory_one_match_multiple_ranges(self): - """Make sure exactly one match exists in the heap memory and multiple address ranges are provided""" - self.assertTrue(self.process, PROCESS_IS_VALID) - self.assertState(self.process.GetState(), lldb.eStateStopped, PROCESS_STOPPED) - - addr_ranges = GetRanges(self) - addr_ranges.Append(lldb.SBAddressRange()) - self.assertGreater(addr_ranges.GetSize(), 2) - error = lldb.SBError() - matches = self.process.FindRangesInMemory( - SINGLE_INSTANCE_PATTERN_STACK, - addr_ranges, - 1, - 10, - error, - ) - - self.assertSuccess(error) - self.assertEqual(matches.GetSize(), 1) - - def test_find_ranges_in_memory_one_match_max(self): - """Make sure at least one matche exists in the heap memory and the right address ranges are provided""" - self.assertTrue(self.process, PROCESS_IS_VALID) - self.assertState(self.process.GetState(), lldb.eStateStopped, PROCESS_STOPPED) - - addr_ranges = GetHeapRanges(self) - error = lldb.SBError() - matches = self.process.FindRangesInMemory( - DOUBLE_INSTANCE_PATTERN_HEAP, - addr_ranges, - 1, - 1, - error, - ) - - self.assertSuccess(error) - self.assertEqual(matches.GetSize(), 1) - - def test_find_ranges_in_memory_invalid_alignment(self): - """Make sure the alignment 0 is failing""" - self.assertTrue(self.process, PROCESS_IS_VALID) - self.assertState(self.process.GetState(), lldb.eStateStopped, PROCESS_STOPPED) - - addr_ranges = GetHeapRanges(self) - error = lldb.SBError() - matches = self.process.FindRangesInMemory( - DOUBLE_INSTANCE_PATTERN_HEAP, - addr_ranges, - 0, - 10, - error, - ) - - self.assertFailure(error) - self.assertEqual(matches.GetSize(), 0) - - def test_find_ranges_in_memory_invalid_range(self): - """Make sure the alignment 0 is failing""" - self.assertTrue(self.process, PROCESS_IS_VALID) - self.assertState(self.process.GetState(), lldb.eStateStopped, PROCESS_STOPPED) - - addr_ranges = lldb.SBAddressRangeList() - addr_ranges.Append(lldb.SBAddressRange()) - error = lldb.SBError() - matches = self.process.FindRangesInMemory( - DOUBLE_INSTANCE_PATTERN_HEAP, - addr_ranges, - 1, - 10, - error, - ) - - self.assertFailure(error) - self.assertIn("unable to resolve any ranges", str(error)) - self.assertEqual(matches.GetSize(), 0) - - def test_find_ranges_in_memory_empty_ranges(self): - """Make sure the empty ranges is failing""" - self.assertTrue(self.process, PROCESS_IS_VALID) - self.assertState(self.process.GetState(), lldb.eStateStopped, PROCESS_STOPPED) - - addr_ranges = lldb.SBAddressRangeList() - error = lldb.SBError() - matches = self.process.FindRangesInMemory( - DOUBLE_INSTANCE_PATTERN_HEAP, - addr_ranges, - 1, - 10, - error, - ) - - self.assertFailure(error) - self.assertEqual(matches.GetSize(), 0) - - def test_find_ranges_in_memory_invalid_buffer(self): - """Make sure the empty buffer is failing""" - self.assertTrue(self.process, PROCESS_IS_VALID) - self.assertState(self.process.GetState(), lldb.eStateStopped, PROCESS_STOPPED) - - addr_ranges = GetHeapRanges(self) - error = lldb.SBError() - matches = self.process.FindRangesInMemory( - "", - addr_ranges, - 1, - 10, - error, - ) - - self.assertFailure(error) - self.assertEqual(matches.GetSize(), 0) - - def test_find_ranges_in_memory_invalid_max_matches(self): - """Make sure the empty buffer is failing""" - self.assertTrue(self.process, PROCESS_IS_VALID) - self.assertState(self.process.GetState(), lldb.eStateStopped, PROCESS_STOPPED) - - addr_ranges = GetHeapRanges(self) - error = lldb.SBError() - matches = self.process.FindRangesInMemory( - DOUBLE_INSTANCE_PATTERN_HEAP, - addr_ranges, - 1, - 0, - error, - ) - - self.assertFailure(error) - self.assertEqual(matches.GetSize(), 0) - - def test_find_in_memory_unaligned(self): - """Make sure the unaligned match exists in the heap memory and is not found with alignment 8""" - self.assertTrue(self.process, PROCESS_IS_VALID) - self.assertState(self.process.GetState(), lldb.eStateStopped, PROCESS_STOPPED) - - addr_ranges = lldb.SBAddressRangeList() - addr_ranges.Append(GetAlignedRange(self)) - error = lldb.SBError() - - matches = self.process.FindRangesInMemory( - UNALIGNED_INSTANCE_PATTERN_HEAP, - addr_ranges, - 1, - 10, - error, - ) - self.assertSuccess(error) - self.assertEqual(matches.GetSize(), 1) - - matches = self.process.FindRangesInMemory( - UNALIGNED_INSTANCE_PATTERN_HEAP, - addr_ranges, - 8, - 10, - error, - ) - self.assertSuccess(error) - self.assertEqual(matches.GetSize(), 0) diff --git a/lldb/test/API/python_api/find_in_memory/address_ranges_helper.py b/lldb/test/API/python_api/find_in_memory/address_ranges_helper.py deleted file mode 100644 index 0544100f97b29..0000000000000 --- a/lldb/test/API/python_api/find_in_memory/address_ranges_helper.py +++ /dev/null @@ -1,73 +0,0 @@ -import lldb - -SINGLE_INSTANCE_PATTERN_STACK = "stack_there_is_only_one_of_me" -DOUBLE_INSTANCE_PATTERN_HEAP = "heap_there_is_exactly_two_of_me" -ALIGNED_INSTANCE_PATTERN_HEAP = "i_am_unaligned_string_on_the_heap" -UNALIGNED_INSTANCE_PATTERN_HEAP = ALIGNED_INSTANCE_PATTERN_HEAP[1:] - - -def GetAlignedRange(test_base): - frame = test_base.thread.GetSelectedFrame() - ex = frame.EvaluateExpression("aligned_string_ptr") - test_base.assertTrue(ex.IsValid()) - return GetRangeFromAddrValue(test_base, ex) - - -def GetStackRange(test_base): - frame = test_base.thread.GetSelectedFrame() - ex = frame.EvaluateExpression("stack_pointer") - test_base.assertTrue(ex.IsValid()) - return GetRangeFromAddrValue(test_base, ex) - - -def GetStackRanges(test_base): - addr_ranges = lldb.SBAddressRangeList() - addr_ranges.Append(GetStackRange(test_base)) - return addr_ranges - - -def GetRangeFromAddrValue(test_base, addr): - region = lldb.SBMemoryRegionInfo() - test_base.assertTrue( - test_base.process.GetMemoryRegionInfo( - addr.GetValueAsUnsigned(), region - ).Success(), - ) - - test_base.assertTrue(region.IsReadable()) - test_base.assertFalse(region.IsExecutable()) - - address_start = lldb.SBAddress(region.GetRegionBase(), test_base.target) - stack_size = region.GetRegionEnd() - region.GetRegionBase() - return lldb.SBAddressRange(address_start, stack_size) - - -def IsWithinRange(addr, range, target): - start_addr = range.GetBaseAddress().GetLoadAddress(target) - end_addr = start_addr + range.GetByteSize() - addr = addr.GetValueAsUnsigned() - return addr >= start_addr and addr < end_addr - - -def GetHeapRanges(test_base): - frame = test_base.thread.GetSelectedFrame() - - ex = frame.EvaluateExpression("heap_pointer1") - test_base.assertTrue(ex.IsValid()) - range = GetRangeFromAddrValue(test_base, ex) - addr_ranges = lldb.SBAddressRangeList() - addr_ranges.Append(range) - - ex = frame.EvaluateExpression("heap_pointer2") - test_base.assertTrue(ex.IsValid()) - if not IsWithinRange(ex, addr_ranges[0], test_base.target): - addr_ranges.Append(GetRangeFromAddrValue(test_base, ex)) - - return addr_ranges - - -def GetRanges(test_base): - ranges = GetHeapRanges(test_base) - ranges.Append(GetStackRanges(test_base)) - - return ranges diff --git a/lldb/test/API/python_api/find_in_memory/main.cpp b/lldb/test/API/python_api/find_in_memory/main.cpp deleted file mode 100644 index 98d378cb48b84..0000000000000 --- a/lldb/test/API/python_api/find_in_memory/main.cpp +++ /dev/null @@ -1,27 +0,0 @@ -#include <cstdlib> -#include <cstring> -#include <string> - -int main() { - // Stack - const char *stack_pointer = "stack_there_is_only_one_of_me"; - - // Heap - const std::string heap_string1("heap_there_is_exactly_two_of_me"); - const std::string heap_string2("heap_there_is_exactly_two_of_me"); - const char *heap_pointer1 = heap_string1.data(); - const char *heap_pointer2 = heap_string2.data(); - - // Aligned Heap - constexpr char aligned_string[] = "i_am_unaligned_string_on_the_heap"; - constexpr size_t len = sizeof(aligned_string) + 1; - // Allocate memory aligned to 8-byte boundary - void *aligned_string_ptr = aligned_alloc(8, len); - memcpy(aligned_string_ptr, aligned_string, len); - - (void)stack_pointer; - (void)heap_pointer1; - (void)heap_pointer2; - (void)aligned_string_ptr; // break here - return 0; -} _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
