diff options
author | Ivan Penkov <ivanpe@chromium.org> | 2016-06-05 22:41:10 -0700 |
---|---|---|
committer | Ivan Penkov <ivanpe@chromium.org> | 2016-06-05 22:41:10 -0700 |
commit | 240ed57ee1ac6a87b91526b8331717d494801826 (patch) | |
tree | 93c085a3fd036fde3bee99c4496db1adae5d2cc6 /src/processor/microdump.cc | |
parent | Make the getting started section in the README better (diff) | |
download | breakpad-240ed57ee1ac6a87b91526b8331717d494801826.tar.xz |
Adding support for overlapping ranges to RangeMap.
When enabled, adding of a new range that overlaps with an existing one can be a successful operation. The range which ends at the higher address will be shrunk down by moving its start position to a higher address so that it does not overlap anymore.
This change is required to fix http://crbug/611824. The actual fix will come in a separate CL.
R=mmandlis@chromium.org
Review URL: https://codereview.chromium.org/2029953003 .
Diffstat (limited to 'src/processor/microdump.cc')
-rw-r--r-- | src/processor/microdump.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/processor/microdump.cc b/src/processor/microdump.cc index 45bc8784..9073fe10 100644 --- a/src/processor/microdump.cc +++ b/src/processor/microdump.cc @@ -104,7 +104,7 @@ namespace google_breakpad { void MicrodumpModules::Add(const CodeModule* module) { linked_ptr<const CodeModule> module_ptr(module); - if (!map_->StoreRange(module->base_address(), module->size(), module_ptr)) { + if (!map_.StoreRange(module->base_address(), module->size(), module_ptr)) { BPLOG(ERROR) << "Module " << module->code_file() << " could not be stored"; } |