From 1a0ed0d6bb5ffa0991a91cd149d3ecaacb11e59f Mon Sep 17 00:00:00 2001 From: Thomas Guyot-Sionnest Date: Sat, 15 Mar 2008 20:47:43 +0000 Subject: - Make equal-less parameters illegal - parameters without argument after '=' are now assumed to be argument-less - Add a testcase for space in stanza and various argument-less parameters git-svn-id: https://nagiosplug.svn.sourceforge.net/svnroot/nagiosplug/nagiosplug/trunk@1948 f882894a-f735-0410-b71e-b25c423dba1c --- lib/parse_ini.c | 5 ++++- lib/tests/plugin.ini | 5 +++++ lib/tests/test_ini.c | 12 ++++++++---- 3 files changed, 17 insertions(+), 5 deletions(-) (limited to 'lib') diff --git a/lib/parse_ini.c b/lib/parse_ini.c index 38bcb39b..ade77e5f 100644 --- a/lib/parse_ini.c +++ b/lib/parse_ini.c @@ -212,6 +212,7 @@ static int add_option(FILE *f, np_arg_list **optlst){ else optend=NULL; } if(optend==NULL) optend=eqptr; +//printf("o1: %c\n", *optptr[optend]); --optend; /* ^[[:space:]]*=foo is a syntax error */ if(optptr==eqptr) die(STATE_UNKNOWN, _("Config file error\n")); @@ -242,6 +243,8 @@ static int add_option(FILE *f, np_arg_list **optlst){ equals=1; cfg_len+=1; } + /* A line with no equal sign isn't valid */ + if(equals==0) die(STATE_UNKNOWN, _("Config file error\n")); /* okay, now we have all the info we need, so we create a new np_arg_list * element and set the argument... @@ -260,8 +263,8 @@ static int add_option(FILE *f, np_arg_list **optlst){ read_pos+=2; } strncpy(&optnew->arg[read_pos], optptr, opt_len); read_pos+=opt_len; - if(equals) optnew->arg[read_pos++]='='; if(value) { + optnew->arg[read_pos++]='='; strncpy(&optnew->arg[read_pos], valptr, val_len); read_pos+=val_len; } optnew->arg[read_pos]='\0'; diff --git a/lib/tests/plugin.ini b/lib/tests/plugin.ini index eb869c63..d07fc4f3 100644 --- a/lib/tests/plugin.ini +++ b/lib/tests/plugin.ini @@ -7,4 +7,9 @@ password=secret # Remember to change later u=admin p=secret +[check space_and_flags] +foo=bar +a= +b= +bar= diff --git a/lib/tests/test_ini.c b/lib/tests/test_ini.c index de9f8adf..9031f7f8 100644 --- a/lib/tests/test_ini.c +++ b/lib/tests/test_ini.c @@ -52,18 +52,18 @@ main (int argc, char **argv) { char *optstr=NULL; - plan_tests(9); + plan_tests(10); optstr=list2str(np_get_defaults("section@./config-tiny.ini", "check_disk")); - ok( !strcmp(optstr, "--one=two --Foo=Bar --this=Your Mother! --blank="), "config-tiny.ini's section as expected"); + ok( !strcmp(optstr, "--one=two --Foo=Bar --this=Your Mother! --blank"), "config-tiny.ini's section as expected"); my_free(optstr); optstr=list2str(np_get_defaults("@./config-tiny.ini", "section")); - ok( !strcmp(optstr, "--one=two --Foo=Bar --this=Your Mother! --blank="), "Used default section name, without specific"); + ok( !strcmp(optstr, "--one=two --Foo=Bar --this=Your Mother! --blank"), "Used default section name, without specific"); my_free(optstr); optstr=list2str(np_get_defaults("section_unknown@./config-tiny.ini", "section")); - ok( !strcmp(optstr, "--one=two --Foo=Bar --this=Your Mother! --blank="), "Used default section name over specified one"); + ok( !strcmp(optstr, "--one=two --Foo=Bar --this=Your Mother! --blank"), "Used default section name over specified one"); my_free(optstr); optstr=list2str(np_get_defaults("Section Two@./config-tiny.ini", "check_disk")); @@ -90,6 +90,10 @@ main (int argc, char **argv) ok( !strcmp(optstr, "-u=admin -p=secret"), "plugin.ini's check_mysql2 as expected"); my_free(optstr); + optstr=list2str(np_get_defaults("check space_and_flags@./plugin.ini", "check_disk")); + ok( !strcmp(optstr, "--foo=bar -a -b --bar"), "plugin.ini space in stanza and flag arguments"); + my_free(optstr); + return exit_status(); } -- cgit v1.2.3-74-g34f1