Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(38)

Unified Diff: tools/tickprocessor.js

Issue 2928083004: [tickprocessor] fix ASLR slide use (Closed)
Patch Set: removed nm changes Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/tickprocessor.js
diff --git a/tools/tickprocessor.js b/tools/tickprocessor.js
index 91b5436eb5432fc3112c1a445d504435f10ad60d..de4e0296b59689a0221284613ea1c9195006ba99 100644
--- a/tools/tickprocessor.js
+++ b/tools/tickprocessor.js
@@ -645,9 +645,11 @@ CppEntriesProvider.prototype.parseVmSymbols = function(
} else if (funcInfo === false) {
break;
}
- funcInfo.start += libASLRSlide;
- if (funcInfo.start < libStart && funcInfo.start < libEnd - libStart) {
+ if (funcInfo.start < libStart - libASLRSlide &&
+ funcInfo.start < libEnd - libStart) {
funcInfo.start += libStart;
+ } else {
+ funcInfo.start += libASLRSlide;
}
if (funcInfo.size) {
funcInfo.end = funcInfo.start + funcInfo.size;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698