Convert arch_within_stack_frames() to use the new unwinder.

Boot tested with CONFIG_HARDENED_USERCOPY.

Signed-off-by: Josh Poimboeuf <jpoim...@redhat.com>
---
 arch/x86/lib/usercopy.c | 25 +++++++++++++++++++------
 1 file changed, 19 insertions(+), 6 deletions(-)

diff --git a/arch/x86/lib/usercopy.c b/arch/x86/lib/usercopy.c
index 96ce151..9d0913c 100644
--- a/arch/x86/lib/usercopy.c
+++ b/arch/x86/lib/usercopy.c
@@ -50,12 +50,21 @@ int arch_within_stack_frames(const void * const stack,
                             const void * const stackend,
                             const void *obj, unsigned long len)
 {
-       const void *frame = NULL;
-       const void *oldframe;
+       struct unwind_state state;
+       const void *frame, *oldframe;
+
+       unwind_start(&state, current, NULL, NULL);
+
+       if (!unwind_next_frame(&state))
+               return 0;
+
+       oldframe = unwind_get_stack_ptr(&state);
+
+       if (!unwind_next_frame(&state))
+               return 0;
+
+       frame = unwind_get_stack_ptr(&state);
 
-       oldframe = __builtin_frame_address(2);
-       if (oldframe)
-               frame = __builtin_frame_address(3);
        /*
         * low ----------------------------------------------> high
         * [saved bp][saved ip][args][local vars][saved bp][saved ip]
@@ -71,8 +80,12 @@ int arch_within_stack_frames(const void * const stack,
                 */
                if (obj + len <= frame)
                        return obj >= oldframe + 2 * sizeof(void *) ? 1 : -1;
+
+               if (!unwind_next_frame(&state))
+                       return 0;
+
                oldframe = frame;
-               frame = *(const void * const *)frame;
+               frame = unwind_get_stack_ptr(&state);
        }
        return -1;
 }
-- 
2.7.4

Reply via email to