From 17958ef62cc5fe8208a965d50f41fc70c5fc116f Mon Sep 17 00:00:00 2001 From: Jake Ehrlich Date: Fri, 4 Oct 2019 13:29:26 -0700 Subject: 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 --- src/common/linux/dump_symbols_unittest.cc | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'src/common/linux/dump_symbols_unittest.cc') 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& 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(&header), "foo", + "Linux", vector(), options, &module)); @@ -130,6 +132,7 @@ TYPED_TEST(DumpSymbols, SimplePublic) { DumpOptions options(ALL_SYMBOL_DATA, true); EXPECT_TRUE(ReadSymbolDataInternal(this->elfdata, "foo", + "Linux", vector(), options, &module)); @@ -186,6 +189,7 @@ TYPED_TEST(DumpSymbols, SimpleBuildID) { DumpOptions options(ALL_SYMBOL_DATA, true); EXPECT_TRUE(ReadSymbolDataInternal(this->elfdata, "foo", + "Linux", vector(), options, &module)); -- cgit v1.2.1