lk/linux-kernel-test
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

[PATCH] uml: Fix skas0 stub return

Browse Source 
It's wrong to pop a fixed number of words from stack before calling sigreturn,
as the number depends on what code is generated by the compiler for the start
of stub_segv_handler().  What we need is esp containing the address of
sigcontext.  So we explicitly load that pointer into esp.

Signed-off-by: Bodo Stroesser <bstroesser@fujitsu-siemens.com>
Signed-off-by: Jeff Dike <jdike@addtoit.com>
Cc: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
This commit is contained in:
Bodo Stroesser
2005-07-28 21:16:06 -07:00
committed by Linus Torvalds
parent 6f313b1233
commit 9057e9deee
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
arch/um/sys-i386/stub_segv.c
View File

@@ -21,10 +21,10 @@ stub_segv_handler(int sig)
__asm__("movl %0, %%eax ; int $0x80": : "g" (__NR_getpid)); __asm__("movl %0, %%eax ; int $0x80": : "g" (__NR_getpid));
__asm__("movl %%eax, %%ebx ; movl %0, %%eax ; movl %1, %%ecx ;" __asm__("movl %%eax, %%ebx ; movl %0, %%eax ; movl %1, %%ecx ;"
"int $0x80": : "g" (__NR_kill), "g" (SIGUSR1)); "int $0x80": : "g" (__NR_kill), "g" (SIGUSR1));
/* Pop the frame pointer and return address since we need to leave /* Load pointer to sigcontext into esp, since we need to leave
* the stack in its original form when we do the sigreturn here, by * the stack in its original form when we do the sigreturn here, by
* hand. * hand.
*/ */
__asm__("popl %%eax ; popl %%eax ; popl %%eax ; movl %0, %%eax ; " __asm__("mov %0,%%esp ; movl %1, %%eax ; "
"int $0x80" : : "g" (__NR_sigreturn)); "int $0x80" : : "a" (sc), "g" (__NR_sigreturn));
} }