commit 559f27828cbc51851978c5781618fe5b3d34e4ba
author Chris Wailes <chriswailes@google.com> Mon May 03 14:27:12 2021 -0700
committer Chris Wailes <chriswailes@google.com> Fri May 07 15:00:55 2021 -0700
tree 4b6cb79ec78207cbe3faa1fab72597db1c436b07
parent 3e0defac5d546c3a0ed0f5faae1a40502ffac003

Clear the stack frame pointer in _start and __bionic_clone

This CL adds an instruction to the _start label that clears the frame
pointer.  This allows stack walking code to determine when it has
reached the end of the stack.

The __bionic_clone function is similarly modified, for architectures
that weren't already doing both.

Test: bionic-unit-tests
Test: CtsBionicTestCases
Change-Id: Iea3949f52c44f7931f9fff2d60d4d9e5c742c120

libc/arch-x86/bionic/__bionic_clone.S

diff --git a/libc/arch-x86/bionic/__bionic_clone.S b/libc/arch-x86/bionic/__bionic_clone.S
index b682b48..f0c58a0 100644
--- a/libc/arch-x86/bionic/__bionic_clone.S
+++ b/libc/arch-x86/bionic/__bionic_clone.S
@@ -45,6 +45,7 @@
 .L_bc_child:
         # We don't want anyone to unwind past this point.
         .cfi_undefined %eip
+        .cfi_undefined %ebp
         call    __start_thread
         hlt