summary refs log tree commit diff
path: root/Src
diff options
context:
space:
mode:
authorClint Adams <clint@users.sourceforge.net>2007-10-21 21:16:07 +0000
committerClint Adams <clint@users.sourceforge.net>2007-10-21 21:16:07 +0000
commit526f86ec15e12405dfda016cf2bf59dab093d8fa (patch)
treeda1ea8e5b8169fc23aa1a474bfb80fc314085984 /Src
parent52ea35c1c6852dd8bacd218e32547afd99c23ba9 (diff)
downloadzsh-526f86ec15e12405dfda016cf2bf59dab093d8fa.tar.gz
zsh-526f86ec15e12405dfda016cf2bf59dab093d8fa.tar.xz
zsh-526f86ec15e12405dfda016cf2bf59dab093d8fa.zip
24005: enforce min/max arguments on zcurses commands.
Diffstat (limited to 'Src')
-rw-r--r--Src/Modules/curses.c10
1 files changed, 8 insertions, 2 deletions
diff --git a/Src/Modules/curses.c b/Src/Modules/curses.c
index aba8028e5..4103d853c 100644
--- a/Src/Modules/curses.c
+++ b/Src/Modules/curses.c
@@ -281,7 +281,7 @@ bin_zcurses(char *nam, char **args, Options ops, UNUSED(int func))
 {
     char **saargs;
     struct zcurses_subcommand *zcsc;
-    int sc;
+    int sc, num_args;
 
     struct zcurses_subcommand scs[] = {
 	{{"init", ZCURSES_SC_INIT}, 0, 0},
@@ -310,7 +310,13 @@ bin_zcurses(char *nam, char **args, Options ops, UNUSED(int func))
 	return 1;
     }
 
-    /* here would be a good place to validate number of args */
+    saargs = args;
+    while (*saargs++);
+    num_args = saargs - (args + 2);
+
+    if (num_args < zcsc->minargs || num_args > zcsc->maxargs)
+	return 1;
+
     saargs = args + 1;
 
     /* Initialise curses */