Author: kib
Date: Fri Aug 7 04:31:02 2015
New Revision: 286396
URL: https://svnweb.freebsd.org/changeset/base/286396
Log:
MFC r285643:
When checking for the valid value of the frame pointer, verify that it
belongs to the kernel stack address range for the thread.
Modified:
stable/10/sys/amd64/amd64/stack_machdep.c
stable/10/sys/cddl/dev/dtrace/amd64/dtrace_isa.c
stable/10/sys/cddl/dev/dtrace/i386/dtrace_isa.c
stable/10/sys/i386/i386/stack_machdep.c
Directory Properties:
stable/10/ (props changed)
Modified: stable/10/sys/amd64/amd64/stack_machdep.c
==
--- stable/10/sys/amd64/amd64/stack_machdep.c Fri Aug 7 04:27:51 2015
(r286395)
+++ stable/10/sys/amd64/amd64/stack_machdep.c Fri Aug 7 04:31:02 2015
(r286396)
@@ -40,7 +40,7 @@ __FBSDID($FreeBSD$);
#include vm/pmap.h
static void
-stack_capture(struct stack *st, register_t rbp)
+stack_capture(struct thread *td, struct stack *st, register_t rbp)
{
struct amd64_frame *frame;
vm_offset_t callpc;
@@ -56,8 +56,8 @@ stack_capture(struct stack *st, register
if (stack_put(st, callpc) == -1)
break;
if (frame-f_frame = frame ||
- (vm_offset_t)frame-f_frame =
- (vm_offset_t)rbp + KSTACK_PAGES * PAGE_SIZE)
+ (vm_offset_t)frame-f_frame = td-td_kstack +
+ td-td_kstack_pages * PAGE_SIZE)
break;
frame = frame-f_frame;
}
@@ -74,7 +74,7 @@ stack_save_td(struct stack *st, struct t
panic(stack_save_td: running);
rbp = td-td_pcb-pcb_rbp;
- stack_capture(st, rbp);
+ stack_capture(td, st, rbp);
}
void
@@ -83,5 +83,5 @@ stack_save(struct stack *st)
register_t rbp;
__asm __volatile(movq %%rbp,%0 : =r (rbp));
- stack_capture(st, rbp);
+ stack_capture(curthread, st, rbp);
}
Modified: stable/10/sys/cddl/dev/dtrace/amd64/dtrace_isa.c
==
--- stable/10/sys/cddl/dev/dtrace/amd64/dtrace_isa.cFri Aug 7 04:27:51
2015(r286395)
+++ stable/10/sys/cddl/dev/dtrace/amd64/dtrace_isa.cFri Aug 7 04:31:02
2015(r286396)
@@ -89,8 +89,8 @@ dtrace_getpcstack(pc_t *pcstack, int pcs
}
if (frame-f_frame = frame ||
- (vm_offset_t)frame-f_frame =
- (vm_offset_t)rbp + KSTACK_PAGES * PAGE_SIZE)
+ (vm_offset_t)frame-f_frame = curthread-td_kstack +
+ curthread-td_kstack_pages * PAGE_SIZE)
break;
frame = frame-f_frame;
}
@@ -469,8 +469,8 @@ dtrace_getstackdepth(int aframes)
break;
depth++;
if (frame-f_frame = frame ||
- (vm_offset_t)frame-f_frame =
- (vm_offset_t)rbp + KSTACK_PAGES * PAGE_SIZE)
+ (vm_offset_t)frame-f_frame = curthread-td_kstack +
+ curthread-td_kstack_pages * PAGE_SIZE)
break;
frame = frame-f_frame;
}
Modified: stable/10/sys/cddl/dev/dtrace/i386/dtrace_isa.c
==
--- stable/10/sys/cddl/dev/dtrace/i386/dtrace_isa.c Fri Aug 7 04:27:51
2015(r286395)
+++ stable/10/sys/cddl/dev/dtrace/i386/dtrace_isa.c Fri Aug 7 04:31:02
2015(r286396)
@@ -92,8 +92,8 @@ dtrace_getpcstack(pc_t *pcstack, int pcs
}
if (frame-f_frame = frame ||
- (vm_offset_t)frame-f_frame =
- (vm_offset_t)ebp + KSTACK_PAGES * PAGE_SIZE)
+ (vm_offset_t)frame-f_frame = curthread-td_kstack +
+ curthread-td_kstack_pages * PAGE_SIZE)
break;
frame = frame-f_frame;
}
@@ -485,8 +485,8 @@ dtrace_getstackdepth(int aframes)
break;
depth++;
if (frame-f_frame = frame ||
- (vm_offset_t)frame-f_frame =
- (vm_offset_t)ebp + KSTACK_PAGES * PAGE_SIZE)
+ (vm_offset_t)frame-f_frame = curthread-td_kstack +
+ curthread-td_kstack_pages * PAGE_SIZE)
break;
frame = frame-f_frame;
}
Modified: stable/10/sys/i386/i386/stack_machdep.c
==
--- stable/10/sys/i386/i386/stack_machdep.c Fri Aug 7 04:27:51 2015
(r286395)
+++ stable/10/sys/i386/i386/stack_machdep.c Fri Aug 7 04:31:02 2015
(r286396)
@@ -40,7 +40,7 @@ __FBSDID($FreeBSD$);
#include vm/pmap.h
static void
-stack_capture(struct stack *st, register_t ebp)
+stack_capture(struct