[Lldb-commits] [PATCH] D71372: [lldb] Add additional validation on return address in 'thread step-out'
Mark Mossberg via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Wed Dec 11 14:19:25 PST 2019
mossberg added a comment.
Something I noticed while updating the patch was that the `GetLoadedAddressPermissions` call would succeed, even if passed an address that is obviously not mapped. In my test case I placed an int 0x22 where the return address would be, expecting the validation to fail at the `GetLoadAddressPermissions` call because 0x22 is not mapped. However, it only ended up failing when the permissions (which were empty) were checked for an execute bit. It seems to me like this might be another bug, but I'm not sure.
F11025433: image.png <https://reviews.llvm.org/F11025433>
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the lldb-commits