diff options
author | huangs <huangs@chromium.org> | 2016-02-04 14:26:43 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-02-04 22:27:34 +0000 |
commit | 7ec152caaf55f64431dcb2c63cc052890c4a93f1 (patch) | |
tree | 3adafe7ee167fc3ae5d91aaab6ee92fb85b3f34b /url/origin.cc | |
parent | 3dfbb081c34eb485535b7f2cc1ea21a4d6b1ed76 (diff) | |
download | chromium_src-7ec152caaf55f64431dcb2c63cc052890c4a93f1.zip chromium_src-7ec152caaf55f64431dcb2c63cc052890c4a93f1.tar.gz chromium_src-7ec152caaf55f64431dcb2c63cc052890c4a93f1.tar.bz2 |
[Courgette] Fix AssemblyProgram parsing for ELF-ARM.
This fixes two problems:
(A) In DisassemblerElf32, RVAToFileOffset() used Program Segment Header,
but FileOffsetToRVA() used Section Header. For consistency, both
should use the same one. We choose the latter (for now).
(B) Even if a section has sh_type=SHT_PROGBITS, it can still have
sh_addr=0. Extracting Rel32 address from these sections would add
overlapping RVA chaos, and so should be avoided.
Also using elf-arm7 in a unit test. The test fails before the fix and
passes after.
BUG= 579206
Review URL: https://codereview.chromium.org/1658463002
Cr-Commit-Position: refs/heads/master@{#373639}
Diffstat (limited to 'url/origin.cc')
0 files changed, 0 insertions, 0 deletions