[Lldb-commits] [PATCH] D71372: [lldb] Add additional validation on return address in 'thread step-out'

Thu Dec 12 13:55:09 PST 2019

jingham added a comment.

In D71372#1782512 <https://reviews.llvm.org/D71372#1782512>, @mossberg wrote:

> Screenshots with new error messages.
>
> F11034423: image.png <https://reviews.llvm.org/F11034423>
>
> F11034433: image.png <https://reviews.llvm.org/F11034433>
>
> (I artificially forced this branch to execute, still unable to make it execute at runtime)
>  F11034446: image.png <https://reviews.llvm.org/F11034446>
>
> I replaced my 0x22 example with 0x200000000, which is unmapped *and* outside the low 32 bit range. `GetLoadAddressPermissions` still seems to succeed.

That sounds like a bug then.

> F11034470: image.png <https://reviews.llvm.org/F11034470>
>  F11034475: image.png <https://reviews.llvm.org/F11034475>

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71372/new/

https://reviews.llvm.org/D71372