steleman / address-sanitizer

Automatically exported from code.google.com/p/address-sanitizer
0 stars 0 forks source link

Extend activation flags limit #364

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
On Android, extra ASAN_OPTIONS are read of "asan.options" system property when 
ASan is activated. System properties are limited to 91 characters, which is not 
always enough.

The options are:
1. Continue reading from asan.options1, asan.options2, etc
2. Read from some hardcoded file (ex. /sdcard/asan.options or /asan.options).
3. Support "@path" syntax in ASAN_OPTIONS, meaning "read more options from the 
file at <path>", similar to "man ld".

Original issue reported on code.google.com by euge...@google.com on 18 Dec 2014 at 10:02

GoogleCodeExporter commented 9 years ago
hardcoded file path works for us. I worry about backwards compatability with 
older builds, do you think we should hack around and directly modify 
app_process / asanwrapper files ?

Original comment by infe...@chromium.org on 18 Dec 2014 at 4:14

GoogleCodeExporter commented 9 years ago
I've added include=/path/to/asan/options flag:
https://code.google.com/p/address-sanitizer/wiki/Flags

About backwards compatibility - I don't think we can support any of the options 
above with old libclang_rt.asan-arm-android.so.

Original comment by euge...@google.com on 16 Jan 2015 at 1:34

GoogleCodeExporter commented 9 years ago

Original comment by euge...@google.com on 16 Jan 2015 at 1:35