diff options
| author | Bernie Innocenti <codewiz@google.com> | 2020-12-19 21:17:16 +0900 |
|---|---|---|
| committer | Bernie Innocenti <codewiz@google.com> | 2020-12-19 21:17:16 +0900 |
| commit | 4760f26857551d86e660d08bc5c4a6d545cc9aba (patch) | |
| tree | 068a5dce6c6a9aa08b35a0955013095a6ae62855 /trusty/coverage/coverage.cpp | |
| parent | f7defa392fd01822538efcf1969486ec1c421f6c (diff) | |
Add explicit Result::has_value() checks where needed
Test: m checkbuild continuous_instrumentation_tests continuous_instrumentation_tests_api_coverage continuous_native_tests device-tests platform_tests
Change-Id: Ifd5dcda48e370d750e243d427e68a77dce333951
Diffstat (limited to 'trusty/coverage/coverage.cpp')
| -rw-r--r-- | trusty/coverage/coverage.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/trusty/coverage/coverage.cpp b/trusty/coverage/coverage.cpp index 9413ce3859..f383dd14ae 100644 --- a/trusty/coverage/coverage.cpp +++ b/trusty/coverage/coverage.cpp @@ -142,7 +142,7 @@ Result<void> CoverageRecord::Open() { void CoverageRecord::ResetFullRecord() { auto header_region = GetRegionBounds(COV_START); - if (!header_region) { + if (!header_region.ok()) { // If the header cannot be parsed, we can't reset the proper region yet. return; } @@ -202,7 +202,7 @@ void CoverageRecord::GetRawData(volatile void** begin, volatile void** end) { void CoverageRecord::GetRawCounts(volatile uint8_t** begin, volatile uint8_t** end) { auto region = GetRegionBounds(COV_8BIT_COUNTERS); - if (!region) { + if (!region.ok()) { *begin = 0; *end = 0; return; @@ -216,7 +216,7 @@ void CoverageRecord::GetRawCounts(volatile uint8_t** begin, volatile uint8_t** e void CoverageRecord::GetRawPCs(volatile uintptr_t** begin, volatile uintptr_t** end) { auto region = GetRegionBounds(COV_INSTR_PCS); - if (!region) { + if (!region.ok()) { *begin = 0; *end = 0; return; |
