Revert: Code cleanup: Move print_command_1 expr variable scope
Simon Marchi:
I think this patch is wrong. Starting with that commit (f30d5c7
),
some tests (e.g. mi-break.exp) started to fail for me, because
of gdb segfaulting.
The address of expr is passed to the cleanup. When the cleanup is ran,
expr is no longer in scope, so what is at that address is probably not
safe to use anymore. That's my guess.
gdb/ChangeLog
2015-03-27 Jan Kratochvil <jan.kratochvil@redhat.com>
Revert:
2015-03-26 Jan Kratochvil <jan.kratochvil@redhat.com>
Code cleanup.
* printcmd.c (print_command_1): Move expr variable scope.
This commit is contained in:
parent
e04de5e3b5
commit
1c4ff0802b
2 changed files with 8 additions and 2 deletions
|
@ -1,3 +1,10 @@
|
|||
2015-03-27 Jan Kratochvil <jan.kratochvil@redhat.com>
|
||||
|
||||
Revert:
|
||||
2015-03-26 Jan Kratochvil <jan.kratochvil@redhat.com>
|
||||
Code cleanup.
|
||||
* printcmd.c (print_command_1): Move expr variable scope.
|
||||
|
||||
2015-03-27 Joel Brobecker <brobecker@adacore.com>
|
||||
|
||||
* dtrace-probe.c (dtrace_process_dof_probe): Initialize expr to NULL.
|
||||
|
|
|
@ -946,6 +946,7 @@ validate_format (struct format_data fmt, const char *cmdname)
|
|||
static void
|
||||
print_command_1 (const char *exp, int voidprint)
|
||||
{
|
||||
struct expression *expr;
|
||||
struct cleanup *old_chain = make_cleanup (null_cleanup, NULL);
|
||||
char format = 0;
|
||||
struct value *val;
|
||||
|
@ -968,8 +969,6 @@ print_command_1 (const char *exp, int voidprint)
|
|||
|
||||
if (exp && *exp)
|
||||
{
|
||||
struct expression *expr;
|
||||
|
||||
expr = parse_expression (exp);
|
||||
make_cleanup (free_current_contents, &expr);
|
||||
val = evaluate_expression (expr);
|
||||
|
|
Loading…
Reference in a new issue