diff options
author | Don Howard <dhoward@redhat.com> | 2002-04-12 22:31:23 +0000 |
---|---|---|
committer | Don Howard <dhoward@redhat.com> | 2002-04-12 22:31:23 +0000 |
commit | 20f01a4665aec81bd667feb25cab93a8bbe3c9af (patch) | |
tree | 6dba79c458d3ab5df20a15a74802917194499f95 /gdb/cli/cli-script.c | |
parent | a88376a3e1a5e2358f568a4a7aed6d4d99e1e87a (diff) | |
download | gdb-20f01a4665aec81bd667feb25cab93a8bbe3c9af.zip gdb-20f01a4665aec81bd667feb25cab93a8bbe3c9af.tar.gz gdb-20f01a4665aec81bd667feb25cab93a8bbe3c9af.tar.bz2 |
2002-04-12 Don Howard <dhoward@redhat.com>
* cli/cli-cmds.c (init_cli_cmds): Add new user settable value:
max_user_call_depth.
(init_cmd_lists): Initialize the new value;
* cli/cli-script.c (execute_user_command): Limit the call depth of
user defined commands. This avoids a core-dump when user commands
are infinitly recursive.
Diffstat (limited to 'gdb/cli/cli-script.c')
-rw-r--r-- | gdb/cli/cli-script.c | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/gdb/cli/cli-script.c b/gdb/cli/cli-script.c index b438fef..fc3261c 100644 --- a/gdb/cli/cli-script.c +++ b/gdb/cli/cli-script.c @@ -247,6 +247,15 @@ execute_cmd_post_hook (struct cmd_list_element *c) } /* Execute the command in CMD. */ +void +do_restore_user_call_depth (void * call_depth) +{ + int * depth = call_depth; + /* We will be returning_to_top_level() at this point, so we want to + reset our depth. */ + (*depth) = 0; +} + void execute_user_command (struct cmd_list_element *c, char *args) @@ -254,6 +263,8 @@ execute_user_command (struct cmd_list_element *c, char *args) register struct command_line *cmdlines; struct cleanup *old_chain; enum command_control_type ret; + static int user_call_depth = 0; + extern int max_user_call_depth; old_chain = setup_user_args (args); @@ -262,6 +273,11 @@ execute_user_command (struct cmd_list_element *c, char *args) /* Null command */ return; + if (++user_call_depth > max_user_call_depth) + error ("Max user call depth exceeded -- command aborted\n"); + + old_chain = make_cleanup (do_restore_user_call_depth, &user_call_depth); + /* Set the instream to 0, indicating execution of a user-defined function. */ old_chain = make_cleanup (do_restore_instream_cleanup, instream); @@ -277,6 +293,8 @@ execute_user_command (struct cmd_list_element *c, char *args) cmdlines = cmdlines->next; } do_cleanups (old_chain); + + user_call_depth--; } enum command_control_type |