aboutsummaryrefslogtreecommitdiff
path: root/gdb
diff options
context:
space:
mode:
Diffstat (limited to 'gdb')
-rw-r--r--gdb/ChangeLog5
-rw-r--r--gdb/testsuite/ChangeLog5
-rw-r--r--gdb/testsuite/gdb.trace/tfile.c36
-rw-r--r--gdb/tracepoint.c3
4 files changed, 46 insertions, 3 deletions
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 <pedro@codesourcery.com>
+ * tracepoint.c (trace_save): Remove X from tracepoint error
+ description.
+
+2010-03-26 Pedro Alves <pedro@codesourcery.com>
+
* 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 <pedro@codesourcery.com>
+
+ * gdb.trace/tfile.c (tohex, bin2hex): New.
+ (write_error_trace_file): Hexify error description.
+
2010-03-25 Stan Shebs <stan@codesourcery.com>
* 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)