aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/test/unit_tests
diff options
context:
space:
mode:
authorEgor Tensin <Egor.Tensin@gmail.com>2020-09-14 16:10:41 +0300
committerEgor Tensin <Egor.Tensin@gmail.com>2020-09-14 16:31:06 +0300
commit965d285ed2dd71331107e82b1b919ce292cb841a (patch)
treef64a23501e907a80f1b36265c9083fd03c14732e /test/unit_tests
parentdisable incremental linking for the tests (diff)
downloadwinapi-debug-965d285ed2dd71331107e82b1b919ce292cb841a.tar.gz
winapi-debug-965d285ed2dd71331107e82b1b919ce292cb841a.zip
unit_tests: fix the tests
Sometimes, 0x0 would creep into a call stack for some reason.
Diffstat (limited to '')
-rw-r--r--test/unit_tests/call_stack.cpp19
1 files changed, 13 insertions, 6 deletions
diff --git a/test/unit_tests/call_stack.cpp b/test/unit_tests/call_stack.cpp
index 481ab6e..cc5b74e 100644
--- a/test/unit_tests/call_stack.cpp
+++ b/test/unit_tests/call_stack.cpp
@@ -3,6 +3,7 @@
#include <pdb/all.hpp>
#include <test_lib.hpp>
+#include <boost/optional.hpp>
#include <boost/test/unit_test.hpp>
#include <algorithm>
@@ -30,15 +31,21 @@ BOOST_AUTO_TEST_CASE(call_stack) {
}
// Second, resolve the symbols:
- std::vector<pdb::SymbolInfo> symbols;
+ std::vector<boost::optional<pdb::SymbolInfo>> symbols;
symbols.reserve(call_stack.length);
BOOST_TEST_MESSAGE("Resolved symbols:");
for (const auto& addr : call_stack) {
- symbols.emplace_back(dbghelp.resolve_symbol(addr));
- const auto& symbol = symbols.back();
- BOOST_TEST_MESSAGE('\t' << pdb::format_address(symbol.get_offline_address()) << ' '
- << symbol.get_name());
+ try {
+ auto symbol = dbghelp.resolve_symbol(addr);
+ BOOST_TEST_MESSAGE('\t' << pdb::format_address(symbol.get_offline_address()) << ' '
+ << symbol.get_name());
+ symbols.emplace_back(std::move(symbol));
+ } catch (const std::system_error& e) {
+ symbols.emplace_back(boost::none);
+ BOOST_TEST_MESSAGE('\t' << pdb::format_address(addr)
+ << " Couldn't resolve symbol: " << e.what());
+ }
}
{
@@ -47,7 +54,7 @@ BOOST_AUTO_TEST_CASE(call_stack) {
const auto check = [&](pdb::Address addr) {
for (const auto& symbol : symbols)
- if (symbol.get_offline_address() == addr)
+ if (symbol && symbol->get_offline_address() == addr)
return true;
return false;
};