[Lldb-commits] [lldb] r227153 - Fix the test to disassemble as if at address zero, not at an invalid address. The default SBAddress constructor sets the offset to 0xffffffffffffffff and the section to NULL.

Greg Clayton gclayton at apple.com
Mon Jan 26 16:22:36 PST 2015


Author: gclayton
Date: Mon Jan 26 18:22:36 2015
New Revision: 227153

URL: http://llvm.org/viewvc/llvm-project?rev=227153&view=rev
Log:
Fix the test to disassemble as if at address zero, not at an invalid address. The default SBAddress constructor sets the offset to 0xffffffffffffffff and the section to NULL.

This was causing problems on clang 602 branches that use MemoryObjects to as the container for opcode bytes instead of a plain array of bytes. So we were asking for 3 bytes to be disassembled at address 0xffffffffffffffff which would cause an unsigned overflow and cause the MemoryObject to refuse to read anymore bytes.


Modified:
    lldb/trunk/test/python_api/disassemble-raw-data/TestDisassembleRawData.py

Modified: lldb/trunk/test/python_api/disassemble-raw-data/TestDisassembleRawData.py
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/test/python_api/disassemble-raw-data/TestDisassembleRawData.py?rev=227153&r1=227152&r2=227153&view=diff
==============================================================================
--- lldb/trunk/test/python_api/disassemble-raw-data/TestDisassembleRawData.py (original)
+++ lldb/trunk/test/python_api/disassemble-raw-data/TestDisassembleRawData.py Mon Jan 26 18:22:36 2015
@@ -26,7 +26,7 @@ class DisassembleRawDataTestCase(TestBas
 
         raw_bytes = bytearray([0x48, 0x89, 0xe5])
 
-        insts = target.GetInstructions(lldb.SBAddress(), raw_bytes)
+        insts = target.GetInstructions(lldb.SBAddress(0, target), raw_bytes)
 
         inst = insts.GetInstructionAtIndex(0)
 





More information about the lldb-commits mailing list