Code

perl plugin: Fixed handling of erroneous configuration options.
authorSebastian Harl <sh@tokkee.org>
Tue, 16 Oct 2007 23:14:18 +0000 (01:14 +0200)
committerFlorian Forster <octo@huhu.verplant.org>
Wed, 24 Oct 2007 18:03:32 +0000 (20:03 +0200)
An empty block was used instead of returning from the perl_config_*()
functions in case of an error. I wonder how that has happened... :-/

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>
src/perl.c

index c0beabfe51ca0ffd8f1b1c9f4810d0bf2e61573c..47844eb95420ec2f00e32ed9df6e6c583b845887 100644 (file)
@@ -907,8 +907,8 @@ static int perl_config_loadplugin (oconfig_item_t *ci)
        char *value = NULL;
 
        if ((0 != ci->children_num) || (1 != ci->values_num)
-                       || (OCONFIG_TYPE_STRING != ci->values[0].type)) {
-       }
+                       || (OCONFIG_TYPE_STRING != ci->values[0].type))
+               return 1;
 
        value = ci->values[0].value.string;
 
@@ -934,8 +934,8 @@ static int perl_config_basename (oconfig_item_t *ci)
        char *value = NULL;
 
        if ((0 != ci->children_num) || (1 != ci->values_num)
-                       || (OCONFIG_TYPE_STRING != ci->values[0].type)) {
-       }
+                       || (OCONFIG_TYPE_STRING != ci->values[0].type))
+               return 1;
 
        value = ci->values[0].value.string;
 
@@ -953,8 +953,8 @@ static int perl_config_enabledebugger (oconfig_item_t *ci)
        char *value = NULL;
 
        if ((0 != ci->children_num) || (1 != ci->values_num)
-                       || (OCONFIG_TYPE_STRING != ci->values[0].type)) {
-       }
+                       || (OCONFIG_TYPE_STRING != ci->values[0].type))
+               return 1;
 
        value = ci->values[0].value.string;
 
@@ -987,8 +987,8 @@ static int perl_config_includedir (oconfig_item_t *ci)
        char *value = NULL;
 
        if ((0 != ci->children_num) || (1 != ci->values_num)
-                       || (OCONFIG_TYPE_STRING != ci->values[0].type)) {
-       }
+                       || (OCONFIG_TYPE_STRING != ci->values[0].type))
+               return 1;
 
        value = ci->values[0].value.string;