From 3011838fe9796f24ad3d197057890042b527ef9d Mon Sep 17 00:00:00 2001 From: Thomas Guyot-Sionnest Date: Wed, 5 Nov 2008 16:15:59 +0000 Subject: [PATCH] Fix calculation of extra-opts parameters. By reading the current list instead of the additional one when counting parameters, extra-opts calculation was allocating insuficient space for the final array if later sections contained more arguments than the sum of all previously parsed sections. git-svn-id: https://nagiosplug.svn.sourceforge.net/svnroot/nagiosplug/nagiosplug/trunk@2069 f882894a-f735-0410-b71e-b25c423dba1c --- lib/extra_opts.c | 5 ++--- lib/tests/test_opts.c | 2 +- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/lib/extra_opts.c b/lib/extra_opts.c index b555417..7e50c17 100644 --- a/lib/extra_opts.c +++ b/lib/extra_opts.c @@ -103,14 +103,13 @@ char **np_extra_opts(int *argc, char **argv, const char *plugin_name){ ea_tmp=extra_args; while(ea_tmp->next) { ea_tmp=ea_tmp->next; - ea_num++; } ea_tmp->next=ea1; + while(ea1=ea1->next) ea_num++; } ea1=ea_tmp=NULL; } - /* lather, rince, repeat */ - } + } /* lather, rince, repeat */ if(ea_num==*argc && extra_args==NULL){ /* No extra-opts */ diff --git a/lib/tests/test_opts.c b/lib/tests/test_opts.c index b9696df..6d6167e 100644 --- a/lib/tests/test_opts.c +++ b/lib/tests/test_opts.c @@ -276,7 +276,7 @@ main (int argc, char **argv) argv_test=(char **)malloc(4*sizeof(char **)); argv_test[0] = "check_tcp"; argv_test[1] = "--extra-opts"; - argv_test[2] = "--extra-opts=tcp_long_lines@plugins.ini"; + argv_test[2] = "--extra-opts=tcp_long_lines"; argv_test[3] = NULL; argc_test=3; argv_known=(char **)realloc(argv_known, 7*sizeof(char **)); -- 2.30.2