From 610197fd468b01ed5bc653b19e7c720f2a8d1f8d Mon Sep 17 00:00:00 2001 From: Pedro Alves Date: Fri, 26 Mar 2010 14:31:48 +0000 Subject: gdb/ * tracepoint.c (trace_save): Remove X from tracepoint error description. gdb/testsuite/ * gdb.trace/tfile.c (tohex, bin2hex): New. (write_error_trace_file): Hexify error description. --- gdb/ChangeLog | 5 +++++ gdb/testsuite/ChangeLog | 5 +++++ gdb/testsuite/gdb.trace/tfile.c | 36 +++++++++++++++++++++++++++++++++++- gdb/tracepoint.c | 3 +-- 4 files changed, 46 insertions(+), 3 deletions(-) (limited to 'gdb') diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 5c015a5..90c01a9 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,10 @@ 2010-03-26 Pedro Alves + * tracepoint.c (trace_save): Remove X from tracepoint error + description. + +2010-03-26 Pedro Alves + * tracepoint.c (parse_trace_status): Don't allow plain strings in the terror description. Don't expect an X prefix. diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 3800958..08946f3 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2010-03-26 Pedro Alves + + * gdb.trace/tfile.c (tohex, bin2hex): New. + (write_error_trace_file): Hexify error description. + 2010-03-25 Stan Shebs * gdb.trace/tfile.c: Generate an additional trace file, improve diff --git a/gdb/testsuite/gdb.trace/tfile.c b/gdb/testsuite/gdb.trace/tfile.c index 5d4c797..4deba06 100644 --- a/gdb/testsuite/gdb.trace/tfile.c +++ b/gdb/testsuite/gdb.trace/tfile.c @@ -105,10 +105,38 @@ write_basic_trace_file (void) finish_trace_file (fd); } +/* Convert number NIB to a hex digit. */ + +static int +tohex (int nib) +{ + if (nib < 10) + return '0' + nib; + else + return 'a' + nib - 10; +} + +int +bin2hex (const char *bin, char *hex, int count) +{ + int i; + + for (i = 0; i < count; i++) + { + *hex++ = tohex ((*bin >> 4) & 0xf); + *hex++ = tohex (*bin++ & 0xf); + } + *hex = 0; + return i; +} + void write_error_trace_file (void) { int fd; + const char made_up[] = "made-up error"; + int len = sizeof (made_up) - 1; + char *hex = alloca (len * 2 + 1); fd = start_trace_file ("error.tf"); @@ -120,8 +148,14 @@ write_error_trace_file (void) snprintf (spbuf, sizeof spbuf, "R %x\n", 500 /* FIXME get from arch */); write (fd, spbuf, strlen (spbuf)); + bin2hex (made_up, hex, len); + /* Dump trace status, in the general form of the qTstatus reply. */ - snprintf (spbuf, sizeof spbuf, "status 0;terror:made-up error:1;tframes:0;tcreated:0;tfree:100;tsize:1000\n"); + snprintf (spbuf, sizeof spbuf, + "status 0;" + "terror:%s:1;" + "tframes:0;tcreated:0;tfree:100;tsize:1000\n", + hex); write (fd, spbuf, strlen (spbuf)); /* Dump tracepoint definitions, in syntax similar to that used diff --git a/gdb/tracepoint.c b/gdb/tracepoint.c index fc364f3..95e3478 100644 --- a/gdb/tracepoint.c +++ b/gdb/tracepoint.c @@ -2484,12 +2484,11 @@ trace_save (const char *filename, int target_does_save) /* Write out status of the tracing run (aka "tstatus" info). */ fprintf (fp, "status %c;%s", (ts->running ? '1' : '0'), stop_reason_names[ts->stop_reason]); - /* Encode the error message in hex, might have weird chars. */ if (ts->stop_reason == tracepoint_error) { char *buf = (char *) alloca (strlen (ts->error_desc) * 2 + 1); bin2hex ((gdb_byte *) ts->error_desc, buf, 0); - fprintf (fp, ":X%s", buf); + fprintf (fp, ":%s", buf); } fprintf (fp, ":%x", ts->stopping_tracepoint); if (ts->traceframe_count >= 0) -- cgit v1.1