From e41482fec418457be9288ca5ce9df2099079c6da Mon Sep 17 00:00:00 2001 From: Andreas Schneider Date: Sun, 2 Jun 2013 18:34:39 +0200 Subject: [PATCH] opts: Fix segfault in option parser. --- src/options.c | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/src/options.c b/src/options.c index c43e41d4..931cb31e 100644 --- a/src/options.c +++ b/src/options.c @@ -931,7 +931,7 @@ int ssh_options_getopt(ssh_session session, int *argcptr, char **argv) { char *cipher = NULL; char *identity = NULL; char *port = NULL; - char **save = NULL; + char **save = NULL, **tmp; int i = 0; int argc = *argcptr; int debuglevel = 0; @@ -990,7 +990,6 @@ int ssh_options_getopt(ssh_session session, int *argcptr, char **argv) { break; default: { - char **tmp; char opt[3]="- "; opt[1] = optopt; tmp = realloc(save, (current + 1) * sizeof(char*)); @@ -1015,7 +1014,16 @@ int ssh_options_getopt(ssh_session session, int *argcptr, char **argv) { } /* while */ opterr = saveopterr; while (optind < argc) { - save[current++] = argv[optind++]; + tmp = realloc(save, (current + 1) * sizeof(char*)); + if (tmp == NULL) { + SAFE_FREE(save); + ssh_set_error_oom(session); + return -1; + } + save = tmp; + save[current] = argv[optind]; + current++; + optind++; } if (usersa && usedss) {