diff options
| author | Tao Bai <michaelbai@chromium.org> | 2016-05-03 18:14:28 -0400 |
|---|---|---|
| committer | Mark Mentovai <mark@chromium.org> | 2016-05-03 18:14:28 -0400 |
| commit | 4f417c8c0ffceb6c2516c6ef00cd91ca5746d852 (patch) | |
| tree | 974f7d18f7e85b3730052a5e01a2aa3b4150265a /src/google_breakpad/processor | |
| parent | Add parentheses to silence clang warning (diff) | |
| download | breakpad-4f417c8c0ffceb6c2516c6ef00cd91ca5746d852.tar.xz | |
Write adjusted range back to module
In Android, the mmap could be overlapped by /dev/ashmem, we adjusted
the range in https://breakpad.appspot.com/9744002/, but adjusted
range isn't written back to module, this caused the corresponding
module be dropped in BasicCodeModules copy constructor.
This also fix a lot of 'unable to store module' warnings
when dumping Android's minidump.
BUG=606972
R=mark@chromium.org, wfh@chromium.org
Review URL: https://codereview.chromium.org/1939333002 .
Patch from Tao Bai <michaelbai@chromium.org>.
Diffstat (limited to 'src/google_breakpad/processor')
| -rw-r--r-- | src/google_breakpad/processor/minidump.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/src/google_breakpad/processor/minidump.h b/src/google_breakpad/processor/minidump.h index 2b5025e4..5ec2d148 100644 --- a/src/google_breakpad/processor/minidump.h +++ b/src/google_breakpad/processor/minidump.h @@ -382,6 +382,11 @@ class MinidumpModule : public MinidumpObject, const MDRawModule* module() const { return valid_ ? &module_ : NULL; } + // This method is intented to handle the case on Android where the module + // could overlap with ashmem, and is not supposed to be used in anywhere + // else. + void set_base_address_and_size(uint64_t base_address, uint64_t size); + // CodeModule implementation virtual uint64_t base_address() const { return valid_ ? module_.base_of_image : static_cast<uint64_t>(-1); |