getAction(n) for n >= number of rules now properly returns nil instead of crashing

diff --git a/pdns/dnsdist-lua2.cc b/pdns/dnsdist-lua2.cc
index c21fe1899c0bbd92c559d3385d00def37206f045..cb641be16809cabd8048bf19e1a8275fbd709813 100644 (file)
--- a/pdns/dnsdist-lua2.cc
+++ b/pdns/dnsdist-lua2.cc
@@ -602,10 +602,11 @@ void moreLua(bool client)
 
     g_lua.writeFunction("getAction", [](unsigned int num) {
         setLuaNoSideEffect();
+        boost::optional<std::shared_ptr<DNSAction>> ret;
         auto rulactions = g_rulactions.getCopy();
-        if(num >= rulactions.size())
-          return std::shared_ptr<DNSAction>();
-        return rulactions[num].second;
+        if(num < rulactions.size())
+          ret=rulactions[num].second;
+        return ret;
       });
 
     g_lua.registerFunction("getStats", &DNSAction::getStats);