DhcpServer: fixed minor issue caused by using incorrect client identifier while updating hostname in reserved leases.

This commit is contained in:
Shreyas Zare
2020-12-12 16:49:18 +05:30
parent d18a480dd2
commit 816f360a58
2 changed files with 7 additions and 2 deletions

View File

@@ -456,7 +456,7 @@ namespace DnsServerCore.Dhcp
//update hostname in reserved leases //update hostname in reserved leases
{ {
Lease reservedLease = scope.GetReservedLease(leaseOffer.ClientIdentifier); Lease reservedLease = scope.GetReservedLease(request);
if (reservedLease != null) if (reservedLease != null)
reservedLease.SetHostName(request.HostName?.HostName); reservedLease.SetHostName(request.HostName?.HostName);
} }

View File

@@ -588,7 +588,12 @@ namespace DnsServerCore.Dhcp
return GetReservedLease(new ClientIdentifierOption((byte)request.HardwareAddressType, request.ClientHardwareAddress)); return GetReservedLease(new ClientIdentifierOption((byte)request.HardwareAddressType, request.ClientHardwareAddress));
} }
internal Lease GetReservedLease(ClientIdentifierOption clientIdentifier) internal Lease GetReservedLease(DhcpMessageHardwareAddressType hardwareAddressType, byte[] identifier)
{
return GetReservedLease(new ClientIdentifierOption((byte)hardwareAddressType, identifier));
}
private Lease GetReservedLease(ClientIdentifierOption clientIdentifier)
{ {
if (_reservedLeases.TryGetValue(clientIdentifier, out Lease reservedLease)) if (_reservedLeases.TryGetValue(clientIdentifier, out Lease reservedLease))
{ {