From 575eb8f58b06657c9cadfe6f7ddfd9e530fe5dea Mon Sep 17 00:00:00 2001 From: Ian Lance Taylor Date: Mon, 7 Jan 2019 20:12:39 +0000 Subject: runtime: in getTraceback, set gp->m before gogo Currently, when collecting a traceback for another goroutine, getTraceback calls gogo(gp) switching to gp, which will resume in mcall, which will call gtraceback, which will set up gp->m. There is a gap between setting the current running g to gp and setting gp->m. If a profiling signal arrives in between, sigtramp will see a non-nil gp with a nil m, and will seg fault. Fix this by setting up gp->m first. Fixes golang/go#29448. Reviewed-on: https://go-review.googlesource.com/c/156038 From-SVN: r267658 --- libgo/runtime/proc.c | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) (limited to 'libgo/runtime/proc.c') diff --git a/libgo/runtime/proc.c b/libgo/runtime/proc.c index 99b2cb1..4004df4 100644 --- a/libgo/runtime/proc.c +++ b/libgo/runtime/proc.c @@ -442,6 +442,11 @@ void getTraceback(G*, G*) __asm__(GOSYM_PREFIX "runtime.getTraceback"); // goroutine stored in the traceback field, which is me. void getTraceback(G* me, G* gp) { + M* holdm; + + holdm = gp->m; + gp->m = me->m; + #ifdef USING_SPLIT_STACK __splitstack_getcontext((void*)(&me->stackcontext[0])); #endif @@ -450,6 +455,8 @@ void getTraceback(G* me, G* gp) if (gp->traceback != 0) { runtime_gogo(gp); } + + gp->m = holdm; } // Do a stack trace of gp, and then restore the context to @@ -459,17 +466,11 @@ void gtraceback(G* gp) { Traceback* traceback; - M* holdm; traceback = (Traceback*)gp->traceback; gp->traceback = 0; - holdm = gp->m; - if(holdm != nil && holdm != g->m) - runtime_throw("gtraceback: m is not nil"); - gp->m = traceback->gp->m; traceback->c = runtime_callers(1, traceback->locbuf, sizeof traceback->locbuf / sizeof traceback->locbuf[0], false); - gp->m = holdm; runtime_gogo(traceback->gp); } -- cgit v1.1