diff options
author | Elliott Hughes <enh@google.com> | 2017-06-20 17:28:42 -0700 |
---|---|---|
committer | Elliott Hughes <enh@google.com> | 2017-06-20 17:28:42 -0700 |
commit | 695713e931f0436aca56be9c0dacf2a5dd4e56e7 (patch) | |
tree | a9620a83744ebd50f711883109918f5733823fbd /libc/async_safe/async_safe_log.cpp | |
parent | e300bf837f1f10bc6020bbb81ed9c5f8b85310ce (diff) | |
download | android_bionic-695713e931f0436aca56be9c0dacf2a5dd4e56e7.tar.gz android_bionic-695713e931f0436aca56be9c0dacf2a5dd4e56e7.tar.bz2 android_bionic-695713e931f0436aca56be9c0dacf2a5dd4e56e7.zip |
Shave a stack frame off asserts.
No-one cares about seeing "async_safe_fatal" (which you have to admit is a
pretty confusing name for an app developer anyway).
On arm:
#00 pc 0001a43c /system/lib/libc.so (abort+63)
#01 pc 0001a627 /system/lib/libc.so (__assert+14)
And aarch64:
#00 pc 000000000001d75c /system/lib64/libc.so (abort+120)
#01 pc 000000000001dad0 /system/lib64/libc.so (__assert+44)
Bug: N/A
Test: ran `crasher assert` and `crasher64 assert`
Change-Id: I00be71c566c74cdb00f8e95d634777155bc3da03
Diffstat (limited to 'libc/async_safe/async_safe_log.cpp')
-rw-r--r-- | libc/async_safe/async_safe_log.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/libc/async_safe/async_safe_log.cpp b/libc/async_safe/async_safe_log.cpp index 372f385b2..99ff0c7f9 100644 --- a/libc/async_safe/async_safe_log.cpp +++ b/libc/async_safe/async_safe_log.cpp @@ -567,10 +567,9 @@ void async_safe_fatal_va_list(const char* prefix, const char* format, va_list ar android_set_abort_message(msg); } -void async_safe_fatal(const char* fmt, ...) { +void async_safe_fatal_no_abort(const char* fmt, ...) { va_list args; va_start(args, fmt); async_safe_fatal_va_list(nullptr, fmt, args); va_end(args); - abort(); } |