[Sanitizers] Revert commit related to SUMMARY report 98/163498/3
authorDenis Khalikov <d.khalikov@partner.samsung.com>
Mon, 11 Dec 2017 14:01:48 +0000 (17:01 +0300)
committerDongkyun Son <dongkyun.s@samsung.com>
Wed, 14 Feb 2018 07:21:44 +0000 (07:21 +0000)
The commit with Change-Id: Ia258f1e3906bb095ca0d33d3354dd9b1f640c89f
adds SanitizerToolName to the report buffer, but the same name
is adding later at the line 200, so as result we will get
SanitzerToolName twice at the same buffer, therefore this
patch breaks thread_leaks1.c test, because the test expects
that report includes SanitizerToolname only once.

Change-Id: Ia98811936e4845e12cdde7a7f1286058a0766ada

libsanitizer/sanitizer_common/sanitizer_common.cc

index 3b73782..51ba4a1 100644 (file)
@@ -210,7 +210,6 @@ void ReportErrorSummary(const char *error_type, const AddressInfo &info) {
   if (!common_flags()->print_summary)
     return;
   InternalScopedString buff(kMaxSummaryLength);
-  buff.append("%s ", SanitizerToolName);
   buff.append("%s ", error_type);
   RenderFrame(&buff, "%L %F", 0, info, common_flags()->symbolize_vs_style,
               common_flags()->strip_path_prefix);