servo / rust-mozjs

DEPRECATED - moved to servo/mozjs instead.
Mozilla Public License 2.0
293 stars 122 forks source link

Compile with ASAN? #378

Open alex opened 6 years ago

alex commented 6 years ago

Rust has the ability to compile code with ASAN, and spidermonkey also supports being compiled under ASAN. However, as far as I can tell, if you compile rust-mozjs with ASAN, the spidermonkey code doesn't actually end up with ASAN.

It'd be great if ASAN were propagated down so that the dangerous C++ also was sanitized. Cheers!

alex commented 6 years ago

I believe this also applies to the sanitizer-coverage flags.

alex commented 6 years ago

From looking at http://doc.crates.io/environment-variables.html#environment-variables-cargo-sets-for-build-scripts, I'm concerned that there's no way for build.rs to know that either ASAN or sancov are being used in compiling the rust code.