tyler.durden/audk
1
0
Fork 0
mirror of https://github.com/acidanthera/audk.git synced 2025-07-30 00:54:06 +02:00
Code Issues Packages Projects Releases Wiki Activity

MdeModulePkg: Update IP4 stack to support point-to-point link with 31-bit mask.

Browse Source 
V2 update: Directly use NetIp4IsUnicast() to check station address in IP driver.

This patch is to follow RFC3021 which allows to use 31-bit mask
in point-to-point link.
If a 31-bit subnet mask is assigned to a point-to-point link, it
leaves the <Host-number> with only 1 bit.  Consequently, only two
possible addresses may result:
  {<Network-number>, 0} and {<Network-number>, -1}
These addresses have historically been associated with network and
broadcast addresses (see Section 2.2).  In a point-to-point link with
a 31-bit subnet mask, the two addresses above MUST be interpreted as
host addresses.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Fu Siyuan <siyuan.fu@intel.com>
Reviewed-by: Wu Jiaxin <jiaxin.wu@intel.com>
This commit is contained in:
Fu Siyuan 2017-10-18 10:13:39 +08:00
parent db04b706b7
commit 29788f178e
3 changed files with 18 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
MdeModulePkg/Include/Library/NetLib.h
View File

@ -414,8 +414,10 @@ NetGetIpClass (
ASSERT if NetMask is zero. ASSERT if NetMask is zero.
If all bits of the host address of IP are 0 or 1, IP is also not a valid unicast address. If all bits of the host address of IP are 0 or 1, IP is also not a valid unicast address,
except when the originator is one of the endpoints of a point-to-point link with a 31-bit
mask (RFC3021).
@param[in] Ip The IP to check against. @param[in] Ip The IP to check against.
@param[in] NetMask The mask of the IP. @param[in] NetMask The mask of the IP.

14
MdeModulePkg/Library/DxeNetLib/DxeNetLib.c
View File

@ -637,7 +637,9 @@ NetGetIpClass (
ASSERT if NetMask is zero. ASSERT if NetMask is zero.
If all bits of the host address of IP are 0 or 1, IP is also not a valid unicast address. If all bits of the host address of IP are 0 or 1, IP is also not a valid unicast address,
except when the originator is one of the endpoints of a point-to-point link with a 31-bit
mask (RFC3021).
@param[in] Ip The IP to check against. @param[in] Ip The IP to check against.
@param[in] NetMask The mask of the IP. @param[in] NetMask The mask of the IP.
@ -657,9 +659,13 @@ NetIp4IsUnicast (
if (Ip == 0 || IP4_IS_LOCAL_BROADCAST (Ip)) { if (Ip == 0 || IP4_IS_LOCAL_BROADCAST (Ip)) {
return FALSE; return FALSE;
} }
if (((Ip &~NetMask) == ~NetMask) || ((Ip &~NetMask) == 0)) { if (NetGetMaskLength (NetMask) != 31) {
return FALSE; if (((Ip &~NetMask) == ~NetMask) || ((Ip &~NetMask) == 0)) {
return FALSE;
}
} else {
return TRUE;
} }
return TRUE; return TRUE;

25
MdeModulePkg/Universal/Network/Ip4Dxe/Ip4Common.c
View File

@ -287,10 +287,6 @@ Ip4StationAddressValid (
IN IP4_ADDR Netmask IN IP4_ADDR Netmask
) )
{ {
IP4_ADDR NetBrdcastMask;
INTN Len;
INTN Type;
// //
// Only support the station address with 0.0.0.0/0 to enable DHCP client. // Only support the station address with 0.0.0.0/0 to enable DHCP client.
// //
@ -301,29 +297,16 @@ Ip4StationAddressValid (
// //
// Only support the continuous net masks // Only support the continuous net masks
// //
if ((Len = NetGetMaskLength (Netmask)) == (IP4_MASK_MAX + 1)) { if (NetGetMaskLength (Netmask) == (IP4_MASK_MAX + 1)) {
return FALSE; return FALSE;
} }
// //
// Station address can't be class D or class E address // Station address can't be class D or class E address
// //
if ((Type = NetGetIpClass (Ip)) > IP4_ADDR_CLASSC) { if (NetGetIpClass (Ip) > IP4_ADDR_CLASSC) {
return FALSE; return FALSE;
} }
// return NetIp4IsUnicast (Ip, Netmask);
// Station address can't be subnet broadcast/net broadcast address }
//
if ((Ip == (Ip & Netmask)) || (Ip == (Ip | ~Netmask))) {
return FALSE;
}
NetBrdcastMask = gIp4AllMasks[MIN (Len, Type << 3)];
if (Ip == (Ip | ~NetBrdcastMask)) {
return FALSE;
}
return TRUE;
}