Skip to content

Commit

Permalink
libsanitizer: cherry-pick commit 8f5962b1ccb5fcd4d4544121d43efb860ac3…
Browse files Browse the repository at this point in the history
…cc6d from upstream

ASAN: keep support for Global::location

We as GCC still emit __asan_global_source_location for global variables
and we would like to use it in the future. On other hand, we don't
support llvm-symbolizer and the default libbacktraace symbolizer
does not support location info.
  • Loading branch information
marxin committed Feb 24, 2023
1 parent 35a0670 commit cab2636
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 3 deletions.
9 changes: 9 additions & 0 deletions libsanitizer/asan/asan_globals.cpp
Expand Up @@ -92,6 +92,10 @@ static void ReportGlobal(const Global &g, const char *prefix) {
if (info.line != 0) {
Report(" location: name=%s, %d\n", info.file, static_cast<int>(info.line));
}
else if (g.gcc_location != 0) {
// Fallback to Global::gcc_location
Report(" location: name=%s, %d\n", g.gcc_location->filename, g.gcc_location->line_no);
}
}

static u32 FindRegistrationSite(const Global *g) {
Expand Down Expand Up @@ -283,6 +287,11 @@ void PrintGlobalLocation(InternalScopedString *str, const __asan_global &g) {

if (info.line != 0) {
str->append("%s:%d", info.file, static_cast<int>(info.line));
} else if (g.gcc_location != 0) {
// Fallback to Global::gcc_location
str->append("%s", g.gcc_location->filename ? g.gcc_location->filename : g.module_name);
if (g.gcc_location->line_no) str->append(":%d", g.gcc_location->line_no);
if (g.gcc_location->column_no) str->append(":%d", g.gcc_location->column_no);
} else {
str->append("%s", g.module_name);
}
Expand Down
7 changes: 4 additions & 3 deletions libsanitizer/asan/asan_interface_internal.h
Expand Up @@ -53,9 +53,10 @@ extern "C" {
const char *module_name; // Module name as a C string. This pointer is a
// unique identifier of a module.
uptr has_dynamic_init; // Non-zero if the global has dynamic initializer.
uptr windows_padding; // TODO: Figure out how to remove this padding
// that's simply here to make the MSVC incremental
// linker happy...
__asan_global_source_location *gcc_location; // Source location of a global,
// used by GCC compiler. LLVM uses
// llvm-symbolizer that relies
// on DWARF debugging info.
uptr odr_indicator; // The address of the ODR indicator symbol.
};

Expand Down

0 comments on commit cab2636

Please sign in to comment.