luci-mod-admin-full: limit dns cachesize to 10000 1742/head
authorMarc Benoit <marcb62185@gmail.com>
Tue, 10 Apr 2018 00:47:32 +0000 (20:47 -0400)
committerMarc Benoit <marcb62185@gmail.com>
Tue, 10 Apr 2018 00:51:14 +0000 (20:51 -0400)
The value of cachesize is hardcoded to 10000 in
dnsmasq-2.79/src/option.c to 10000 max

    case 'c':  /* --cache-size */
      {
        int size;

        if (!atoi_check(arg, &size))
          ret_err(gen_err);
        else
          {
            /* zero is OK, and means no caching. */

            if (size < 0)
              size = 0;
            else if (size > 10000)
              size = 10000;

            daemon->cachesize = size;
          }
        break;
      }

Tested on Netgear R7800
Signed-off-by: Marc Benoit <marcb62185@gmail.com>
modules/luci-mod-admin-full/luasrc/model/cbi/admin_network/dhcp.lua

index d2984804081f8ab0ccaebdf6b52685c2d077f0f3..fbde431df8df7c5a67773adbd4ef43600095130e 100644 (file)
@@ -214,9 +214,9 @@ cq.placeholder = 150
 
 cs = s:taboption("advanced", Value, "cachesize",
        translate("Size of DNS query cache"),
-       translate("Maximum number of cached DNS entries"))
+       translate("Number of cached DNS entries (max is 10000, 0 is no caching)"))
 cs.optional = true
-cs.datatype = "uinteger"
+cs.datatype = "range(0,10000)"
 cs.placeholder = 150
 
 s:taboption("tftp", Flag, "enable_tftp",