From 4ea8f1c3ee204d257684d2613bec69ef2b297360 Mon Sep 17 00:00:00 2001
From: Yuta Iwama <ganmacs@gmail.com>
Date: Wed, 18 Dec 2019 18:12:43 +0900
Subject: [PATCH] fixup! Add StaticConfigAnalysis

Signed-off-by: Yuta Iwama <ganmacs@gmail.com>
---
 lib/fluent/static_config_analysis.rb |  4 ++++
 test/test_static_config_analysis.rb  | 13 +++++++++++++
 2 files changed, 17 insertions(+)

diff --git a/lib/fluent/static_config_analysis.rb b/lib/fluent/static_config_analysis.rb
index 692503b6af..2ed3efa06c 100644
--- a/lib/fluent/static_config_analysis.rb
+++ b/lib/fluent/static_config_analysis.rb
@@ -128,6 +128,10 @@ def static_label_analyse(conf)
       ret = []
       conf.elements(name: 'label').each do |e|
         name = e.arg
+        if name.empty?
+          raise ConfigError, 'Missing symbol argument on <label> directive'
+        end
+
         if @labels[name]
           raise ConfigError, "Section <label #{name}> appears twice"
         end
diff --git a/test/test_static_config_analysis.rb b/test/test_static_config_analysis.rb
index 5ae4398994..6c6bb87cea 100644
--- a/test/test_static_config_analysis.rb
+++ b/test/test_static_config_analysis.rb
@@ -117,6 +117,19 @@ class StaticConfigAnalysisTest < ::Test::Unit::TestCase
         end
       end
 
+      test 'empty label' do
+        conf_data = <<-CONF
+<label>
+</label>
+CONF
+
+        c = Fluent::Config.parse(conf_data, '(test)', '(test_dir)', true)
+        assert_raise(Fluent::ConfigError.new('Missing symbol argument on <label> directive')) do
+          Fluent::StaticConfigAnalysis.call(c, workers: 2)
+        end
+      end
+
+
       data(
         'in filter' => 'filter',
         'in source' => 'source',