AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

[tickprocessor] fix ASLR slide use and nm on Mac

Browse Source 
`libStart` already has ALSR slide added to it. Do not add it twice.

https: //codereview.chromium.org/2696903002/
Review-Url: https://codereview.chromium.org/2928083004
Cr-Commit-Position: refs/heads/master@{#46152}
This commit is contained in:
fedor 2017-06-22 21:23:26 -07:00 committed by Commit Bot
parent 4a48b6e527
commit a8273f7e09
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tools/tickprocessor.js
View File

@ -645,9 +645,11 @@ CppEntriesProvider.prototype.parseVmSymbols = function(
} else if (funcInfo === false) { } else if (funcInfo === false) {
break; break;
} }
funcInfo.start += libASLRSlide; if (funcInfo.start < libStart - libASLRSlide &&
if (funcInfo.start < libStart && funcInfo.start < libEnd - libStart) { funcInfo.start < libEnd - libStart) {
funcInfo.start += libStart; funcInfo.start += libStart;
} else {
funcInfo.start += libASLRSlide;
} }
if (funcInfo.size) { if (funcInfo.size) {
funcInfo.end = funcInfo.start + funcInfo.size; funcInfo.end = funcInfo.start + funcInfo.size;