diff options
author | Greg Kaiser <gkaiser@google.com> | 2020-02-04 06:43:06 -0800 |
---|---|---|
committer | Greg Kaiser <gkaiser@google.com> | 2020-02-04 06:43:06 -0800 |
commit | d25a3436d6bc3abdb7417f9102bb952e05b126b6 (patch) | |
tree | 9c14114ed121d2c0ff8a9c6c5734223baeb7e0ca /memory_replay | |
parent | 1174bce393f6992df8aca73d654f1b6e52ca9f1a (diff) | |
download | extras-d25a3436d6bc3abdb7417f9102bb952e05b126b6.tar.gz |
memory_replay: Avoid use after erase()
We swap lines to avoid using the iterator after it has been
erase()d.
Test: TreeHugger
Change-Id: I2a889eb98806507d9a76c01142ad625c5a0e9c26
Diffstat (limited to 'memory_replay')
-rw-r--r-- | memory_replay/TraceBenchmark.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/memory_replay/TraceBenchmark.cpp b/memory_replay/TraceBenchmark.cpp index fc393f09..0a7dc90b 100644 --- a/memory_replay/TraceBenchmark.cpp +++ b/memory_replay/TraceBenchmark.cpp @@ -132,8 +132,8 @@ static void GetTraceData(const std::string& filename, TraceDataType* trace_data) errx(1, "File Error: Unable to find free pointer %" PRIx64, entry->ptr); } free_indices.push(idx_entry->second); - ptr_to_index.erase(idx_entry); entry->ptr = idx_entry->second + 1; + ptr_to_index.erase(idx_entry); } break; case THREAD_DONE: |