[PATCH] D111840: [hwasan] Add default "/" prefix.

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 14 14:29:10 PDT 2021


eugenis created this revision.
eugenis added reviewers: vitalybuka, hctim.
eugenis requested review of this revision.
Herald added a project: Sanitizers.
Herald added a subscriber: Sanitizers.

Add a default "/" prefix to the symbol search path in the
symbolization script. Without this, the binary itself is not considered
a valid source of symbol info.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D111840

Files:
  compiler-rt/lib/hwasan/scripts/hwasan_symbolize


Index: compiler-rt/lib/hwasan/scripts/hwasan_symbolize
===================================================================
--- compiler-rt/lib/hwasan/scripts/hwasan_symbolize
+++ compiler-rt/lib/hwasan/scripts/hwasan_symbolize
@@ -195,7 +195,7 @@
 args = parser.parse_args()
 
 # Unstripped binaries location.
-binary_prefixes = args.symbols or []
+binary_prefixes = args.symbols or ['/']
 if not binary_prefixes:
   if 'ANDROID_PRODUCT_OUT' in os.environ:
     product_out = os.path.join(os.environ['ANDROID_PRODUCT_OUT'], 'symbols')


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D111840.379842.patch
Type: text/x-patch
Size: 535 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211014/cb4e2fc2/attachment.bin>


More information about the llvm-commits mailing list