New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 916828 link

Starred by 3 users

Issue metadata

Status: Started
Owner:
Cc:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Task

Blocked on:
issue 917419



Sign in to add a comment

Create a build of the browser with HWASAN on Android

Project Member Reported by p...@chromium.org, Dec 20

Issue description

HWASAN is a hardware-assisted memory error detector. For details see: http://clang.llvm.org/docs/HardwareAssistedAddressSanitizerDesign.html

HWASAN has a much lower memory overhead than ASAN and does not require all allocations to be intercepted, which makes it more practical for deployment in apps on unmodified devices. However, it does require kernel support (patches have landed in the Pixel 2 kernel and are under review upstream).

The goal is to create a build of the browser with HWASAN that can run on a device with the kernel patches but with no other changes.
 
Blockedon: 917419
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 9

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/aa595e0ebb8c5ecd64e1598bf1359f0cf076b818

commit aa595e0ebb8c5ecd64e1598bf1359f0cf076b818
Author: Peter Collingbourne <pcc@chromium.org>
Date: Wed Jan 09 00:00:53 2019

Android: Compress clang runtimes only in APKs with embedded crazy linker.

If the crazy linker is embedded then the runtime needs to be compressed
so that it will be extracted alongside the crazy linker which will also
depend on the runtime. However, if the crazy linker is not embedded
(e.g. in Monochrome) then Android will require that the runtime is
uncompressed so that it can be loaded directly from the APK.

Bug: 916828
Change-Id: I88e3fc3297606b7efdc817fe9f895d0348ba5722
Reviewed-on: https://chromium-review.googlesource.com/c/1385478
Reviewed-by: agrieve <agrieve@chromium.org>
Commit-Queue: Peter Collingbourne <pcc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#620951}
[modify] https://crrev.com/aa595e0ebb8c5ecd64e1598bf1359f0cf076b818/build/android/gyp/apkbuilder.py

Project Member

Comment 3 by bugdroid1@chromium.org, Jan 16 (6 days ago)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/eee55f546d178db52b469ec19549f5f76b19ca79

commit eee55f546d178db52b469ec19549f5f76b19ca79
Author: Peter Collingbourne <pcc@chromium.org>
Date: Wed Jan 16 16:50:26 2019

clang: Add HWASAN Android runtime to the package.

Bug: 916828
Change-Id: I89ecdf1b4c72933f0323572bd233b17417ee969b
Reviewed-on: https://chromium-review.googlesource.com/c/1410266
Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Peter Collingbourne <pcc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#623259}
[modify] https://crrev.com/eee55f546d178db52b469ec19549f5f76b19ca79/tools/clang/scripts/package.py
[modify] https://crrev.com/eee55f546d178db52b469ec19549f5f76b19ca79/tools/clang/scripts/update.py

Sign in to add a comment