[PATCH] D91765: [sanitizer_common] Add facility to get the full report path

[Vitaly Buka via Phabricator via llvm-commits]

vitalybuka accepted this revision.
vitalybuka added a comment.
This revision is now accepted and ready to land.

LGTM with /tmp/ fix



================
Comment at: compiler-rt/test/sanitizer_common/TestCases/Posix/sanitizer_set_report_path_test.cpp:11
+int main(int argc, char **argv) {
+  __sanitizer_set_report_path("/tmp/foobar");
+  printf("Path %s\n", __sanitizer_get_report_path());
----------------
it should not work on android

maybe this without FileCHeck?
sprintf(buff, "%s.report_path", arg[0]);
 __sanitizer_set_report_path(buff);
assert(strncmp(buff, __sanitizer_get_report_path(), strlen(buff)) == 0)



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91765