diff options
| author | Josh Gao <jmgao@google.com> | 2017-02-14 21:03:23 -0800 |
|---|---|---|
| committer | Josh Gao <jmgao@google.com> | 2017-02-15 12:24:09 -0800 |
| commit | 60515bf9f1b9eb222ed5fff274a8917ed9548add (patch) | |
| tree | d87f38e2a7d508713dcf5b3888cb67679226404d /debuggerd/handler/debuggerd_fallback.cpp | |
| parent | af776fd518df4f677bf8c3f4a8d22d23a3479181 (diff) | |
debuggerd_handler: don't use snprintf in handler.
snprintf isn't safe to call in the linker after initialization, because
it uses MB_CUR_MAX which is implemented via pthread_getspecific, which
uses TLS slots shared with libc. If the TLS slots are assigned in a
different order between libc.so and the linker, MB_CUR_MAX will
evaluate to an incorrect value, and lead to snprintf doing bad things.
Switch to __libc_format_buffer.
Bug: http://b/35367169
Test: debuggerd -b `pidof zygote`
Change-Id: I9d315cf63e5f3fd2f4545d6e3f707cdbe94ec606
Diffstat (limited to 'debuggerd/handler/debuggerd_fallback.cpp')
0 files changed, 0 insertions, 0 deletions