Re: [Patch, ARM] Enable libsanitizer

2013-04-19 Thread Konstantin Serebryany
On Fri, Apr 19, 2013 at 7:59 AM, Christophe Lyon wrote: > On 18 April 2013 11:30, Christophe Lyon wrote: >> On 4 April 2013 14:19, Christophe Lyon wrote: >>> ~/src/qemu/qemu-git/arm-linux-user/qemu-arm -cpu cortex-a9 -R 0 -L >>> /home/lyon/src/GCC/builds/gcc-fsf-asan-arm-none-linux-gnueabihf/sys

Re: [Patch, ARM] Enable libsanitizer

2013-04-19 Thread Christophe Lyon
On 18 April 2013 11:30, Christophe Lyon wrote: > On 4 April 2013 14:19, Christophe Lyon wrote: >> ~/src/qemu/qemu-git/arm-linux-user/qemu-arm -cpu cortex-a9 -R 0 -L >> /home/lyon/src/GCC/builds/gcc-fsf-asan-arm-none-linux-gnueabihf/sysroot >> ./heap-overflow-1.arm >> >> On 28 March 2013 15:33, C

Re: [Patch, ARM] Enable libsanitizer

2013-04-18 Thread Christophe Lyon
On 4 April 2013 14:19, Christophe Lyon wrote: > ~/src/qemu/qemu-git/arm-linux-user/qemu-arm -cpu cortex-a9 -R 0 -L > /home/lyon/src/GCC/builds/gcc-fsf-asan-arm-none-linux-gnueabihf/sysroot > ./heap-overflow-1.arm > > On 28 March 2013 15:33, Christophe Lyon wrote: - libsanitizer detects if i

Re: [Patch, ARM] Enable libsanitizer

2013-04-04 Thread Christophe Lyon
~/src/qemu/qemu-git/arm-linux-user/qemu-arm -cpu cortex-a9 -R 0 -L /home/lyon/src/GCC/builds/gcc-fsf-asan-arm-none-linux-gnueabihf/sysroot ./heap-overflow-1.arm On 28 March 2013 15:33, Christophe Lyon wrote: >>> - libsanitizer detects if its output is a tty, and when GCC testsuite >>> is execute

Re: [Patch, ARM] Enable libsanitizer

2013-03-28 Thread Christophe Lyon
>> Note that the testsuite currently fails when executing under qemu: >> - support of /proc/self/maps does not conform to the kernel format. >> One extra space is missing from some lines, which confuses libsanitizer. >> Patch proposed to upstream qemu: >> http://lists.gnu.org/archive/html/qem

Re: [Patch, ARM] Enable libsanitizer

2013-03-28 Thread Konstantin Serebryany
On Thu, Mar 28, 2013 at 12:07 PM, Jakub Jelinek wrote: > On Thu, Mar 28, 2013 at 12:00:23PM +0400, Evgeniy Stepanov wrote: >> We do it because newer versions of Android use PIE binaries, and, >> combined with other specifics of address space on Linux/ARM, there is >> no space for ASan shadow anywh

Re: [Patch, ARM] Enable libsanitizer

2013-03-28 Thread Jakub Jelinek
On Thu, Mar 28, 2013 at 12:00:23PM +0400, Evgeniy Stepanov wrote: > We do it because newer versions of Android use PIE binaries, and, > combined with other specifics of address space on Linux/ARM, there is > no space for ASan shadow anywhere else. And it's faster. > > Zero-based shadow requires PI

Re: [Patch, ARM] Enable libsanitizer

2013-03-28 Thread Evgeniy Stepanov
On Thu, Mar 28, 2013 at 11:36 AM, Konstantin Serebryany wrote: > +euge...@google.com > > Hi Christophe, > > On Thu, Mar 28, 2013 at 2:09 AM, Christophe Lyon > wrote: >> Hi, >> This small patch enables libsanitizer on ARM. >> It has been tested successfully on cortex-a9 hardware (via the GCC >> t

Re: [Patch, ARM] Enable libsanitizer

2013-03-28 Thread Konstantin Serebryany
+euge...@google.com Hi Christophe, On Thu, Mar 28, 2013 at 2:09 AM, Christophe Lyon wrote: > Hi, > This small patch enables libsanitizer on ARM. > It has been tested successfully on cortex-a9 hardware (via the GCC testsuite). > > I have chosen to bundle -funwind-table with -fsanitize=* so that a

[Patch, ARM] Enable libsanitizer

2013-03-27 Thread Christophe Lyon
Hi, This small patch enables libsanitizer on ARM. It has been tested successfully on cortex-a9 hardware (via the GCC testsuite). I have chosen to bundle -funwind-table with -fsanitize=* so that a useful backtrace can be printed to the user in case of error, otherwise the reporting is limited to on