diff options
| author | Jake Ehrlich <jakehehrlich@google.com> | 2019-10-04 13:29:26 -0700 |
|---|---|---|
| committer | Mike Frysinger <vapier@chromium.org> | 2019-10-24 22:05:33 +0000 |
| commit | 17958ef62cc5fe8208a965d50f41fc70c5fc116f (patch) | |
| tree | d31cd13007f583b786030653977910c5eecaf6d1 /src/common/linux/dump_symbols_unittest.cc | |
| parent | linux, dump_syms: set module name from DT_SONAME (diff) | |
| download | breakpad-17958ef62cc5fe8208a965d50f41fc70c5fc116f.tar.xz | |
Add options to set OS and filename
This allows Fuchsia to use dump_syms directly without a
postprocessing step.
Change-Id: I84507f8bedddfcdcdb237119457c8ddf8ac354d5
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/1850718
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Diffstat (limited to 'src/common/linux/dump_symbols_unittest.cc')
| -rw-r--r-- | src/common/linux/dump_symbols_unittest.cc | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/common/linux/dump_symbols_unittest.cc b/src/common/linux/dump_symbols_unittest.cc index f011c8a7..54c21096 100644 --- a/src/common/linux/dump_symbols_unittest.cc +++ b/src/common/linux/dump_symbols_unittest.cc @@ -51,6 +51,7 @@ namespace google_breakpad { bool ReadSymbolDataInternal(const uint8_t* obj_file, const string& obj_filename, + const string& obj_os, const std::vector<string>& debug_dir, const DumpOptions& options, Module** module); @@ -94,6 +95,7 @@ TYPED_TEST(DumpSymbols, Invalid) { DumpOptions options(ALL_SYMBOL_DATA, true); EXPECT_FALSE(ReadSymbolDataInternal(reinterpret_cast<uint8_t*>(&header), "foo", + "Linux", vector<string>(), options, &module)); @@ -130,6 +132,7 @@ TYPED_TEST(DumpSymbols, SimplePublic) { DumpOptions options(ALL_SYMBOL_DATA, true); EXPECT_TRUE(ReadSymbolDataInternal(this->elfdata, "foo", + "Linux", vector<string>(), options, &module)); @@ -186,6 +189,7 @@ TYPED_TEST(DumpSymbols, SimpleBuildID) { DumpOptions options(ALL_SYMBOL_DATA, true); EXPECT_TRUE(ReadSymbolDataInternal(this->elfdata, "foo", + "Linux", vector<string>(), options, &module)); |