d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

In stack_save, stop when a trap-frame is encountered. This prevents

Browse Source 
trying to access user-space stack addresses when a user fault
is encountered, as occurs when GEOM KTR code is handling a page fault
and is using stack_save() to capture a trace for debug purposes.

It may be possible to walk beyond the trap-frame if it is a kernel fault,
as db_backtrace() does, but I don't think that complexity is needed in
this routine.

MFC after:	3 days
This commit is contained in:
Peter Grehan 2005-10-30 07:56:10 +00:00
parent fe8388fec2
commit c0498352c1
1 changed files with 10 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
sys/powerpc/powerpc/db_trace.c
View File

@ -305,6 +305,16 @@ stack_save(struct stack *st)
callpc = *(vm_offset_t *)(stackframe + 4) - 4;
if ((callpc & 3) || (callpc < 0x100))
break;
/*
* Don't bother traversing trap-frames - there should
* be enough info down to the frame to work out where
* things are going wrong. Plus, prevents this shortened
* version of code from accessing user-space frames
*/
if (callpc + 4 == (db_addr_t) &trapexit)
break;
if (stack_put(st, callpc) == -1)
break;
}