[PATCH] D56603: [HWASAN] Improve tag-mismatch diagnostics

Eugene Leviant via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 15 06:57:13 PST 2019


evgeny777 marked an inline comment as done.
evgeny777 added inline comments.


================
Comment at: lib/hwasan/hwasan_report.cc:408
+  // correct tag
+  uptr granule = untagged_addr;
+  for (; *tag_ptr == ptr_tag;
----------------
eugenis wrote:
> Reuse __hwasan_test_shadow.
This function looks a bit strange to me. For instance it can return -1 either in case of error or in case of a tag mismatch when address is equal to start of granule plus one. It would make much more sense if this function returned pointer instead of offset


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

https://reviews.llvm.org/D56603





More information about the llvm-commits mailing list