Merge pull request #22414 from pprindeville/isc-dhcp-no-empty-ranges

isc-dhcp: refuse to add empty DHCP ranges
This commit is contained in:
Philip Prindeville 2023-10-15 22:14:24 -06:00 committed by GitHub
commit a6d6475cd5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -374,7 +374,9 @@ gen_dhcp_subnet() {
echo " range $START $END;"
fi
echo " option subnet-mask $netmask;"
if [ "$BROADCAST" != "0.0.0.0" ] ; then
# check for 0.0.0.0 until all active releases of ipcalc.sh omit it
# for small networks:
if [ -n "$BROADCAST" ] && [ "$BROADCAST" != "0.0.0.0" ] ; then
echo " option broadcast-address $BROADCAST;"
fi
if [ "$dynamicdhcp" -eq 0 ] ; then
@ -443,7 +445,10 @@ dhcpd_add() {
dhcp_ifs="$dhcp_ifs $ifname"
eval "$(ipcalc.sh $subnet $start $limit)"
if ! ipcalc "$subnet" "$start" "$limit"; then
echo "invalid range params: $subnet start: $start limit $limit" >&2
return 1
fi
config_get netmask "$cfg" "netmask" "$NETMASK"
config_get leasetime "$cfg" "leasetime"