]> source.dussan.org Git - rspamd.git/commitdiff
[Minor] logical error, get_variable() accepts either one or two arguments 2506/head
authorMikhail Galanin <mgalanin@mimecast.com>
Thu, 20 Sep 2018 08:41:36 +0000 (09:41 +0100)
committerMikhail Galanin <mgalanin@mimecast.com>
Thu, 20 Sep 2018 08:41:36 +0000 (09:41 +0100)
When second argument is nil it still considered as two arguments call, however 2nd one is expected to be string.

lualib/lua_selectors.lua

index d93b11df6c3e47703d93389c8ebb2d8f74c75919..983f28589c2e74f4780856ae8dd1681af2d79b5f 100644 (file)
@@ -244,7 +244,8 @@ e.g. `get_user`]],
   -- the second argument is optional and defines the type (string by default)
   ['pool_var'] = {
     ['get_value'] = function(task, args)
-      return task:get_mempool():get_variable(args[1], args[2]),(args[2] or 'string')
+      local type = args[2] or 'string'
+      return task:get_mempool():get_variable(args[1], type),(type)
     end,
     ['description'] = [[Get specific pool var. The first argument must be variable name,
 the second argument is optional and defines the type (string by default)]],