From eac6bee7c1aba48dd4f677dc354acba003d75d41 Mon Sep 17 00:00:00 2001 From: Dmitriy Vyukov Date: Thu, 27 Jun 2013 20:43:43 +0400 Subject: [PATCH] runtime: fix argument printing during traceback Current code can print more arguments than necessary and also incorrectly prints "...". Update #5723. R=golang-dev, rsc CC=golang-dev https://golang.org/cl/10689043 --- src/pkg/runtime/traceback_arm.c | 10 +++++----- src/pkg/runtime/traceback_x86.c | 10 +++++----- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/src/pkg/runtime/traceback_arm.c b/src/pkg/runtime/traceback_arm.c index c4d78872bb..b91017eea5 100644 --- a/src/pkg/runtime/traceback_arm.c +++ b/src/pkg/runtime/traceback_arm.c @@ -139,14 +139,14 @@ runtime·gentraceback(uintptr pc0, uintptr sp0, uintptr lr0, G *gp, int32 skip, if(n > 0 && frame.pc > f->entry && !waspanic) tracepc -= sizeof(uintptr); runtime·printf("%S(", f->name); - for(i = 0; i < f->args/sizeof(uintptr); i++) { - if(i != 0) - runtime·prints(", "); - runtime·printhex(((uintptr*)frame.argp)[i]); - if(i >= 4) { + for(i = 0; i < frame.arglen/sizeof(uintptr); i++) { + if(i >= 5) { runtime·prints(", ..."); break; } + if(i != 0) + runtime·prints(", "); + runtime·printhex(((uintptr*)frame.argp)[i]); } runtime·prints(")\n"); runtime·printf("\t%S:%d", f->src, runtime·funcline(f, tracepc)); diff --git a/src/pkg/runtime/traceback_x86.c b/src/pkg/runtime/traceback_x86.c index 739d93aa8c..a6c6ebcae9 100644 --- a/src/pkg/runtime/traceback_x86.c +++ b/src/pkg/runtime/traceback_x86.c @@ -142,14 +142,14 @@ runtime·gentraceback(uintptr pc0, uintptr sp0, uintptr lr0, G *gp, int32 skip, if(n > 0 && frame.pc > f->entry && !waspanic) tracepc--; runtime·printf("%S(", f->name); - for(i = 0; i < f->args/sizeof(uintptr); i++) { - if(i != 0) - runtime·prints(", "); - runtime·printhex(((uintptr*)frame.argp)[i]); - if(i >= 4) { + for(i = 0; i < frame.arglen/sizeof(uintptr); i++) { + if(i >= 5) { runtime·prints(", ..."); break; } + if(i != 0) + runtime·prints(", "); + runtime·printhex(((uintptr*)frame.argp)[i]); } runtime·prints(")\n"); runtime·printf("\t%S:%d", f->src, runtime·funcline(f, tracepc)); -- 2.48.1