1
0
mirror of https://github.com/irssi/irssi.git synced 2024-12-04 14:46:39 -05:00

Fix /eval recursion crashing (debian/patches/04eval_recurse.dpatch by David Pashley)

git-svn-id: http://svn.irssi.org/repos/irssi/trunk@3865 dbcabf3a-b0e7-0310-adc4-f8d773084564
This commit is contained in:
Wouter Coekaerts 2005-07-17 16:43:18 +00:00 committed by coekie
parent 181f00c15e
commit 547065cb42
5 changed files with 12 additions and 2 deletions

View File

@ -945,12 +945,18 @@ static void event_command(const char *line, SERVER_REC *server, void *item)
parse_command(line, expand_aliases, server, item); parse_command(line, expand_aliases, server, item);
} }
static int eval_recursion_depth=0;
/* SYNTAX: EVAL <command(s)> */ /* SYNTAX: EVAL <command(s)> */
static void cmd_eval(const char *data, SERVER_REC *server, void *item) static void cmd_eval(const char *data, SERVER_REC *server, void *item)
{ {
g_return_if_fail(data != NULL); g_return_if_fail(data != NULL);
if (eval_recursion_depth > 100)
cmd_return_error(CMDERR_EVAL_MAX_RECURSE);
eval_recursion_depth++;
eval_special_string(data, "", server, item); eval_special_string(data, "", server, item);
eval_recursion_depth--;
} }
/* SYNTAX: CD <directory> */ /* SYNTAX: CD <directory> */

View File

@ -39,7 +39,8 @@ enum {
CMDERR_ILLEGAL_PROTO, /* requires different chat protocol than the active server */ CMDERR_ILLEGAL_PROTO, /* requires different chat protocol than the active server */
CMDERR_NOT_GOOD_IDEA, /* not good idea to do, -yes overrides this */ CMDERR_NOT_GOOD_IDEA, /* not good idea to do, -yes overrides this */
CMDERR_INVALID_TIME, /* invalid time specification */ CMDERR_INVALID_TIME, /* invalid time specification */
CMDERR_INVALID_CHARSET /* invalid charset specification */ CMDERR_INVALID_CHARSET, /* invalid charset specification */
CMDERR_EVAL_MAX_RECURSE /* eval hit recursion limit */
}; };
/* Return the full command for `alias' */ /* Return the full command for `alias' */

View File

@ -49,7 +49,8 @@ static int ret_texts[] = {
TXT_ILLEGAL_PROTO, TXT_ILLEGAL_PROTO,
TXT_NOT_GOOD_IDEA, TXT_NOT_GOOD_IDEA,
TXT_INVALID_TIME, TXT_INVALID_TIME,
TXT_INVALID_CHARSET TXT_INVALID_CHARSET,
TXT_EVAL_MAX_RECURSE
}; };
int command_hide_output; int command_hide_output;

View File

@ -219,6 +219,7 @@ FORMAT_REC fecommon_core_formats[] = {
{ "invalid_level", "Invalid message level", 0 }, { "invalid_level", "Invalid message level", 0 },
{ "invalid_size", "Invalid size", 0 }, { "invalid_size", "Invalid size", 0 },
{ "invalid_charset", "Invalid charset: $0", 1, { 0 } }, { "invalid_charset", "Invalid charset: $0", 1, { 0 } },
{ "eval_max_recurse", "/eval hit maximum recursion limit", 0 },
/* ---- */ /* ---- */
{ NULL, "Themes", 0 }, { NULL, "Themes", 0 },

View File

@ -188,6 +188,7 @@ enum {
TXT_INVALID_LEVEL, TXT_INVALID_LEVEL,
TXT_INVALID_SIZE, TXT_INVALID_SIZE,
TXT_INVALID_CHARSET, TXT_INVALID_CHARSET,
TXT_EVAL_MAX_RECURSE,
TXT_FILL_11, TXT_FILL_11,