This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[RFA] print error message if (auto) disassembly failed
- From: Joel Brobecker <brobecker at adacore dot com>
- To: gdb-patches at sourceware dot org
- Date: Thu, 16 Apr 2009 10:39:18 -0700
- Subject: [RFA] print error message if (auto) disassembly failed
I noticed this by accident after we introduced the new feature where
the debugger prints the assembly of the next line each time we stop.
Basically, I loaded the wrong core file and got:
(gdb) core core
warning: core file may not match specified executable file.
Core was generated by `./crash'.
Program terminated with signal 6, Aborted.
#0 0x00007f8befcc8307 in ?? ()
!!!-> 0x00007f8befcc8307: (gdb)
As you can see, GDB tried to print the instruction at the address
where the core file says the program stopped, but instead printed
nothing. This has two consequences:
1. The user does not really know what the problem was;
2. A testsuite driver that relies on seeing the prompt being placed
at the beginning of the line would wait indefinitely for the prompt
to come back.
I propose to change the behavior to print the error message:
(gdb) core core
warning: core file may not match specified executable file.
Core was generated by `./crash'.
Program terminated with signal 6, Aborted.
#0 0x00007f8befcc8307 in ?? ()
0x00007f8befcc8307: Cannot access memory at address 0x7f8befcc8307
(gdb)
2009-04-16 Joel Brobecker <brobecker@adacore.com>
* stack.c (do_gdb_disassembly): Print the exception message if an
error was thrown while trying to perform the disassembly.
Tested on x86_64-linux. Any objections?
(FWIW: I did not realize that GDB would now print the current assembly
instruction by default when the change came in. I don't think that's
a useful default based on my own usage, but no big deal)
--
Joel
commit 056d77e1b180988399b18ea5d109813dbd934da8
Author: Joel Brobecker <brobecker@adacore.com>
Date: Tue Apr 14 15:23:25 2009 -0700
* stack.c (do_gdb_disassembly): Print the exception message if an
error was thrown while trying to perform the disassembly.
diff --git a/gdb/stack.c b/gdb/stack.c
index dfe3900..956aaa0 100644
--- a/gdb/stack.c
+++ b/gdb/stack.c
@@ -507,6 +507,10 @@ do_gdb_disassembly (int how_many, CORE_ADDR low, CORE_ADDR high)
{
gdb_disassembly_stub (&args);
}
+ /* If an exception was thrown while doing the disassembly, print
+ the error message, to give the user a clue of what happened. */
+ if (exception.reason == RETURN_ERROR)
+ exception_print (gdb_stderr, exception);
}
/* Print information about frame FRAME. The output is format according