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>


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71372



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to