config: minor fixes in configure_plugins_type

Signed-off-by: Bradley Laney <[email protected]>

Author: stoksc

src/flb_config.c

@@ -795,7 +795,6 @@ static int configure_plugins_type(struct flb_config *config, struct flb_cf *cf,
             flb_sds_destroy(name);
             return -1;
         }
-        flb_sds_destroy(name);
 
         /*
          * iterate section properties and populate instance by using specific
@@ -857,6 +856,8 @@ static int configure_plugins_type(struct flb_config *config, struct flb_cf *cf,
                 flb_error("[config] could not configure property '%s' on "
                           "%s plugin with section name '%s'",
                           kv->key, s_type, name);
+                flb_sds_destroy(name);
+                return -1;
             }
         }
 
@@ -873,6 +874,8 @@ static int configure_plugins_type(struct flb_config *config, struct flb_cf *cf,
                 flb_error("[config] section '%s' does not support processors", s_type);
             }
         }
+
+        flb_sds_destroy(name);
     }
 
     return 0;

tests/internal/config_format_yaml.c

@@ -833,6 +833,27 @@ static void test_upstream_servers()
     flb_cf_destroy(cf);
 }
 
+static void test_invalid_property()
+{
+    struct flb_cf *cf;
+    struct flb_config *config;
+    int ret;
+    int i;
+    char* file_path = FLB_TESTS_CONF_PATH "/invalid_input_property.yaml";
+
+    cf = flb_cf_yaml_create(NULL, file_path, NULL, 0);
+    TEST_ASSERT(cf != NULL);
+
+    config = flb_config_init();
+    TEST_ASSERT(config != NULL);
+    
+    ret = flb_config_load_config_format(config, cf);
+    TEST_ASSERT_(ret == -1, "expected invalid property to return an error in file %s", file_path);
+
+    flb_cf_destroy(cf);
+    flb_config_exit(config);
+}
+
 TEST_LIST = {
     { "basic"    , test_basic},
     { "customs section", test_customs_section},
@@ -846,5 +867,6 @@ TEST_LIST = {
     { "stream_processor", test_stream_processor},
     { "plugins", test_plugins},
     { "upstream_servers", test_upstream_servers},
+    { "invalid_input_property", test_invalid_property},
     { 0 }
 };

tests/internal/data/config_format/yaml/invalid_input_property.yaml

@@ -0,0 +1,4 @@
+pipeline:
+  inputs:
+    - name: dummy
+      log_level: thisdoesnotexist

tests/internal/data/config_format/yaml/invalid_output_property.yaml

@@ -0,0 +1,5 @@
+pipeline:
+  outputs:
+    - name: stdout
+      match: '*'
+      log_level: thisdoesnotexist