diff options
| author | Jiaxin Wu <jiaxin.wu@intel.com> | 2017-02-23 13:27:18 +0800 |
|---|---|---|
| committer | Jiaxin Wu <jiaxin.wu@intel.com> | 2017-02-27 14:57:11 +0800 |
| commit | a2f4b153ac5d0039f81bb689ad32cd48ac2aa51d (patch) | |
| tree | a6c98d70a6db26a41f473a378e741adfaa86cd9a /MdeModulePkg/Universal | |
| parent | 29c90f14c91bf134a202c1b4ed600c9766bc1411 (diff) | |
| download | edk2-platforms-a2f4b153ac5d0039f81bb689ad32cd48ac2aa51d.tar.xz | |
MdeModulePkg/Ip4Dxe: Ignore duplicated DNS address check
Having duplicated DNS server IPs specified is not an ideal
configuration, but not an error condition. This patch is to
remove the duplicated DNS address check to allow the same DNS
address setting in SetData().
Cc: Hegde Nagaraj P <nagaraj-p.hegde@hpe.com>
Cc: Subramanian Sriram <sriram-s@hpe.com>
Cc: Ye Ting <ting.ye@intel.com>
Cc: Fu Siyuan <siyuan.fu@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Wu Jiaxin <jiaxin.wu@intel.com>
Reviewed-by: Subramanian Sriram <sriram-s@hpe.com>
Reviewed-by: Hegde Nagaraj P <nagaraj-p.hegde@hpe.com>
Tested-by: Hegde Nagaraj P <nagaraj-p.hegde@hpe.com>
Diffstat (limited to 'MdeModulePkg/Universal')
| -rw-r--r-- | MdeModulePkg/Universal/Network/Ip4Dxe/Ip4Config2Impl.c | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/MdeModulePkg/Universal/Network/Ip4Dxe/Ip4Config2Impl.c b/MdeModulePkg/Universal/Network/Ip4Dxe/Ip4Config2Impl.c index 504f2477da..6c7ac68791 100644 --- a/MdeModulePkg/Universal/Network/Ip4Dxe/Ip4Config2Impl.c +++ b/MdeModulePkg/Universal/Network/Ip4Dxe/Ip4Config2Impl.c @@ -705,7 +705,6 @@ Ip4Config2SetDnsServerWorker ( {
UINTN OldIndex;
UINTN NewIndex;
- UINTN Index1;
EFI_IPv4_ADDRESS *OldDns;
EFI_IPv4_ADDRESS *NewDns;
UINTN OldDnsCount;
@@ -747,15 +746,6 @@ Ip4Config2SetDnsServerWorker ( return EFI_INVALID_PARAMETER;
}
- for (Index1 = NewIndex + 1; Index1 < NewDnsCount; Index1++) {
- if (EFI_IP4_EQUAL (NewDns + NewIndex, NewDns + Index1)) {
- if (Tmp != NULL) {
- FreePool (Tmp);
- }
- return EFI_INVALID_PARAMETER;
- }
- }
-
if (OneAdded) {
//
// If any address in the new setting is not in the old settings, skip the
|