task_stack: Fix end_of_stack() for architectures with upwards-growing stack
authorHelge Deller <deller@gmx.de>
Mon, 4 Oct 2021 22:05:43 +0000 (00:05 +0200)
committerHelge Deller <deller@gmx.de>
Sat, 30 Oct 2021 21:11:01 +0000 (23:11 +0200)
The function end_of_stack() returns a pointer to the last entry of a
stack. For architectures like parisc where the stack grows upwards
return the pointer to the highest address in the stack.

Without this change I faced a crash on parisc, because the stackleak
functionality wrote STACKLEAK_POISON to the lowest address and thus
overwrote the first 4 bytes of the task_struct which included the
TIF_FLAGS.

Signed-off-by: Helge Deller <deller@gmx.de>
include/linux/sched/task_stack.h

index 2413427..d101505 100644 (file)
@@ -25,7 +25,11 @@ static inline void *task_stack_page(const struct task_struct *task)
 
 static inline unsigned long *end_of_stack(const struct task_struct *task)
 {
+#ifdef CONFIG_STACK_GROWSUP
+       return (unsigned long *)((unsigned long)task->stack + THREAD_SIZE) - 1;
+#else
        return task->stack;
+#endif
 }
 
 #elif !defined(__HAVE_THREAD_FUNCTIONS)