[PATCH] D31695: Windows asan_device_setup.bat port of linux shell script

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 28 15:58:38 PDT 2017


eugenis added a comment.

In https://reviews.llvm.org/D31695#824674, @gpx1000 wrote:

> In https://reviews.llvm.org/D31695#824667, @eugenis wrote:
>
> > This is great! Wrap.sh is definitely a better approach for 99% of users. Why are you saying that it needs an eng dev build though? I thought the idea behind wrap.sh is that it works on regular user devices, as long as the app has DEBUGGABLE in the manifest.
>
>
> It actually doesn't need to be an eng or dev build.  I'm saying it needs root (setenforce 0 / setenforce 1 are the keys).  If you have root, either through su or through adb root, then the wrap method will work.


I see. That's still bad, we'd like ASan to work on non-rooted devices. Do you know why setenforce 0 is necessary?


https://reviews.llvm.org/D31695





More information about the llvm-commits mailing list