diff options
author | Thomas Haller <thaller@redhat.com> | 2022-03-19 11:02:39 +0100 |
---|---|---|
committer | Thomas Haller <thaller@redhat.com> | 2022-03-19 11:04:11 +0100 |
commit | 625d70721dc5efbe4dea19030b16f1dc0a94646d (patch) | |
tree | e67774423bfb3eed50a63937fce21f828a142b48 | |
parent | b3192d2d46fac196c6eb91af0764f7a3b652b697 (diff) | |
download | NetworkManager-th/dnsmasq-shared-max-leases.tar.gz |
dnsmasq/shared: drop "--dhcp-lease-max=50" optionth/dnsmasq-shared-max-leases
It's not clear why this option is set or why it's necessary. It was
set ever since.
Note that we automatically determine the DHCP range
nm_dnsmasq_utils_get_range(), and with /24 subnets (or larger)
the range spans up to 244 addresses.
With such a configuration it doesn't seem to make sense to limit the
number of leases.
https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/issues/ ## 941
-rw-r--r-- | src/core/dnsmasq/nm-dnsmasq-manager.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/core/dnsmasq/nm-dnsmasq-manager.c b/src/core/dnsmasq/nm-dnsmasq-manager.c index 4ab91e7e33..c272a743af 100644 --- a/src/core/dnsmasq/nm-dnsmasq-manager.c +++ b/src/core/dnsmasq/nm-dnsmasq-manager.c @@ -188,8 +188,6 @@ create_dm_cmd_line(const char *iface, nm_strv_ptrarray_add_string_dup(cmd, "--dhcp-option-force=43,ANDROID_METERED"); } - nm_strv_ptrarray_add_string_dup(cmd, "--dhcp-lease-max=50"); - nm_strv_ptrarray_add_string_printf(cmd, "--dhcp-leasefile=%s/dnsmasq-%s.leases", NMSTATEDIR, |