forked from Minki/linux
adf9ae0d15
In commit9f0b4807a4("um: rework userspace stubs to not hard-code stub location") I changed stub_segv_handler() to do a calculation with a pointer to a stack variable to find the data page that we're using for the stack and the rest of the data. This same commit was meant to do it as well for stub_clone_handler(), but the change inadvertently went into commit84b2789d61("um: separate child and parent errors in clone stub") instead. This was reported to not be compiled correctly by gcc 5, causing the code to crash here. I'm not sure why, perhaps it's UB because the var isn't initialized? In any case, this trick always seemed bad, so just create a new inline function that does the calculation in assembly. Reported-by: subashab@codeaurora.org Fixes:9f0b4807a4("um: rework userspace stubs to not hard-code stub location") Fixes:84b2789d61("um: separate child and parent errors in clone stub") Signed-off-by: Johannes Berg <johannes.berg@intel.com> Signed-off-by: Richard Weinberger <richard@nod.at>
21 lines
455 B
C
21 lines
455 B
C
/*
|
|
* Copyright (C) 2004 - 2007 Jeff Dike (jdike@{addtoit,linux.intel}.com)
|
|
* Licensed under the GPL
|
|
*/
|
|
|
|
#include <sysdep/stub.h>
|
|
#include <sysdep/faultinfo.h>
|
|
#include <sysdep/mcontext.h>
|
|
#include <sys/ucontext.h>
|
|
|
|
void __attribute__ ((__section__ (".__syscall_stub")))
|
|
stub_segv_handler(int sig, siginfo_t *info, void *p)
|
|
{
|
|
struct faultinfo *f = get_stub_page();
|
|
ucontext_t *uc = p;
|
|
|
|
GET_FAULTINFO_FROM_MC(*f, &uc->uc_mcontext);
|
|
trap_myself();
|
|
}
|
|
|