commit d5c19f32f60a76dcc6e87625460a052df8db4df9
author Steven Moreland <smoreland@google.com> Fri Dec 23 02:56:59 2022 +0000
committer Steven Moreland <smoreland@google.com> Fri Dec 23 02:57:08 2022 +0000
tree 6d533759ae0568c9f24bff772c68f5859ddb6ff0
parent b453c9755fb6f7cd711e544d8808ee2d15990395

stack guard - use __builtin_frame_address(0)

Pointers on the stack are pretty whack.
But clang builtins, they got yo' back.

Fixes: 258970199 (: or it'll get re-filed :)
Test: libbinderthreadstateutils_test

Change-Id: I4a7af39c680562a1b7a7c2c71c54eb371d857d5e

IPCThreadState.cpp

diff --git a/IPCThreadState.cpp b/IPCThreadState.cpp
index aa050fc..8631644 100644
--- a/IPCThreadState.cpp
+++ b/IPCThreadState.cpp
@@ -1147,7 +1147,7 @@
                 tr.offsets_size/sizeof(binder_size_t), freeBuffer, this);
 
             const void* origServingStackPointer = mServingStackPointer;
-            mServingStackPointer = &origServingStackPointer; // anything on the stack
+            mServingStackPointer = __builtin_frame_address(0);
 
             const pid_t origPid = mCallingPid;
             const char* origSid = mCallingSid;