diff options
author | Mark Wells <mark@freeside.biz> | 2013-12-10 20:44:01 -0800 |
---|---|---|
committer | Mark Wells <mark@freeside.biz> | 2013-12-10 20:44:01 -0800 |
commit | 7cc8431906315822e8107143920bbca85e5b53ce (patch) | |
tree | 0f254ca8ef72c016e9ea0c79831d76bd03485a4b /FS/FS/addr_block.pm | |
parent | a4c1077430ac3b053c30084dcf76c54be45dca08 (diff) |
actually make auto-assignment avoid forbidden ranges, #25530
Diffstat (limited to 'FS/FS/addr_block.pm')
-rwxr-xr-x | FS/FS/addr_block.pm | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/FS/FS/addr_block.pm b/FS/FS/addr_block.pm index 6a62777..1f4000b 100755 --- a/FS/FS/addr_block.pm +++ b/FS/FS/addr_block.pm @@ -256,7 +256,17 @@ sub next_free_addr { # just do a linear search of the block my $freeaddr = $selfaddr->network + 1; while ( $freeaddr < $selfaddr->broadcast ) { - return $freeaddr unless $used{ $freeaddr->addr }; + # also make sure it's not blocked from assignment by an address range + if ( !$used{$freeaddr->addr } ) { + my ($range) = grep { !$_->allow_use } + FS::addr_range->any_contains($freeaddr); + if ( !$range ) { + # then we've found a free address + return $freeaddr; + } + # otherwise, skip to the end of the range + $freeaddr = NetAddr::IP->new($range->end, $self->ip_netmask); + } $freeaddr++; } return; |