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

@@ -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 */

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 **));