Remove unchecked casts to mi_interp
authorTom Tromey <tromey@adacore.com>
Thu, 8 Jun 2023 20:06:45 +0000 (14:06 -0600)
committerTom Tromey <tromey@adacore.com>
Fri, 7 Jul 2023 19:55:56 +0000 (13:55 -0600)
commit73aa9ef81be869176814cac635723e920f7ca612
treeae7d66f71ef3829c691e0e737dfd81069a0f8fa3
parent13f5f57e0d2fb3e06e15c57d67a40499a5910ba6
Remove unchecked casts to mi_interp

Simon noticed a crash that could be caused via new Python
gdb.execute_mi function.  Looking into this, I found a few unchecked
casts to mi_interp, like:

-  struct mi_interp *mi = (struct mi_interp *) command_interp ();

This patch replaces all such casts with safer variants.

For -gdb-exit and mi_load_progress, I chose to have the functions
simply not generate any output.  It didn't seem useful to do so.

Some casts I eliminated by adding a parameter to a function.  Then, in
mi_execute_command, I changed the code to use
gdb::checked_static_cast.  This is appropriate because this particular
overload can only be called by the MI interpreter.

There does not seem to be a very good way to test -gdb-exit.

Regression tested on x86-64 Fedora 36.
gdb/mi/mi-interp.c
gdb/mi/mi-interp.h
gdb/mi/mi-main.c