~itachi-san/telegram-desktop/breakpad

Viewing all changes in revision 1820.

  • Committer: Joshua Peraza
  • Date: 2020-02-21 04:06:35 UTC
  • Revision ID: git-v1:5e1c53a4427d65fa5d7798aea3893dc64572717e
android: truncate lower of two overlapping modules

Previously, the processor truncated the upper of two
overlapping module ranges to compensate for incorrect
reporting of module ranges by the Breakpad client.
Crashpad correctly reports module load ranges, so
this truncation strategy is no longer necessary.

However, when partitioned libraries are used, the base
library may have a range which encompasses the other
partitions. When this is combined with the truncate
upper merge strategy, the base library's executable
segment is truncated causing symbolization failures.

This patch changes Android's merge strategy to truncate
the lower range (which is still the base library, but
this strategy truncates from the high end of the
library's range, instead of its base).

Bug: b/149845120
Change-Id: Ic75ecd3e919432690740eb21ebd4265fc0bbaa86
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2067952
Reviewed-by: Mark Mentovai <mark@chromium.org>

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: