[asan] disable fast unwind on ARM

[garious]

Summary

Fast unwind is disabled for aarch64. Following suit. This patch fixes the following lit tests:

Linux/malloc_delete_mismatch.cc
Posix/start-deactivated.cc
deep_stack_uaf.cc
double-free.cc
invalid-free.cc
large_func_test.cc

[kcc]

We do use fast unwind on Android ARM -- it is a performance-critical
feature, please don't disable it, but fix instead.
eugenis@ knows the details.

[garious]

[garious]

Per Kostya, this version does not disable fast unwind on Android.

With this patch, the error messages are significantly more useful. Without this patch, in the use-after-free test for example, the stack trace only has references to the 'malloc/free' wrappers in the ASan runtime and not references to anything in use-after-free.cc.

[kcc]

I think this is a mistake.
The slow unwinder is so terribly slow that you will not be able to use asan
on anything other than small test cases.
Instead, you need to make the fast unwinder work by enabling frame pointers
at compile time.
We do it on Android ARM, so I think this can and should be done on Linux
ARM as well.

Note also, that there are run-time flags that control the unwinder:
fast_unwind_on_fatal0Use fast unwinder when found a bug. The fast one
relies on frame pointers, while slow one does not. See also
CallStackhttps://code.google.com/p/address-sanitizer/wiki/CallStack
fast_unwind_on_malloc1Same as fast_unwind_on_fatal, but control unwinding
inside malloc/free, i.e. performance critical. See also
CallStackhttps://code.google.com/p/address-sanitizer/wiki/CallStack
--kcc

[kcc]

+eugenis

[eugenis]

Fast stack unwind should work on ARM for Clang-compiled code, no matter Android or not.
It requires -marm -fno-omit-frame-pointer.

[ygribov]

Fast stack unwind should work on ARM for Clang-compiled code, no matter Android or not.
It requires -marm -fno-omit-frame-pointer.

AFAIK supporting frame pointer violates standard ARM ABIs so this will force user to recompile complete stack (glibc and other system libs). That's perfectly fine for some users but I doubt about majority. Are we sure that fast_unwind_on_malloc should be default on platforms without default frame pointer?

[eugenis]

It's quite uncommon to have frame pointers in glibc or system libraries anyway, even on x86 linux.
But with the dwarf unwinder tool performance becomes completely unacceptable.

[ygribov]

It's quite uncommon to have frame pointers in glibc or system libraries anyway, even on x86 linux.

Ok, you win.

[garious]

Fast stack unwind should work on ARM for Clang-compiled code, no matter Android or not.

It requires -marm -fno-omit-frame-pointer.

Hmm, I'm not seeing that. Zooming in on "TestCases/double-free.cc", I see that it is being compiled with -fno-omit-frame-pointer (and -marm is the default for armv7 targets). I double-checked and recompiled with those flags tacked onto the end of the clang invocation, but the result is the same:

==12127==ERROR: AddressSanitizer: attempting double-free on 0x41e007f0 in thread T0:
    #0 0x7269b in free compiler-rt/lib/asan/asan_malloc_linux.cc:65
    #1 0x896e3 in main compiler-rt/test/asan/TestCases/double-free.cc:15
  LLVMSymbolizer: error reading file: No such file or directory.
    #2 0x40a142cb (/lib/arm-linux-gnueabi/libc.so.6+0x182cb)
  
  0x41e007f0 is located 0 bytes inside of 10-byte region [0x41e007f0,0x41e007fa)
  freed by thread T0 here:
    #0 0x7269b in free compiler-rt/lib/asan/asan_malloc_linux.cc:65
  
  previously allocated by thread T0 here:
    #0 0x72859 in malloc compiler-rt/lib/asan/asan_malloc_linux.cc:75
  
  SUMMARY: AddressSanitizer: double-free compiler-rt/lib/asan/asan_malloc_linux.cc:65 free
  ==12127==ABORTING

What am I missing?

lib/sanitizer_common/sanitizer_stacktrace.h

 
 #if SANITIZER_LINUX && (defined(__aarch64__) || defined(__powerpc__) || \
                         defined(__powerpc64__) || defined(__sparc__) || \
-                        defined(__mips__))
+                        defined(__mips__) || \
+                        (!SANITIZER_ANDROID && defined(__arm__)))
 # define SANITIZER_CAN_FAST_UNWIND 0
 #elif SANITIZER_WINDOWS
 # define SANITIZER_CAN_FAST_UNWIND 0