mir/kea
2
0
Fork 0
mirror of https://gitlab.isc.org/isc-projects/kea synced 2025-08-31 22:15:23 +00:00
Code Issues Releases Wiki Activity

[#1040] address review

Browse Source
This commit is contained in:
Razvan Becheriu
2020-01-24 14:11:50 +02:00
parent 59ecbbb61a
commit f3cc85ce2e
1 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/lib/dhcpsrv/alloc_engine.cc
View File

@@ -1475,6 +1475,8 @@ AllocEngine::removeNonmatchingReservedLeases6(ClientContext6& ctx,
// Remove this lease from LeaseMgr as it is reserved to someone // Remove this lease from LeaseMgr as it is reserved to someone
// else or doesn't belong to a pool. // else or doesn't belong to a pool.
if (!LeaseMgrFactory::instance().deleteLease(candidate)) { if (!LeaseMgrFactory::instance().deleteLease(candidate)) {
// Concurrent delete performed by other instance which should
// properly handle dns and stats updates.
continue; continue;
} }
@@ -1520,6 +1522,8 @@ AllocEngine::removeNonmatchingReservedNoHostLeases6(ClientContext6& ctx,
// Remove this lease from LeaseMgr as it doesn't belong to a pool. // Remove this lease from LeaseMgr as it doesn't belong to a pool.
if (!LeaseMgrFactory::instance().deleteLease(candidate)) { if (!LeaseMgrFactory::instance().deleteLease(candidate)) {
// Concurrent delete performed by other instance which should
// properly handle dns and stats updates.
continue; continue;
} }
@@ -1589,6 +1593,8 @@ AllocEngine::removeNonreservedLeases6(ClientContext6& ctx,
// We have reservations, but not for this lease. Release it. // We have reservations, but not for this lease. Release it.
// Remove this lease from LeaseMgr // Remove this lease from LeaseMgr
if (!LeaseMgrFactory::instance().deleteLease(*lease)) { if (!LeaseMgrFactory::instance().deleteLease(*lease)) {
// Concurrent delete performed by other instance which should
// properly handle dns and stats updates.
continue; continue;
} }
@@ -1991,6 +1997,8 @@ AllocEngine::extendLease6(ClientContext6& ctx, Lease6Ptr lease) {
// Remove this lease from LeaseMgr // Remove this lease from LeaseMgr
if (!LeaseMgrFactory::instance().deleteLease(lease)) { if (!LeaseMgrFactory::instance().deleteLease(lease)) {
// Concurrent delete performed by other instance which should
// properly handle dns and stats updates.
return; return;
} }