[PATCH] D34149: [ASAN] ASAN is not properly calling libbacktrace to symbolize program

Maxim Ostapenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 13 09:05:23 PDT 2017


m.ostapenko added a subscriber: llvm-commits.
m.ostapenko added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_symbolizer_libbacktrace.cc:113
   SymbolizeCodeCallbackArg *cdata = (SymbolizeCodeCallbackArg *)vdata;
+  if (function != NULL && cdata->frames_symbolized > 0)
+    return 0;
----------------
This doesn't look correct to me. Why do we bail out if function is not NULL? Shouldn't we use `(!function && cdata->frames_symbolized > 0)` istead? Also please include more context into the patch (git diff -U999999) or something like that.


https://reviews.llvm.org/D34149





More information about the llvm-commits mailing list