Add extra-opts argument parsing with tests

git-svn-id: https://nagiosplug.svn.sourceforge.net/svnroot/nagiosplug/nagiosplug/trunk@1961 f882894a-f735-0410-b71e-b25c423dba1c

diff --git a/lib/extra_opts.c b/lib/extra_opts.c
new file mode 100644 (file)
index 0000000..3a0ce04
--- /dev/null
+++ b/lib/extra_opts.c
@@ -0,0 +1,155 @@
+/*****************************************************************************
+* 
+* Nagios-plugins extra_opts library
+* 
+* License: GPL
+* Copyright (c) 2007 Nagios Plugins Development Team
+* 
+* Last Modified: $Date: 2008-03-15 18:42:01 -0400 (Sat, 15 Mar 2008) $
+* 
+* 
+* This program is free software: you can redistribute it and/or modify
+* it under the terms of the GNU General Public License as published by
+* the Free Software Foundation, either version 3 of the License, or
+* (at your option) any later version.
+* 
+* This program is distributed in the hope that it will be useful,
+* but WITHOUT ANY WARRANTY; without even the implied warranty of
+* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+* GNU General Public License for more details.
+* 
+* You should have received a copy of the GNU General Public License
+* along with this program.  If not, see <http://www.gnu.org/licenses/>.
+* 
+* $Id: parse_ini.c 1950 2008-03-15 22:42:01Z dermoth $
+* 
+*****************************************************************************/
+
+#include "common.h"
+#include "extra_opts.h"
+#include "parse_ini.h"
+#include "utils_base.h"
+#include <ctype.h>
+
+/* FIXME: copied from utils.h; we should move a bunch of libs! */
+int
+is_option (char *str)
+{
+       if (!str)
+               return 0;
+       else if (strspn (str, "-") == 1 || strspn (str, "-") == 2)
+               return 1;
+       else
+               return 0;
+}
+
+/* this is the externally visible function used by plugins */
+/* Shouldn't se modify directly **argv (passed as a char ***) and argc
+ * (as int *) ?
+ */
+char **np_extra_opts(int argc, char **argv, const char *plugin_name, int *argc_new){
+       np_arg_list *extra_args=NULL, *ea_tmp1=NULL, *ea_tmp2=NULL;
+       char **argv_new=NULL;
+       char *argptr=NULL;
+       int i, j, optfound, ea_num=argc;
+
+       if(argc<2) {
+               /* No arguments provided */
+               *argc_new=argc;
+               argv_new=argv;
+               return argv_new;
+       }
+
+       for(i=1; i<argc; i++){
+               argptr=NULL;
+               optfound=0;
+
+               /* Do we have an extra-opts parameter? */
+               if(strncmp(argv[i], "--extra-opts=", 13)==0){
+                       /* It is a single argument with value */
+                       argptr=argv[i]+13;
+                       /* Delete the extra opts argument */
+                       for(j=i;j<argc;j++) argv[j]=argv[j+1];
+                       i--;
+                       argc--;
+               }else if(strcmp(argv[i], "--extra-opts")==0){
+                       if(!is_option(argv[i+1])){
+                               /* It is a argument with separate value */
+                               argptr=argv[i+1];
+                               /* Delete the extra-opts argument/value */
+                               for(j=i;j<argc-1;j++) argv[j]=argv[j+2];
+                               i-=2;
+                               argc-=2;
+                               ea_num--;
+                       }else{
+                               /* It has no value */
+                               optfound=1;
+                               /* Delete the extra opts argument */
+                               for(j=i;j<argc;j++) argv[j]=argv[j+1];
+                               i--;
+                               argc--;
+                       }
+               }
+
+               if(argptr||optfound){
+                       /* Process ini section, returning a linked list of arguments */
+                       ea_tmp1=np_get_defaults(argptr, plugin_name);
+                       if(ea_tmp1==NULL) {
+                               /* no extra args? */
+                               ea_num--;
+                               continue;
+                       }
+
+                       /* append the list to extra_args */
+                       if(extra_args==NULL){
+                               extra_args=ea_tmp2=ea_tmp1;
+                               while(ea_tmp2->next) {
+                                       ea_tmp2=ea_tmp2->next;
+                                       ea_num++;
+                               }
+                       }else{
+                               ea_tmp2=extra_args;
+                               while(ea_tmp2->next) {
+                                       ea_tmp2=ea_tmp2->next;
+                                       ea_num++;
+                               }
+                               ea_tmp2->next=ea_tmp1;
+                       }
+                       ea_tmp1=ea_tmp2=NULL;
+               }
+               /* lather, rince, repeat */
+       }
+
+       if(ea_num==argc && extra_args==NULL){
+               /* No extra-opts */
+               *argc_new=argc;
+               argv_new=argv;
+               return argv_new;
+       }
+
+       /* done processing arguments. now create a new argc/argv set... */
+       argv_new=(char**)malloc((ea_num+1)*sizeof(char**));
+       if(argv_new==NULL) die(STATE_UNKNOWN, _("malloc() failed!\n"));
+
+       /* starting with program name (Should we strdup or just use the poiter?) */
+       argv_new[0]=strdup(argv[0]);
+       *argc_new=1;
+       /* then parsed ini opts (frying them up in the same run) */
+       while(extra_args){
+               argv_new[*argc_new]=strdup(extra_args->arg);
+               *argc_new+=1;
+               ea_tmp1=extra_args;
+               extra_args=extra_args->next;
+               free(ea_tmp1);
+       }
+       /* finally the rest of the argv array (Should we strdup or just use the poiter?) */
+       for (i=1; i<argc; i++){
+               argv_new[*argc_new]=strdup(argv[i]);
+               *argc_new+=1;
+       }
+       /* and terminate. */
+       argv_new[*argc_new]=NULL;
+
+       return argv_new;
+}
+

diff --git a/lib/extra_opts.h b/lib/extra_opts.h
new file mode 100644 (file)
index 0000000..5f89d2b
--- /dev/null
+++ b/lib/extra_opts.h
@@ -0,0 +1,15 @@
+#ifndef _EXTRA_OPTS_H_
+#define _EXTRA_OPTS_H_
+
+/*
+ * extra_opts.h: routines for loading nagios-plugin defaults from ini
+ * configuration files.
+ */
+
+/* np_extra_opts: Process the --extra-opts arguments and create a new argument
+ * array load the default configuration (if present) for
+ * a plugin from the ini file
+ */
+char **np_extra_opts(int argc, char **argv, const char *plugin_name, int *argc_new);
+
+#endif /* _EXTRA_OPTS_H_ */

diff --git a/lib/tests/config-opts.ini b/lib/tests/config-opts.ini
new file mode 100644 (file)
index 0000000..5c93572
--- /dev/null
+++ b/lib/tests/config-opts.ini
@@ -0,0 +1,14 @@
+# This config file is for testing test_opts
+
+[sect1]
+one=two
+
+[check_disk]
+foo=Bar
+this=Your Mother!
+blank=
+
+[sect2]
+something else=oops
+this=that
+

diff --git a/lib/tests/test_opts.c b/lib/tests/test_opts.c
new file mode 100644 (file)
index 0000000..1272605
--- /dev/null
+++ b/lib/tests/test_opts.c
@@ -0,0 +1,208 @@
+/*****************************************************************************
+* 
+* This program is free software: you can redistribute it and/or modify
+* it under the terms of the GNU General Public License as published by
+* the Free Software Foundation, either version 3 of the License, or
+* (at your option) any later version.
+* 
+* This program is distributed in the hope that it will be useful,
+* but WITHOUT ANY WARRANTY; without even the implied warranty of
+* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+* GNU General Public License for more details.
+* 
+* You should have received a copy of the GNU General Public License
+* along with this program.  If not, see <http://www.gnu.org/licenses/>.
+* 
+* $Id: test_ini.c 1951 2008-03-16 18:10:47Z dermoth $
+* 
+*****************************************************************************/
+
+#include "common.h"
+#include "extra_opts.h"
+#include "utils_base.h"
+
+#include "tap.h"
+
+void my_free(int *argc, char **argv) {
+       int i;
+       printf ("    Arg(%i): ", *argc);
+       for (i=1; i<*argc; i++) printf ("'%s' ", argv[i]);
+       printf ("\n");
+       free(argv);
+       *argc=0;
+}
+
+int array_diff(int i1, char **a1, int i2, char **a2) {
+       int i;
+
+       if (i1 != i2) {
+               printf("    Argument count doesn't match!\n");
+               return 0;
+       }
+       for (i=0; i<=i1; i++) {
+               if (a1[i]==NULL && a2[i]==NULL) continue;
+               if (a1[i]==NULL || a2[i]==NULL) {
+                       printf("    Argument # %i null in one array!\n", i);
+                       return 0;
+               }
+               if (strcmp(a1[i], a2[i])) {
+                       printf("    Argument # %i doesn't match!\n", i);
+                       return 0;
+               }
+       }
+       return 1;
+}
+
+int
+main (int argc, char **argv)
+{
+       char **argv_test=NULL, **argv_known=NULL;
+       int i, argc_test, argc_new;
+
+       plan_tests(8);
+
+       argv_test=(char **)malloc(2*sizeof(char **));
+       argv_test[0] = "prog_name";
+       argv_test[1] = NULL;
+       argc_test=1;
+       argv_known=(char **)realloc(argv_known, 2*sizeof(char **));
+       argv_known[0] = "prog_name";
+       argv_known[1] = NULL;
+       argv_test=np_extra_opts(argc_test, argv_test, "check_disk", &argc_new);
+       ok(array_diff(argc_new, argv_test, 1, argv_known), "No opts, returns correct argv/argc");
+       my_free(&argc_new, argv_test);
+
+       argv_test=(char **)malloc(6*sizeof(char **));
+       argv_test[0] = "prog_name";
+       argv_test[1] = "arg1";
+       argv_test[2] = "--arg2=val1";
+       argv_test[3] = "--arg3";
+       argv_test[4] = "val2";
+       argv_test[5] = NULL;
+       argc_test=5;
+       argv_known=(char **)realloc(argv_known, 6*sizeof(char **));
+       argv_known[0] = "prog_name";
+       argv_known[1] = "arg1";
+       argv_known[2] = "--arg2=val1";
+       argv_known[3] = "--arg3";
+       argv_known[4] = "val2";
+       argv_known[5] = NULL;
+       argv_test=np_extra_opts(argc_test, argv_test, "check_disk", &argc_new);
+       ok(array_diff(argc_new, argv_test, 5, argv_known), "No extra opts, verbatim copy of argv");
+       my_free(&argc_new,argv_test);
+
+       argv_test=(char **)malloc(3*sizeof(char **));
+       argv_test[0] = "prog_name";
+       argv_test[1] = "--extra-opts=@./config-opts.ini";
+       argv_test[2] = NULL;
+       argc_test=2;
+       argv_known=(char **)realloc(argv_known, 5*sizeof(char **));
+       argv_known[0] = "prog_name";
+       argv_known[1] = "--foo=Bar";
+       argv_known[2] = "--this=Your Mother!";
+       argv_known[3] = "--blank";
+       argv_known[4] = NULL;
+       argv_test=np_extra_opts(argc_test, argv_test, "check_disk", &argc_new);
+       ok(array_diff(argc_new, argv_test, 4, argv_known), "Only extra opts using default section");
+       my_free(&argc_new,argv_test);
+
+       argv_test=(char **)malloc(5*sizeof(char **));
+       argv_test[0] = "prog_name";
+       argv_test[1] = "--extra-opts=sect1@./config-opts.ini";
+       argv_test[2] = "--extra-opts";
+       argv_test[3] = "sect2@./config-opts.ini";
+       argv_test[4] = NULL;
+       argc_test=4;
+       argv_known=(char **)realloc(argv_known, 5*sizeof(char **));
+       argv_known[0] = "prog_name";
+       argv_known[1] = "--one=two";
+       argv_known[2] = "--something else=oops";
+       argv_known[3] = "--this=that";
+       argv_known[4] = NULL;
+       argv_test=np_extra_opts(argc_test, argv_test, "check_disk", &argc_new);
+       ok(array_diff(argc_new, argv_test, 4, argv_known), "Only extra opts specified twice");
+       my_free(&argc_new,argv_test);
+
+       argv_test=(char **)malloc(7*sizeof(char **));
+       argv_test[0] = "prog_name";
+       argv_test[1] = "--arg1=val1";
+       argv_test[2] = "--extra-opts=@./config-opts.ini";
+       argv_test[3] = "--extra-opts";
+       argv_test[4] = "sect1@./config-opts.ini";
+       argv_test[5] = "--arg2";
+       argv_test[6] = NULL;
+       argc_test=6;
+       argv_known=(char **)realloc(argv_known, 8*sizeof(char **));
+       argv_known[0] = "prog_name";
+       argv_known[1] = "--foo=Bar";
+       argv_known[2] = "--this=Your Mother!";
+       argv_known[3] = "--blank";
+       argv_known[4] = "--one=two";
+       argv_known[5] = "--arg1=val1";
+       argv_known[6] = "--arg2";
+       argv_known[7] = NULL;
+       argv_test=np_extra_opts(argc_test, argv_test, "check_disk", &argc_new);
+       ok(array_diff(argc_new, argv_test, 7, argv_known), "twice extra opts using two sections");
+       my_free(&argc_new,argv_test);
+
+       /* Next three checks should die according to N::P - for now they're useful
+        * to test code is working properly (i.e. no srash or unexpected behavior)
+        */
+       argv_test=(char **)malloc(6*sizeof(char **));
+       argv_test[0] = "prog_name";
+       argv_test[1] = "arg1";
+       argv_test[2] = "--extra-opts=missing@./config-opts.ini";
+       argv_test[3] = "--arg3";
+       argv_test[4] = "val2";
+       argv_test[5] = NULL;
+       argc_test=5;
+       argv_known=(char **)realloc(argv_known, 5*sizeof(char **));
+       argv_known[0] = "prog_name";
+       argv_known[1] = "arg1";
+       argv_known[2] = "--arg3";
+       argv_known[3] = "val2";
+       argv_known[4] = NULL;
+       argv_test=np_extra_opts(argc_test, argv_test, "check_missing", &argc_new);
+       ok(array_diff(argc_new, argv_test, 4, argv_known), "Missing section 1");
+       my_free(&argc_new,argv_test);
+
+       argv_test=(char **)malloc(7*sizeof(char **));
+       argv_test[0] = "prog_name";
+       argv_test[1] = "arg1";
+       argv_test[2] = "--extra-opts";
+       argv_test[3] = "missing@./config-opts.ini";
+       argv_test[4] = "--arg3";
+       argv_test[5] = "val2";
+       argv_test[6] = NULL;
+       argc_test=6;
+       argv_known=(char **)realloc(argv_known, 5*sizeof(char **));
+       argv_known[0] = "prog_name";
+       argv_known[1] = "arg1";
+       argv_known[2] = "--arg3";
+       argv_known[3] = "val2";
+       argv_known[4] = NULL;
+       argv_test=np_extra_opts(argc_test, argv_test, "check_missing", &argc_new);
+       ok(array_diff(argc_new, argv_test, 4, argv_known), "Missing section 2");
+       my_free(&argc_new,argv_test);
+
+       argv_test=(char **)malloc(6*sizeof(char **));
+       argv_test[0] = "prog_name";
+       argv_test[1] = "arg1";
+       argv_test[2] = "--extra-opts";
+       argv_test[3] = "--arg3";
+       argv_test[4] = "val2";
+       argv_test[5] = NULL;
+       argc_test=5;
+       argv_known=(char **)realloc(argv_known, 5*sizeof(char **));
+       argv_known[0] = "prog_name";
+       argv_known[1] = "arg1";
+       argv_known[2] = "--arg3";
+       argv_known[3] = "val2";
+       argv_known[4] = NULL;
+       argv_test=np_extra_opts(argc_test, argv_test, "check_missing", &argc_new);
+       ok(array_diff(argc_new, argv_test, 4, argv_known), "Missing section 3");
+       my_free(&argc_new,argv_test);
+
+       return exit_status();
+}
+

diff --git a/lib/tests/test_opts.t b/lib/tests/test_opts.t
new file mode 100755 (executable)
index 0000000..86a5b21
--- /dev/null
+++ b/lib/tests/test_opts.t
@@ -0,0 +1,6 @@
+#!/usr/bin/perl
+use Test::More;
+if (! -e "./test_opts") {
+       plan skip_all => "./test_opts not compiled - please install tap library to test";
+}
+exec "./test_opts";