storage_local: Fix incorrect validation clauses

fluent · Apr 22, 2016 · 6c138c9 · 6c138c9
1 parent b90d479
commit 6c138c9
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/fluent/plugin/storage_local.rb b/lib/fluent/plugin/storage_local.rb
@@ -26,7 +26,7 @@ def configure(conf)
         super
 
         @on_memory = false
-        if !@path && !@_plugin_id_configured
+        if !@path || !@_plugin_id_configured
           if @autosave || @persistent
             raise Fluent::ConfigError, "Plugin @id or path for <storage> required to save data"
           else
@@ -54,7 +54,7 @@ def configure(conf)
               raise Fluent::ConfigError, "Unexpected error: failed to read data from plugin storage file: '#{@path}'"
             end
           else
-            raise Fluent::ConfigError, "Directory is not writable for plugin storage file '#{@path}'" unless File.writable?(@path)
+            raise Fluent::ConfigError, "Directory is not writable for plugin storage file '#{dir}'" unless File.writable?(dir)
           end
         end
       end