]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Library/DxeNetLib/DxeNetLib.c
BaseTools:Change the path of the file that Binary Cache
[mirror_edk2.git] / MdeModulePkg / Library / DxeNetLib / DxeNetLib.c
index 04d8345a6d45f49ee5ee15e10f2fe0ee96d7ebaa..8e2f720666eaa3213e179e8e6200e8e3bde4de5e 100644 (file)
@@ -1,15 +1,9 @@
 /** @file\r
   Network library.\r
 \r
-Copyright (c) 2005 - 2016, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2005 - 2018, Intel Corporation. All rights reserved.<BR>\r
 (C) Copyright 2015 Hewlett Packard Enterprise Development LP<BR>\r
-This program and the accompanying materials\r
-are licensed and made available under the terms and conditions of the BSD License\r
-which accompanies this distribution.  The full text of the license may be found at\r
-http://opensource.org/licenses/bsd-license.php\r
-\r
-THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
-WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
+SPDX-License-Identifier: BSD-2-Clause-Patent\r
 **/\r
 \r
 #include <Uefi.h>\r
@@ -19,6 +13,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #include <Protocol/DriverBinding.h>\r
 #include <Protocol/ServiceBinding.h>\r
 #include <Protocol/SimpleNetwork.h>\r
+#include <Protocol/AdapterInformation.h>\r
 #include <Protocol/ManagedNetwork.h>\r
 #include <Protocol/Ip4Config2.h>\r
 #include <Protocol/ComponentName.h>\r
@@ -37,7 +32,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #include <Library/PrintLib.h>\r
 #include <Library/UefiLib.h>\r
 \r
-#define NIC_ITEM_CONFIG_SIZE   sizeof (NIC_IP4_CONFIG_INFO) + sizeof (EFI_IP4_ROUTE_TABLE) * MAX_IP4_CONFIG_IN_VARIABLE\r
+#define NIC_ITEM_CONFIG_SIZE   (sizeof (NIC_IP4_CONFIG_INFO) + sizeof (EFI_IP4_ROUTE_TABLE) * MAX_IP4_CONFIG_IN_VARIABLE)\r
 #define DEFAULT_ZERO_START     ((UINTN) ~0)\r
 \r
 //\r
@@ -197,6 +192,7 @@ SyslogLocateSnp (
   fill in the source MAC because it will try to locate a SNP each\r
   time it is called to avoid the problem if SNP is unloaded.\r
   This code snip is copied from MNP.\r
+  If Packet is NULL, then ASSERT().\r
 \r
   @param[in] Packet          The Syslog packet\r
   @param[in] Length          The length of the packet\r
@@ -218,6 +214,8 @@ SyslogSendPacket (
   EFI_EVENT                   TimeoutEvent;\r
   UINT8                       *TxBuf;\r
 \r
+  ASSERT (Packet != NULL);\r
+\r
   Snp = SyslogLocateSnp ();\r
 \r
   if (Snp == NULL) {\r
@@ -309,7 +307,7 @@ ON_EXIT:
   @param[in]  BufLen    The lenght of the Buf\r
   @param[out] Buf       The buffer to put the packet data\r
 \r
-  @return The length of the syslog packet built.\r
+  @return The length of the syslog packet built, 0 represents no packet is built.\r
 \r
 **/\r
 UINT32\r
@@ -323,6 +321,7 @@ SyslogBuildPacket (
   OUT CHAR8                 *Buf\r
   )\r
 {\r
+  EFI_STATUS                Status;\r
   ETHER_HEAD                *Ether;\r
   IP4_HEAD                  *Ip4;\r
   EFI_UDP_HEADER            *Udp4;\r
@@ -378,8 +377,10 @@ SyslogBuildPacket (
   // Build the syslog message body with <PRI> Timestamp  machine module Message\r
   //\r
   Pri = ((NET_SYSLOG_FACILITY & 31) << 3) | (Level & 7);\r
-  gRT->GetTime (&Time, NULL);\r
-  ASSERT ((Time.Month <= 12) && (Time.Month >= 1));\r
+  Status = gRT->GetTime (&Time, NULL);\r
+  if (EFI_ERROR (Status)) {\r
+    return 0;\r
+  }\r
 \r
   //\r
   // Use %a to format the ASCII strings, %s to format UNICODE strings\r
@@ -396,7 +397,6 @@ SyslogBuildPacket (
                     Time.Minute,\r
                     Time.Second\r
                     );\r
-  Len--;\r
 \r
   Len += (UINT32) AsciiSPrint (\r
                     Buf + Len,\r
@@ -407,7 +407,7 @@ SyslogBuildPacket (
                     Line,\r
                     File\r
                     );\r
-  Len--;\r
+  Len ++;\r
 \r
   //\r
   // OK, patch the IP length/checksum and UDP length fields.\r
@@ -438,6 +438,8 @@ SyslogBuildPacket (
            NetDebugASPrint ("State transit to %a\n", Name)\r
          )\r
 \r
+  If Format is NULL, then ASSERT().\r
+\r
   @param Format  The ASCII format string.\r
   @param ...     The variable length parameter whose format is determined\r
                  by the Format string.\r
@@ -456,6 +458,8 @@ NetDebugASPrint (
   VA_LIST                   Marker;\r
   CHAR8                     *Buf;\r
 \r
+  ASSERT (Format != NULL);\r
+\r
   Buf = (CHAR8 *) AllocatePool (NET_DEBUG_MSG_LEN);\r
 \r
   if (Buf == NULL) {\r
@@ -482,7 +486,8 @@ NetDebugASPrint (
   @param Message  The user message to log.\r
 \r
   @retval EFI_INVALID_PARAMETER Any input parameter is invalid.\r
-  @retval EFI_OUT_OF_RESOURCES  Failed to allocate memory for the packet\r
+  @retval EFI_OUT_OF_RESOURCES  Failed to allocate memory for the packet.\r
+  @retval EFI_DEVICE_ERROR      Device error occurs.\r
   @retval EFI_SUCCESS           The log is discard because that it is more verbose\r
                                 than the mNetDebugLevelMax. Or, it has been sent out.\r
 **/\r
@@ -503,7 +508,7 @@ NetDebugOutput (
   //\r
   // Check whether the message should be sent out\r
   //\r
-  if (Message == NULL) {\r
+  if (Message == NULL || File == NULL || Module == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -536,9 +541,13 @@ NetDebugOutput (
           NET_SYSLOG_PACKET_LEN,\r
           Packet\r
           );\r
+  if (Len == 0) {\r
+    Status = EFI_DEVICE_ERROR;\r
+  } else {\r
+    mSyslogPacketSeq++;\r
+    Status = SyslogSendPacket (Packet, Len);\r
+  }\r
 \r
-  mSyslogPacketSeq++;\r
-  Status = SyslogSendPacket (Packet, Len);\r
   FreePool (Packet);\r
 \r
 ON_EXIT:\r
@@ -633,11 +642,14 @@ NetGetIpClass (
 \r
 /**\r
   Check whether the IP is a valid unicast address according to\r
-  the netmask. \r
+  the netmask.\r
 \r
   ASSERT if NetMask is zero.\r
-  \r
-  If all bits of the host address of IP are 0 or 1, IP is also not a valid unicast address.\r
+\r
+  If all bits of the host address of IP are 0 or 1, IP is also not a valid unicast address,\r
+  except when the originator is one of the endpoints of a point-to-point link with a 31-bit\r
+  mask (RFC3021), or a 32bit NetMask (all 0xFF) is used for special network environment (e.g.\r
+  PPP link).\r
 \r
   @param[in]  Ip                    The IP to check against.\r
   @param[in]  NetMask               The mask of the IP.\r
@@ -652,14 +664,20 @@ NetIp4IsUnicast (
   IN IP4_ADDR               NetMask\r
   )\r
 {\r
+  INTN   MaskLength;\r
+\r
   ASSERT (NetMask != 0);\r
-  \r
+\r
   if (Ip == 0 || IP4_IS_LOCAL_BROADCAST (Ip)) {\r
     return FALSE;\r
   }\r
-  \r
-  if (((Ip &~NetMask) == ~NetMask) || ((Ip &~NetMask) == 0)) {\r
-    return FALSE;\r
+\r
+  MaskLength = NetGetMaskLength (NetMask);\r
+  ASSERT ((MaskLength >= 0) && (MaskLength <= IP4_MASK_NUM));\r
+  if (MaskLength < 31) {\r
+    if (((Ip &~NetMask) == ~NetMask) || ((Ip &~NetMask) == 0)) {\r
+      return FALSE;\r
+    }\r
   }\r
 \r
   return TRUE;\r
@@ -668,6 +686,8 @@ NetIp4IsUnicast (
 /**\r
   Check whether the incoming IPv6 address is a valid unicast address.\r
 \r
+  ASSERT if Ip6 is NULL.\r
+\r
   If the address is a multicast address has binary 0xFF at the start, it is not\r
   a valid unicast address. If the address is unspecified ::, it is not a valid\r
   unicast address to be assigned to any node. If the address is loopback address\r
@@ -688,6 +708,8 @@ NetIp6IsValidUnicast (
   UINT8 Byte;\r
   UINT8 Index;\r
 \r
+  ASSERT (Ip6 != NULL);\r
+\r
   if (Ip6->Addr[0] == 0xFF) {\r
     return FALSE;\r
   }\r
@@ -710,6 +732,8 @@ NetIp6IsValidUnicast (
 /**\r
   Check whether the incoming Ipv6 address is the unspecified address or not.\r
 \r
+  ASSERT if Ip6 is NULL.\r
+\r
   @param[in] Ip6   - Ip6 address, in network order.\r
 \r
   @retval TRUE     - Yes, unspecified\r
@@ -724,6 +748,8 @@ NetIp6IsUnspecifiedAddr (
 {\r
   UINT8 Index;\r
 \r
+  ASSERT (Ip6 != NULL);\r
+\r
   for (Index = 0; Index < 16; Index++) {\r
     if (Ip6->Addr[Index] != 0) {\r
       return FALSE;\r
@@ -736,6 +762,8 @@ NetIp6IsUnspecifiedAddr (
 /**\r
   Check whether the incoming Ipv6 address is a link-local address.\r
 \r
+  ASSERT if Ip6 is NULL.\r
+\r
   @param[in] Ip6   - Ip6 address, in network order.\r
 \r
   @retval TRUE  - Yes, link-local address\r
@@ -772,6 +800,9 @@ NetIp6IsLinkLocalAddr (
 /**\r
   Check whether the Ipv6 address1 and address2 are on the connected network.\r
 \r
+  ASSERT if Ip1 or Ip2 is NULL.\r
+  ASSERT if PrefixLength exceeds or equals to IP6_PREFIX_MAX.\r
+\r
   @param[in] Ip1          - Ip6 address1, in network order.\r
   @param[in] Ip2          - Ip6 address2, in network order.\r
   @param[in] PrefixLength - The prefix length of the checking net.\r
@@ -792,7 +823,7 @@ NetIp6IsNetEqual (
   UINT8 Bit;\r
   UINT8 Mask;\r
 \r
-  ASSERT ((Ip1 != NULL) && (Ip2 != NULL) && (PrefixLength <= IP6_PREFIX_MAX));\r
+  ASSERT ((Ip1 != NULL) && (Ip2 != NULL) && (PrefixLength < IP6_PREFIX_MAX));\r
 \r
   if (PrefixLength == 0) {\r
     return TRUE;\r
@@ -809,6 +840,9 @@ NetIp6IsNetEqual (
     Mask = (UINT8) (0xFF << (8 - Bit));\r
 \r
     ASSERT (Byte < 16);\r
+    if (Byte >= 16) {\r
+      return FALSE;\r
+    }\r
     if ((Ip1->Addr[Byte] & Mask) != (Ip2->Addr[Byte] & Mask)) {\r
       return FALSE;\r
     }\r
@@ -821,6 +855,8 @@ NetIp6IsNetEqual (
 /**\r
   Switches the endianess of an IPv6 address\r
 \r
+  ASSERT if Ip6 is NULL.\r
+\r
   This function swaps the bytes in a 128-bit IPv6 address to switch the value\r
   from little endian to big endian or vice versa. The byte swapped value is\r
   returned.\r
@@ -839,6 +875,8 @@ Ip6Swap128 (
   UINT64 High;\r
   UINT64 Low;\r
 \r
+  ASSERT (Ip6 != NULL);\r
+\r
   CopyMem (&High, Ip6, sizeof (UINT64));\r
   CopyMem (&Low, &Ip6->Addr[8], sizeof (UINT64));\r
 \r
@@ -854,7 +892,7 @@ Ip6Swap128 (
 /**\r
   Initialize a random seed using current time and monotonic count.\r
 \r
-  Get current time and monotonic count first. Then initialize a random seed \r
+  Get current time and monotonic count first. Then initialize a random seed\r
   based on some basic mathematics operation on the hour, day, minute, second,\r
   nanosecond and year of the current time and the monotonic count value.\r
 \r
@@ -872,7 +910,7 @@ NetRandomInitSeed (
   UINT64                    MonotonicCount;\r
 \r
   gRT->GetTime (&Time, NULL);\r
-  Seed = (~Time.Hour << 24 | Time.Day << 16 | Time.Minute << 8 | Time.Second);\r
+  Seed = (Time.Hour << 24 | Time.Day << 16 | Time.Minute << 8 | Time.Second);\r
   Seed ^= Time.Nanosecond;\r
   Seed ^= Time.Year << 7;\r
 \r
@@ -886,6 +924,8 @@ NetRandomInitSeed (
 /**\r
   Extract a UINT32 from a byte stream.\r
 \r
+  ASSERT if Buf is NULL.\r
+\r
   Copy a UINT32 from a byte stream, then converts it from Network\r
   byte order to host byte order. Use this function to avoid alignment error.\r
 \r
@@ -902,6 +942,8 @@ NetGetUint32 (
 {\r
   UINT32                    Value;\r
 \r
+  ASSERT (Buf != NULL);\r
+\r
   CopyMem (&Value, Buf, sizeof (UINT32));\r
   return NTOHL (Value);\r
 }\r
@@ -910,6 +952,8 @@ NetGetUint32 (
 /**\r
   Put a UINT32 to the byte stream in network byte order.\r
 \r
+  ASSERT if Buf is NULL.\r
+\r
   Converts a UINT32 from host byte order to network byte order. Then copy it to the\r
   byte stream.\r
 \r
@@ -924,6 +968,8 @@ NetPutUint32 (
   IN     UINT32                Data\r
   )\r
 {\r
+  ASSERT (Buf != NULL);\r
+\r
   Data = HTONL (Data);\r
   CopyMem (Buf, &Data, sizeof (UINT32));\r
 }\r
@@ -1022,6 +1068,8 @@ NetListRemoveTail (
 /**\r
   Insert a new node entry after a designated node entry of a doubly linked list.\r
 \r
+  ASSERT if PrevEntry or NewEntry is NULL.\r
+\r
   Inserts a new node entry donated by NewEntry after the node entry donated by PrevEntry\r
   of the doubly linked list.\r
 \r
@@ -1036,6 +1084,8 @@ NetListInsertAfter (
   IN OUT LIST_ENTRY         *NewEntry\r
   )\r
 {\r
+  ASSERT (PrevEntry != NULL && NewEntry != NULL);\r
+\r
   NewEntry->BackLink                = PrevEntry;\r
   NewEntry->ForwardLink             = PrevEntry->ForwardLink;\r
   PrevEntry->ForwardLink->BackLink  = NewEntry;\r
@@ -1046,6 +1096,8 @@ NetListInsertAfter (
 /**\r
   Insert a new node entry before a designated node entry of a doubly linked list.\r
 \r
+  ASSERT if PostEntry or NewEntry is NULL.\r
+\r
   Inserts a new node entry donated by NewEntry after the node entry donated by PostEntry\r
   of the doubly linked list.\r
 \r
@@ -1060,6 +1112,8 @@ NetListInsertBefore (
   IN OUT LIST_ENTRY     *NewEntry\r
   )\r
 {\r
+  ASSERT (PostEntry != NULL && NewEntry != NULL);\r
+\r
   NewEntry->ForwardLink             = PostEntry;\r
   NewEntry->BackLink                = PostEntry->BackLink;\r
   PostEntry->BackLink->ForwardLink  = NewEntry;\r
@@ -1075,7 +1129,7 @@ NetListInsertBefore (
   If it has been removed, then restart the traversal from the head.\r
   If it hasn't been removed, then continue with the next node directly.\r
   This function will end the iterate and return the CallBack's last return value if error happens,\r
-  or retrun EFI_SUCCESS if 2 complete passes are made with no changes in the number of children in the list.  \r
+  or retrun EFI_SUCCESS if 2 complete passes are made with no changes in the number of children in the list.\r
 \r
   @param[in]    List             The head of the list.\r
   @param[in]    CallBack         Pointer to the callback function to destroy one node in the list.\r
@@ -1162,7 +1216,7 @@ NetIsInHandleBuffer (
   )\r
 {\r
   UINTN     Index;\r
-  \r
+\r
   if (NumberOfChildren == 0 || ChildHandleBuffer == NULL) {\r
     return FALSE;\r
   }\r
@@ -1258,7 +1312,6 @@ NetMapClean (
 \r
   If Map is NULL, then ASSERT().\r
 \r
-\r
   @param[in]  Map                   The net map to test.\r
 \r
   @return TRUE if the netmap is empty, otherwise FALSE.\r
@@ -1278,6 +1331,8 @@ NetMapIsEmpty (
 /**\r
   Return the number of the <Key, Value> pairs in the netmap.\r
 \r
+  If Map is NULL, then ASSERT().\r
+\r
   @param[in]  Map                   The netmap to get the entry number.\r
 \r
   @return The entry number in the netmap.\r
@@ -1289,6 +1344,7 @@ NetMapGetCount (
   IN NET_MAP                *Map\r
   )\r
 {\r
+  ASSERT (Map != NULL);\r
   return Map->Count;\r
 }\r
 \r
@@ -1353,6 +1409,7 @@ NetMapAllocItem (
   pairs in the netmap increase by 1.\r
 \r
   If Map is NULL, then ASSERT().\r
+  If Key is NULL, then ASSERT().\r
 \r
   @param[in, out]  Map                   The netmap to insert into.\r
   @param[in]       Key                   The user's key.\r
@@ -1372,7 +1429,7 @@ NetMapInsertHead (
 {\r
   NET_MAP_ITEM              *Item;\r
 \r
-  ASSERT (Map != NULL);\r
+  ASSERT (Map != NULL && Key != NULL);\r
 \r
   Item = NetMapAllocItem (Map);\r
 \r
@@ -1397,6 +1454,7 @@ NetMapInsertHead (
   pairs in the netmap increase by 1.\r
 \r
   If Map is NULL, then ASSERT().\r
+  If Key is NULL, then ASSERT().\r
 \r
   @param[in, out]  Map                   The netmap to insert into.\r
   @param[in]       Key                   The user's key.\r
@@ -1416,7 +1474,7 @@ NetMapInsertTail (
 {\r
   NET_MAP_ITEM              *Item;\r
 \r
-  ASSERT (Map != NULL);\r
+  ASSERT (Map != NULL && Key != NULL);\r
 \r
   Item = NetMapAllocItem (Map);\r
 \r
@@ -1437,6 +1495,9 @@ NetMapInsertTail (
 /**\r
   Check whether the item is in the Map and return TRUE if it is.\r
 \r
+  If Map is NULL, then ASSERT().\r
+  If Item is NULL, then ASSERT().\r
+\r
   @param[in]  Map                   The netmap to search within.\r
   @param[in]  Item                  The item to search.\r
 \r
@@ -1451,6 +1512,8 @@ NetItemInMap (
 {\r
   LIST_ENTRY            *ListEntry;\r
 \r
+  ASSERT (Map != NULL && Item != NULL);\r
+\r
   NET_LIST_FOR_EACH (ListEntry, &Map->Used) {\r
     if (ListEntry == &Item->Link) {\r
       return TRUE;\r
@@ -1468,6 +1531,7 @@ NetItemInMap (
   item with the key to search. It returns the point to the item contains the Key if found.\r
 \r
   If Map is NULL, then ASSERT().\r
+  If Key is NULL, then ASSERT().\r
 \r
   @param[in]  Map                   The netmap to search within.\r
   @param[in]  Key                   The key to search.\r
@@ -1485,7 +1549,7 @@ NetMapFindKey (
   LIST_ENTRY              *Entry;\r
   NET_MAP_ITEM            *Item;\r
 \r
-  ASSERT (Map != NULL);\r
+  ASSERT (Map != NULL && Key != NULL);\r
 \r
   NET_LIST_FOR_EACH (Entry, &Map->Used) {\r
     Item = NET_LIST_USER_STRUCT (Entry, NET_MAP_ITEM, Link);\r
@@ -1743,7 +1807,7 @@ NetLibDefaultUnload (
     if (DriverBinding->ImageHandle != ImageHandle) {\r
       continue;\r
     }\r
-    \r
+\r
     //\r
     // Disconnect the driver specified by ImageHandle from all\r
     // the devices in the handle database.\r
@@ -1755,16 +1819,16 @@ NetLibDefaultUnload (
                       NULL\r
                       );\r
     }\r
-    \r
+\r
     //\r
     // Uninstall all the protocols installed in the driver entry point\r
-    //    \r
+    //\r
     gBS->UninstallProtocolInterface (\r
           DriverBinding->DriverBindingHandle,\r
           &gEfiDriverBindingProtocolGuid,\r
           DriverBinding\r
           );\r
-    \r
+\r
     Status = gBS->HandleProtocol (\r
                     DeviceHandleBuffer[Index],\r
                     &gEfiComponentNameProtocolGuid,\r
@@ -2088,6 +2152,9 @@ NetLibGetVlanHandle (
 /**\r
   Get MAC address associated with the network service handle.\r
 \r
+  If MacAddress is NULL, then ASSERT().\r
+  If AddressSize is NULL, then ASSERT().\r
+\r
   There should be MNP Service Binding Protocol installed on the input ServiceHandle.\r
   If SNP is installed on the ServiceHandle or its parent handle, MAC address will\r
   be retrieved from SNP. If no SNP found, try to get SNP mode data use MNP.\r
@@ -2192,6 +2259,8 @@ NetLibGetMacAddress (
   Convert MAC address of the NIC associated with specified Service Binding Handle\r
   to a unicode string. Callers are responsible for freeing the string storage.\r
 \r
+  If MacString is NULL, then ASSERT().\r
+\r
   Locate simple network protocol associated with the Service Binding Handle and\r
   get the mac address from SNP. Then convert the mac address into a unicode\r
   string. It takes 2 unicode characters to represent a 1 byte binary buffer.\r
@@ -2225,6 +2294,7 @@ NetLibGetMacString (
   UINT16                       VlanId;\r
   CHAR16                       *String;\r
   UINTN                        Index;\r
+  UINTN                        BufferSize;\r
 \r
   ASSERT (MacString != NULL);\r
 \r
@@ -2241,7 +2311,8 @@ NetLibGetMacString (
   // If VLAN is configured, it will need extra 5 characters like "\0005".\r
   // Plus one unicode character for the null-terminator.\r
   //\r
-  String = AllocateZeroPool ((2 * HwAddressSize + 5 + 1) * sizeof (CHAR16));\r
+  BufferSize = (2 * HwAddressSize + 5 + 1) * sizeof (CHAR16);\r
+  String = AllocateZeroPool (BufferSize);\r
   if (String == NULL) {\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
@@ -2252,7 +2323,14 @@ NetLibGetMacString (
   //\r
   HwAddress = &MacAddress.Addr[0];\r
   for (Index = 0; Index < HwAddressSize; Index++) {\r
-    String += UnicodeValueToString (String, PREFIX_ZERO | RADIX_HEX, *(HwAddress++), 2);\r
+    UnicodeValueToStringS (\r
+      String,\r
+      BufferSize - ((UINTN)String - (UINTN)*MacString),\r
+      PREFIX_ZERO | RADIX_HEX,\r
+      *(HwAddress++),\r
+      2\r
+      );\r
+    String += StrnLenS (String, (BufferSize - ((UINTN)String - (UINTN)*MacString)) / sizeof (CHAR16));\r
   }\r
 \r
   //\r
@@ -2261,7 +2339,14 @@ NetLibGetMacString (
   VlanId = NetLibGetVlanId (ServiceHandle);\r
   if (VlanId != 0) {\r
     *String++ = L'\\';\r
-    String += UnicodeValueToString (String, PREFIX_ZERO | RADIX_HEX, VlanId, 4);\r
+    UnicodeValueToStringS (\r
+      String,\r
+      BufferSize - ((UINTN)String - (UINTN)*MacString),\r
+      PREFIX_ZERO | RADIX_HEX,\r
+      VlanId,\r
+      4\r
+      );\r
+    String += StrnLenS (String, (BufferSize - ((UINTN)String - (UINTN)*MacString)) / sizeof (CHAR16));\r
   }\r
 \r
   //\r
@@ -2275,6 +2360,8 @@ NetLibGetMacString (
 /**\r
   Detect media status for specified network device.\r
 \r
+  If MediaPresent is NULL, then ASSERT().\r
+\r
   The underlying UNDI driver may or may not support reporting media status from\r
   GET_STATUS command (PXE_STATFLAGS_GET_STATUS_NO_MEDIA_SUPPORTED). This routine\r
   will try to invoke Snp->GetStatus() to get the media status: if media already\r
@@ -2383,6 +2470,10 @@ NetLibDetectMedia (
                       Snp->Mode->MCastFilter\r
                       );\r
       ASSERT (MCastFilter != NULL);\r
+      if (MCastFilter == NULL) {\r
+        Status = EFI_OUT_OF_RESOURCES;\r
+        goto Exit;\r
+      }\r
 \r
       ResetMCastFilters = FALSE;\r
     }\r
@@ -2480,15 +2571,194 @@ Exit:
   return Status;\r
 }\r
 \r
+/**\r
+\r
+  Detect media state for a network device. This routine will wait for a period of time at\r
+  a specified checking interval when a certain network is under connecting until connection\r
+  process finishs or timeout. If Aip protocol is supported by low layer drivers, three kinds\r
+  of media states can be detected: EFI_SUCCESS, EFI_NOT_READY and EFI_NO_MEDIA, represents\r
+  connected state, connecting state and no media state respectively. When function detects\r
+  the current state is EFI_NOT_READY, it will loop to wait for next time's check until state\r
+  turns to be EFI_SUCCESS or EFI_NO_MEDIA. If Aip protocol is not supported, function will\r
+  call NetLibDetectMedia() and return state directly.\r
+\r
+  @param[in]   ServiceHandle    The handle where network service binding protocols are\r
+                                installed on.\r
+  @param[in]   Timeout          The maximum number of 100ns units to wait when network\r
+                                is connecting. Zero value means detect once and return\r
+                                immediately.\r
+  @param[out]  MediaState       The pointer to the detected media state.\r
+\r
+  @retval EFI_SUCCESS           Media detection success.\r
+  @retval EFI_INVALID_PARAMETER ServiceHandle is not a valid network device handle or\r
+                                MediaState pointer is NULL.\r
+  @retval EFI_DEVICE_ERROR      A device error occurred.\r
+  @retval EFI_TIMEOUT           Network is connecting but timeout.\r
+\r
+**/\r
+EFI_STATUS\r
+EFIAPI\r
+NetLibDetectMediaWaitTimeout (\r
+  IN  EFI_HANDLE            ServiceHandle,\r
+  IN  UINT64                Timeout,\r
+  OUT EFI_STATUS            *MediaState\r
+  )\r
+{\r
+  EFI_STATUS                        Status;\r
+  EFI_HANDLE                        SnpHandle;\r
+  EFI_SIMPLE_NETWORK_PROTOCOL       *Snp;\r
+  EFI_ADAPTER_INFORMATION_PROTOCOL  *Aip;\r
+  EFI_ADAPTER_INFO_MEDIA_STATE      *MediaInfo;\r
+  BOOLEAN                           MediaPresent;\r
+  UINTN                             DataSize;\r
+  EFI_STATUS                        TimerStatus;\r
+  EFI_EVENT                         Timer;\r
+  UINT64                            TimeRemained;\r
+\r
+  if (MediaState == NULL) {\r
+    return EFI_INVALID_PARAMETER;\r
+  }\r
+  *MediaState = EFI_SUCCESS;\r
+  MediaInfo   = NULL;\r
+\r
+  //\r
+  // Get SNP handle\r
+  //\r
+  Snp = NULL;\r
+  SnpHandle = NetLibGetSnpHandle (ServiceHandle, &Snp);\r
+  if (SnpHandle == NULL) {\r
+    return EFI_INVALID_PARAMETER;\r
+  }\r
+\r
+  Status = gBS->HandleProtocol (\r
+                  SnpHandle,\r
+                  &gEfiAdapterInformationProtocolGuid,\r
+                  (VOID *) &Aip\r
+                  );\r
+  if (EFI_ERROR (Status)) {\r
+\r
+    MediaPresent = TRUE;\r
+    Status = NetLibDetectMedia (ServiceHandle, &MediaPresent);\r
+    if (!EFI_ERROR (Status)) {\r
+      if (MediaPresent) {\r
+        *MediaState = EFI_SUCCESS;\r
+      } else {\r
+        *MediaState = EFI_NO_MEDIA;\r
+      }\r
+    }\r
+\r
+    //\r
+    // NetLibDetectMedia doesn't support EFI_NOT_READY status, return now!\r
+    //\r
+    return Status;\r
+  }\r
+\r
+  Status = Aip->GetInformation (\r
+                  Aip,\r
+                  &gEfiAdapterInfoMediaStateGuid,\r
+                  (VOID **) &MediaInfo,\r
+                  &DataSize\r
+                  );\r
+  if (!EFI_ERROR (Status)) {\r
+\r
+    *MediaState = MediaInfo->MediaState;\r
+    FreePool (MediaInfo);\r
+    if (*MediaState != EFI_NOT_READY || Timeout < MEDIA_STATE_DETECT_TIME_INTERVAL) {\r
+\r
+      return EFI_SUCCESS;\r
+    }\r
+  } else {\r
+\r
+    if (MediaInfo != NULL) {\r
+      FreePool (MediaInfo);\r
+    }\r
+\r
+    if (Status == EFI_UNSUPPORTED) {\r
+\r
+      //\r
+      // If gEfiAdapterInfoMediaStateGuid is not supported, call NetLibDetectMedia to get media state!\r
+      //\r
+      MediaPresent = TRUE;\r
+      Status = NetLibDetectMedia (ServiceHandle, &MediaPresent);\r
+      if (!EFI_ERROR (Status)) {\r
+        if (MediaPresent) {\r
+          *MediaState = EFI_SUCCESS;\r
+        } else {\r
+          *MediaState = EFI_NO_MEDIA;\r
+        }\r
+      }\r
+      return Status;\r
+    }\r
+\r
+    return Status;\r
+  }\r
+\r
+  //\r
+  // Loop to check media state\r
+  //\r
+\r
+  Timer        = NULL;\r
+  TimeRemained = Timeout;\r
+  Status = gBS->CreateEvent (EVT_TIMER, TPL_CALLBACK, NULL, NULL, &Timer);\r
+  if (EFI_ERROR (Status)) {\r
+    return EFI_DEVICE_ERROR;\r
+  }\r
+\r
+  do {\r
+    Status = gBS->SetTimer (\r
+                    Timer,\r
+                    TimerRelative,\r
+                    MEDIA_STATE_DETECT_TIME_INTERVAL\r
+                    );\r
+    if (EFI_ERROR (Status)) {\r
+      gBS->CloseEvent(Timer);\r
+      return EFI_DEVICE_ERROR;\r
+    }\r
+\r
+    do {\r
+      TimerStatus = gBS->CheckEvent (Timer);\r
+      if (!EFI_ERROR (TimerStatus)) {\r
+\r
+        TimeRemained -= MEDIA_STATE_DETECT_TIME_INTERVAL;\r
+        Status = Aip->GetInformation (\r
+                        Aip,\r
+                        &gEfiAdapterInfoMediaStateGuid,\r
+                        (VOID **) &MediaInfo,\r
+                        &DataSize\r
+                        );\r
+        if (!EFI_ERROR (Status)) {\r
+\r
+          *MediaState = MediaInfo->MediaState;\r
+          FreePool (MediaInfo);\r
+        } else {\r
+\r
+          if (MediaInfo != NULL) {\r
+            FreePool (MediaInfo);\r
+          }\r
+          gBS->CloseEvent(Timer);\r
+          return Status;\r
+        }\r
+      }\r
+    } while (TimerStatus == EFI_NOT_READY);\r
+  } while (*MediaState == EFI_NOT_READY && TimeRemained >= MEDIA_STATE_DETECT_TIME_INTERVAL);\r
+\r
+  gBS->CloseEvent(Timer);\r
+  if (*MediaState == EFI_NOT_READY && TimeRemained < MEDIA_STATE_DETECT_TIME_INTERVAL) {\r
+    return EFI_TIMEOUT;\r
+  } else {\r
+    return EFI_SUCCESS;\r
+  }\r
+}\r
+\r
 /**\r
   Check the default address used by the IPv4 driver is static or dynamic (acquired\r
   from DHCP).\r
 \r
   If the controller handle does not have the EFI_IP4_CONFIG2_PROTOCOL installed, the\r
-  default address is static. If failed to get the policy from Ip4 Config2 Protocol, \r
+  default address is static. If failed to get the policy from Ip4 Config2 Protocol,\r
   the default address is static. Otherwise, get the result from Ip4 Config2 Protocol.\r
 \r
-  @param[in]   Controller     The controller handle which has the EFI_IP4_CONFIG2_PROTOCOL \r
+  @param[in]   Controller     The controller handle which has the EFI_IP4_CONFIG2_PROTOCOL\r
                               relative with the default address to judge.\r
 \r
   @retval TRUE           If the default address is static.\r
@@ -2502,12 +2772,12 @@ NetLibDefaultAddressIsStatic (
 {\r
   EFI_STATUS                       Status;\r
   EFI_IP4_CONFIG2_PROTOCOL         *Ip4Config2;\r
-  UINTN                            DataSize;  \r
+  UINTN                            DataSize;\r
   EFI_IP4_CONFIG2_POLICY           Policy;\r
   BOOLEAN                          IsStatic;\r
 \r
   Ip4Config2 = NULL;\r
-  \r
+\r
   DataSize = sizeof (EFI_IP4_CONFIG2_POLICY);\r
 \r
   IsStatic   = TRUE;\r
@@ -2524,17 +2794,19 @@ NetLibDefaultAddressIsStatic (
   if (EFI_ERROR (Status)) {\r
     goto ON_EXIT;\r
   }\r
-  \r
+\r
   IsStatic = (BOOLEAN) (Policy == Ip4Config2PolicyStatic);\r
 \r
 ON_EXIT:\r
-  \r
+\r
   return IsStatic;\r
 }\r
 \r
 /**\r
   Create an IPv4 device path node.\r
 \r
+  If Node is NULL, then ASSERT().\r
+\r
   The header type of IPv4 device path node is MESSAGING_DEVICE_PATH.\r
   The header subtype of IPv4 device path node is MSG_IPv4_DP.\r
   Get other info from parameters to make up the whole IPv4 device path node.\r
@@ -2562,6 +2834,8 @@ NetLibCreateIPv4DPathNode (
   IN BOOLEAN               UseDefaultAddress\r
   )\r
 {\r
+  ASSERT (Node != NULL);\r
+\r
   Node->Header.Type    = MESSAGING_DEVICE_PATH;\r
   Node->Header.SubType = MSG_IPv4_DP;\r
   SetDevicePathNodeLength (&Node->Header, sizeof (IPv4_DEVICE_PATH));\r
@@ -2592,6 +2866,10 @@ NetLibCreateIPv4DPathNode (
 /**\r
   Create an IPv6 device path node.\r
 \r
+  If Node is NULL, then ASSERT().\r
+  If LocalIp is NULL, then ASSERT().\r
+  If RemoteIp is NULL, then ASSERT().\r
+\r
   The header type of IPv6 device path node is MESSAGING_DEVICE_PATH.\r
   The header subtype of IPv6 device path node is MSG_IPv6_DP.\r
   Get other info from parameters to make up the whole IPv6 device path node.\r
@@ -2617,6 +2895,8 @@ NetLibCreateIPv6DPathNode (
   IN UINT16                Protocol\r
   )\r
 {\r
+  ASSERT (Node != NULL && LocalIp != NULL && RemoteIp != NULL);\r
+\r
   Node->Header.Type    = MESSAGING_DEVICE_PATH;\r
   Node->Header.SubType = MSG_IPv6_DP;\r
   SetDevicePathNodeLength (&Node->Header, sizeof (IPv6_DEVICE_PATH));\r
@@ -2641,6 +2921,8 @@ NetLibCreateIPv6DPathNode (
 /**\r
   Find the UNDI/SNP handle from controller and protocol GUID.\r
 \r
+  If ProtocolGuid is NULL, then ASSERT().\r
+\r
   For example, IP will open a MNP child to transmit/receive\r
   packets, when MNP is stopped, IP should also be stopped. IP\r
   needs to find its own private data which is related the IP's\r
@@ -2668,6 +2950,8 @@ NetLibGetNicHandle (
   UINTN                               OpenCount;\r
   UINTN                               Index;\r
 \r
+  ASSERT (ProtocolGuid != NULL);\r
+\r
   Status = gBS->OpenProtocolInformation (\r
                   Controller,\r
                   ProtocolGuid,\r
@@ -2709,55 +2993,15 @@ NetLibAsciiStrToIp4 (
   OUT      EFI_IPv4_ADDRESS      *Ip4Address\r
   )\r
 {\r
-  UINT8                          Index;\r
-  CHAR8                          *Ip4Str;\r
-  CHAR8                          *TempStr;\r
-  UINTN                          NodeVal;\r
+  RETURN_STATUS                  Status;\r
+  CHAR8                          *EndPointer;\r
 \r
-  if ((String == NULL) || (Ip4Address == NULL)) {\r
+  Status = AsciiStrToIpv4Address (String, &EndPointer, Ip4Address, NULL);\r
+  if (RETURN_ERROR (Status) || (*EndPointer != '\0')) {\r
     return EFI_INVALID_PARAMETER;\r
+  } else {\r
+    return EFI_SUCCESS;\r
   }\r
-\r
-  Ip4Str = (CHAR8 *) String;\r
-\r
-  for (Index = 0; Index < 4; Index++) {\r
-    TempStr = Ip4Str;\r
-\r
-    while ((*Ip4Str != '\0') && (*Ip4Str != '.')) {\r
-      if (!NET_IS_DIGIT (*Ip4Str)) {\r
-        return EFI_INVALID_PARAMETER;\r
-      }\r
-      Ip4Str++;\r
-    }\r
-\r
-    //\r
-    // The IPv4 address is X.X.X.X\r
-    //\r
-    if (*Ip4Str == '.') {\r
-      if (Index == 3) {\r
-        return EFI_INVALID_PARAMETER;\r
-      }\r
-    } else {\r
-      if (Index != 3) {\r
-        return EFI_INVALID_PARAMETER;\r
-      }\r
-    }\r
-\r
-    //\r
-    // Convert the string to IPv4 address. AsciiStrDecimalToUintn stops at the\r
-    // first character that is not a valid decimal character, '.' or '\0' here.\r
-    //\r
-    NodeVal = AsciiStrDecimalToUintn (TempStr);\r
-    if (NodeVal > 0xFF) {\r
-      return EFI_INVALID_PARAMETER;\r
-    }\r
-\r
-    Ip4Address->Addr[Index] = (UINT8) NodeVal;\r
-\r
-    Ip4Str++;\r
-  }\r
-\r
-  return EFI_SUCCESS;\r
 }\r
 \r
 \r
@@ -2779,182 +3023,15 @@ NetLibAsciiStrToIp6 (
   OUT      EFI_IPv6_ADDRESS      *Ip6Address\r
   )\r
 {\r
-  UINT8                          Index;\r
-  CHAR8                          *Ip6Str;\r
-  CHAR8                          *TempStr;\r
-  CHAR8                          *TempStr2;\r
-  UINT8                          NodeCnt;\r
-  UINT8                          TailNodeCnt;\r
-  UINT8                          AllowedCnt;\r
-  UINTN                          NodeVal;\r
-  BOOLEAN                        Short;\r
-  BOOLEAN                        Update;\r
-  BOOLEAN                        LeadZero;\r
-  UINT8                          LeadZeroCnt;\r
-  UINT8                          Cnt;\r
-\r
-  if ((String == NULL) || (Ip6Address == NULL)) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  Ip6Str      = (CHAR8 *) String;\r
-  AllowedCnt  = 6;\r
-  LeadZeroCnt = 0;\r
-\r
-  //\r
-  // An IPv6 address leading with : looks strange.\r
-  //\r
-  if (*Ip6Str == ':') {\r
-    if (*(Ip6Str + 1) != ':') {\r
-      return EFI_INVALID_PARAMETER;\r
-    } else {\r
-      AllowedCnt = 7;\r
-    }\r
-  }\r
-\r
-  ZeroMem (Ip6Address, sizeof (EFI_IPv6_ADDRESS));\r
-\r
-  NodeCnt     = 0;\r
-  TailNodeCnt = 0;\r
-  Short       = FALSE;\r
-  Update      = FALSE;\r
-  LeadZero    = FALSE;\r
-\r
-  for (Index = 0; Index < 15; Index = (UINT8) (Index + 2)) {\r
-    TempStr = Ip6Str;\r
-\r
-    while ((*Ip6Str != '\0') && (*Ip6Str != ':')) {\r
-      Ip6Str++;\r
-    }\r
-\r
-    if ((*Ip6Str == '\0') && (Index != 14)) {\r
-      return EFI_INVALID_PARAMETER;\r
-    }\r
-\r
-    if (*Ip6Str == ':') {\r
-      if (*(Ip6Str + 1) == ':') {\r
-        if ((NodeCnt > 6) || \r
-            ((*(Ip6Str + 2) != '\0') && (AsciiStrHexToUintn (Ip6Str + 2) == 0))) {\r
-          //\r
-          // ::0 looks strange. report error to user.\r
-          //\r
-          return EFI_INVALID_PARAMETER;\r
-        }\r
-        if ((NodeCnt == 6) && (*(Ip6Str + 2) != '\0') && \r
-            (AsciiStrHexToUintn (Ip6Str + 2) != 0)) {\r
-          return EFI_INVALID_PARAMETER;\r
-        }\r
+  RETURN_STATUS                  Status;\r
+  CHAR8                          *EndPointer;\r
 \r
-        //\r
-        // Skip the abbreviation part of IPv6 address.\r
-        //\r
-        TempStr2 = Ip6Str + 2;\r
-        while ((*TempStr2 != '\0')) {\r
-          if (*TempStr2 == ':') {\r
-            if (*(TempStr2 + 1) == ':') {\r
-              //\r
-              // :: can only appear once in IPv6 address.\r
-              //\r
-              return EFI_INVALID_PARAMETER;\r
-            }\r
-\r
-            TailNodeCnt++;\r
-            if (TailNodeCnt >= (AllowedCnt - NodeCnt)) {\r
-              //\r
-              // :: indicates one or more groups of 16 bits of zeros.\r
-              //\r
-              return EFI_INVALID_PARAMETER;\r
-            }\r
-          }\r
-\r
-          TempStr2++;\r
-        }\r
-\r
-        Short  = TRUE;\r
-        Update = TRUE;\r
-\r
-        Ip6Str = Ip6Str + 2;\r
-      } else {\r
-        if (*(Ip6Str + 1) == '\0') {\r
-          return EFI_INVALID_PARAMETER;\r
-        }\r
-        Ip6Str++;\r
-        NodeCnt++;\r
-        if ((Short && (NodeCnt > 6)) || (!Short && (NodeCnt > 7))) {\r
-          //\r
-          // There are more than 8 groups of 16 bits of zeros.\r
-          //\r
-          return EFI_INVALID_PARAMETER;\r
-        }\r
-      }\r
-    }\r
-\r
-    //\r
-    // Convert the string to IPv6 address. AsciiStrHexToUintn stops at the first\r
-    // character that is not a valid hexadecimal character, ':' or '\0' here.\r
-    //\r
-    NodeVal = AsciiStrHexToUintn (TempStr);\r
-    if ((NodeVal > 0xFFFF) || (Index > 14)) {\r
-      return EFI_INVALID_PARAMETER;\r
-    }\r
-    if (NodeVal != 0) {\r
-      if ((*TempStr  == '0') && \r
-          ((*(TempStr + 2) == ':') || (*(TempStr + 3) == ':') || \r
-          (*(TempStr + 2) == '\0') || (*(TempStr + 3) == '\0'))) {\r
-        return EFI_INVALID_PARAMETER;\r
-      }\r
-      if ((*TempStr  == '0') && (*(TempStr + 4) != '\0') && \r
-          (*(TempStr + 4) != ':')) { \r
-        return EFI_INVALID_PARAMETER;\r
-      }\r
-    } else {\r
-      if (((*TempStr  == '0') && (*(TempStr + 1) == '0') && \r
-          ((*(TempStr + 2) == ':') || (*(TempStr + 2) == '\0'))) ||\r
-          ((*TempStr  == '0') && (*(TempStr + 1) == '0') && (*(TempStr + 2) == '0') && \r
-          ((*(TempStr + 3) == ':') || (*(TempStr + 3) == '\0')))) {\r
-        return EFI_INVALID_PARAMETER;\r
-      }\r
-    }\r
-\r
-    Cnt = 0;\r
-    while ((TempStr[Cnt] != ':') && (TempStr[Cnt] != '\0')) {\r
-      Cnt++; \r
-    }\r
-    if (LeadZeroCnt == 0) {\r
-      if ((Cnt == 4) && (*TempStr  == '0')) {\r
-        LeadZero = TRUE;\r
-        LeadZeroCnt++;\r
-      }\r
-      if ((Cnt != 0) && (Cnt < 4)) {\r
-        LeadZero = FALSE;\r
-        LeadZeroCnt++;\r
-      }\r
-    } else {\r
-      if ((Cnt == 4) && (*TempStr  == '0') && !LeadZero) {\r
-        return EFI_INVALID_PARAMETER;\r
-      }\r
-      if ((Cnt != 0) && (Cnt < 4) && LeadZero) {\r
-        return EFI_INVALID_PARAMETER;\r
-      }\r
-    } \r
-\r
-    Ip6Address->Addr[Index] = (UINT8) (NodeVal >> 8);\r
-    Ip6Address->Addr[Index + 1] = (UINT8) (NodeVal & 0xFF);\r
-\r
-    //\r
-    // Skip the groups of zeros by ::\r
-    //\r
-    if (Short && Update) {\r
-      Index  = (UINT8) (16 - (TailNodeCnt + 2) * 2);\r
-      Update = FALSE;\r
-    }\r
-  }\r
-\r
-  if ((!Short && Index != 16) || (*Ip6Str != '\0')) {\r
+  Status = AsciiStrToIpv6Address (String, &EndPointer, Ip6Address, NULL);\r
+  if (RETURN_ERROR (Status) || (*EndPointer != '\0')) {\r
     return EFI_INVALID_PARAMETER;\r
+  } else {\r
+    return EFI_SUCCESS;\r
   }\r
-\r
-  return EFI_SUCCESS;\r
 }\r
 \r
 \r
@@ -2966,7 +3043,6 @@ NetLibAsciiStrToIp6 (
 \r
   @retval EFI_SUCCESS            Convert to IPv4 address successfully.\r
   @retval EFI_INVALID_PARAMETER  The string is mal-formated or Ip4Address is NULL.\r
-  @retval EFI_OUT_OF_RESOURCES   Fail to perform the operation due to lack of resource.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -2976,27 +3052,15 @@ NetLibStrToIp4 (
   OUT      EFI_IPv4_ADDRESS      *Ip4Address\r
   )\r
 {\r
-  CHAR8                          *Ip4Str;\r
-  UINTN                          StringSize;\r
-  EFI_STATUS                     Status;\r
+  RETURN_STATUS                  Status;\r
+  CHAR16                         *EndPointer;\r
 \r
-  if ((String == NULL) || (Ip4Address == NULL)) {\r
+  Status = StrToIpv4Address (String, &EndPointer, Ip4Address, NULL);\r
+  if (RETURN_ERROR (Status) || (*EndPointer != L'\0')) {\r
     return EFI_INVALID_PARAMETER;\r
+  } else {\r
+    return EFI_SUCCESS;\r
   }\r
-\r
-  StringSize = StrLen (String) + 1;\r
-  Ip4Str = (CHAR8 *) AllocatePool (StringSize * sizeof (CHAR8));\r
-  if (Ip4Str == NULL) {\r
-    return EFI_OUT_OF_RESOURCES;\r
-  }\r
-\r
-  UnicodeStrToAsciiStrS (String, Ip4Str, StringSize);\r
-\r
-  Status = NetLibAsciiStrToIp4 (Ip4Str, Ip4Address);\r
-\r
-  FreePool (Ip4Str);\r
-\r
-  return Status;\r
 }\r
 \r
 \r
@@ -3009,7 +3073,6 @@ NetLibStrToIp4 (
 \r
   @retval EFI_SUCCESS            Convert to IPv6 address successfully.\r
   @retval EFI_INVALID_PARAMETER  The string is mal-formated or Ip6Address is NULL.\r
-  @retval EFI_OUT_OF_RESOURCES   Fail to perform the operation due to lack of resource.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -3019,27 +3082,15 @@ NetLibStrToIp6 (
   OUT      EFI_IPv6_ADDRESS      *Ip6Address\r
   )\r
 {\r
-  CHAR8                          *Ip6Str;\r
-  UINTN                          StringSize;\r
-  EFI_STATUS                     Status;\r
+  RETURN_STATUS                  Status;\r
+  CHAR16                         *EndPointer;\r
 \r
-  if ((String == NULL) || (Ip6Address == NULL)) {\r
+  Status = StrToIpv6Address (String, &EndPointer, Ip6Address, NULL);\r
+  if (RETURN_ERROR (Status) || (*EndPointer != L'\0')) {\r
     return EFI_INVALID_PARAMETER;\r
+  } else {\r
+    return EFI_SUCCESS;\r
   }\r
-\r
-  StringSize = StrLen (String) + 1;\r
-  Ip6Str = (CHAR8 *) AllocatePool (StringSize * sizeof (CHAR8));\r
-  if (Ip6Str == NULL) {\r
-    return EFI_OUT_OF_RESOURCES;\r
-  }\r
-\r
-  UnicodeStrToAsciiStrS (String, Ip6Str, StringSize);\r
-\r
-  Status = NetLibAsciiStrToIp6 (Ip6Str, Ip6Address);\r
-\r
-  FreePool (Ip6Str);\r
-\r
-  return Status;\r
 }\r
 \r
 /**\r
@@ -3053,7 +3104,6 @@ NetLibStrToIp6 (
 \r
   @retval EFI_SUCCESS            Convert to IPv6 address successfully.\r
   @retval EFI_INVALID_PARAMETER  The string is mal-formated or Ip6Address is NULL.\r
-  @retval EFI_OUT_OF_RESOURCES   Fail to perform the operation due to lack of resource.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -3064,96 +3114,29 @@ NetLibStrToIp6andPrefix (
   OUT      UINT8                 *PrefixLength\r
   )\r
 {\r
-  CHAR8                          *Ip6Str;\r
-  UINTN                          StringSize;\r
-  CHAR8                          *PrefixStr;\r
-  CHAR8                          *TempStr;\r
-  EFI_STATUS                     Status;\r
-  UINT8                          Length;\r
-\r
-  if ((String == NULL) || (Ip6Address == NULL) || (PrefixLength == NULL)) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  StringSize = StrLen (String) + 1;\r
-  Ip6Str = (CHAR8 *) AllocatePool (StringSize * sizeof (CHAR8));\r
-  if (Ip6Str == NULL) {\r
-    return EFI_OUT_OF_RESOURCES;\r
-  }\r
-\r
-  UnicodeStrToAsciiStrS (String, Ip6Str, StringSize);\r
+  RETURN_STATUS                  Status;\r
+  CHAR16                         *EndPointer;\r
 \r
-  //\r
-  // Get the sub string describing prefix length.\r
-  //\r
-  TempStr = Ip6Str;\r
-  while (*TempStr != '\0' && (*TempStr != '/')) {\r
-    TempStr++;\r
-  }\r
-\r
-  if (*TempStr == '/') {\r
-    PrefixStr = TempStr + 1;\r
+  Status = StrToIpv6Address (String, &EndPointer, Ip6Address, PrefixLength);\r
+  if (RETURN_ERROR (Status) || (*EndPointer != L'\0')) {\r
+    return EFI_INVALID_PARAMETER;\r
   } else {\r
-    PrefixStr = NULL;\r
-  }\r
-\r
-  //\r
-  // Get the sub string describing IPv6 address and convert it.\r
-  //\r
-  *TempStr = '\0';\r
-\r
-  Status = NetLibAsciiStrToIp6 (Ip6Str, Ip6Address);\r
-  if (EFI_ERROR (Status)) {\r
-    goto Exit;\r
-  }\r
-\r
-  //\r
-  // If input string doesn't indicate the prefix length, return 0xff.\r
-  //\r
-  Length = 0xFF;\r
-\r
-  //\r
-  // Convert the string to prefix length\r
-  //\r
-  if (PrefixStr != NULL) {\r
-\r
-    Status = EFI_INVALID_PARAMETER;\r
-    Length = 0;\r
-    while (*PrefixStr != '\0') {\r
-      if (NET_IS_DIGIT (*PrefixStr)) {\r
-        Length = (UINT8) (Length * 10 + (*PrefixStr - '0'));\r
-        if (Length > IP6_PREFIX_MAX) {\r
-          goto Exit;\r
-        }\r
-      } else {\r
-        goto Exit;\r
-      }\r
-\r
-      PrefixStr++;\r
-    }\r
+    return EFI_SUCCESS;\r
   }\r
-\r
-  *PrefixLength = Length;\r
-  Status        = EFI_SUCCESS;\r
-\r
-Exit:\r
-\r
-  FreePool (Ip6Str);\r
-  return Status;\r
 }\r
 \r
 /**\r
 \r
   Convert one EFI_IPv6_ADDRESS to Null-terminated Unicode string.\r
   The text representation of address is defined in RFC 4291.\r
-  \r
+\r
   @param[in]       Ip6Address     The pointer to the IPv6 address.\r
   @param[out]      String         The buffer to return the converted string.\r
   @param[in]       StringSize     The length in bytes of the input String.\r
-                                  \r
+\r
   @retval EFI_SUCCESS             Convert to string successfully.\r
   @retval EFI_INVALID_PARAMETER   The input parameter is invalid.\r
-  @retval EFI_BUFFER_TOO_SMALL    The BufferSize is too small for the result. BufferSize has been \r
+  @retval EFI_BUFFER_TOO_SMALL    The BufferSize is too small for the result. BufferSize has been\r
                                   updated with the size needed to complete the request.\r
 **/\r
 EFI_STATUS\r
@@ -3179,7 +3162,7 @@ NetLibIp6ToStr (
 \r
   //\r
   // Convert the UINT8 array to an UINT16 array for easy handling.\r
-  // \r
+  //\r
   ZeroMem (Ip6Addr, sizeof (Ip6Addr));\r
   for (Index = 0; Index < 16; Index++) {\r
     Ip6Addr[Index / 2] |= (Ip6Address->Addr[Index] << ((1 - (Index % 2)) << 3));\r
@@ -3211,7 +3194,7 @@ NetLibIp6ToStr (
       }\r
     }\r
   }\r
-  \r
+\r
   if (CurrentZerosStart != DEFAULT_ZERO_START && CurrentZerosLength > 2) {\r
     if (LongestZerosStart == DEFAULT_ZERO_START || LongestZerosLength < CurrentZerosLength) {\r
       LongestZerosStart  = CurrentZerosStart;\r
@@ -3232,7 +3215,7 @@ NetLibIp6ToStr (
     }\r
     Ptr += UnicodeSPrint(Ptr, 10, L"%x", Ip6Addr[Index]);\r
   }\r
-  \r
+\r
   if (LongestZerosStart != DEFAULT_ZERO_START && LongestZerosStart + LongestZerosLength == 8) {\r
     *Ptr++ = L':';\r
   }\r
@@ -3250,6 +3233,8 @@ NetLibIp6ToStr (
 /**\r
   This function obtains the system guid from the smbios table.\r
 \r
+  If SystemGuid is NULL, then ASSERT().\r
+\r
   @param[out]  SystemGuid     The pointer of the returned system guid.\r
 \r
   @retval EFI_SUCCESS         Successfully obtained the system guid.\r
@@ -3269,6 +3254,8 @@ NetLibGetSystemGuid (
   SMBIOS_STRUCTURE_POINTER      SmbiosEnd;\r
   CHAR8                         *String;\r
 \r
+  ASSERT (SystemGuid != NULL);\r
+\r
   SmbiosTable = NULL;\r
   Status = EfiGetSystemConfigurationTable (&gEfiSmbios3TableGuid, (VOID **) &Smbios30Table);\r
   if (!(EFI_ERROR (Status) || Smbios30Table == NULL)) {\r
@@ -3280,7 +3267,7 @@ NetLibGetSystemGuid (
       return EFI_NOT_FOUND;\r
     }\r
     Smbios.Hdr    = (SMBIOS_STRUCTURE *) (UINTN) SmbiosTable->TableAddress;\r
-    SmbiosEnd.Raw = (UINT8 *) (UINTN) (SmbiosTable->TableAddress + SmbiosTable->TableLength);\r
+    SmbiosEnd.Raw = (UINT8 *) ((UINTN) SmbiosTable->TableAddress + SmbiosTable->TableLength);\r
   }\r
 \r
   do {\r
@@ -3291,7 +3278,7 @@ NetLibGetSystemGuid (
         //\r
         return EFI_NOT_FOUND;\r
       }\r
-      \r
+\r
       //\r
       // SMBIOS tables are byte packed so we need to do a byte copy to\r
       // prevend alignment faults on Itanium-based platform.\r
@@ -3305,12 +3292,12 @@ NetLibGetSystemGuid (
     // 1. Formatted section; 2. Unformatted string section. So, 2 steps are needed\r
     // to skip one SMBIOS structure.\r
     //\r
-    \r
+\r
     //\r
     // Step 1: Skip over formatted section.\r
     //\r
     String = (CHAR8 *) (Smbios.Raw + Smbios.Hdr->Length);\r
-  \r
+\r
     //\r
     // Step 2: Skip over unformated string section.\r
     //\r
@@ -3328,26 +3315,29 @@ NetLibGetSystemGuid (
         //\r
         Smbios.Raw = (UINT8 *)++String;\r
         break;\r
-      }    \r
+      }\r
     } while (TRUE);\r
   } while (Smbios.Raw < SmbiosEnd.Raw);\r
   return EFI_NOT_FOUND;\r
 }\r
 \r
 /**\r
-  Create Dns QName according the queried domain name. \r
-  QName is a domain name represented as a sequence of labels, \r
-  where each label consists of a length octet followed by that \r
-  number of octets. The QName terminates with the zero \r
-  length octet for the null label of the root. Caller should \r
+  Create Dns QName according the queried domain name.\r
+\r
+  If DomainName is NULL, then ASSERT().\r
+\r
+  QName is a domain name represented as a sequence of labels,\r
+  where each label consists of a length octet followed by that\r
+  number of octets. The QName terminates with the zero\r
+  length octet for the null label of the root. Caller should\r
   take responsibility to free the buffer in returned pointer.\r
 \r
-  @param  DomainName    The pointer to the queried domain name string.  \r
+  @param  DomainName    The pointer to the queried domain name string.\r
 \r
   @retval NULL          Failed to fill QName.\r
   @return               QName filled successfully.\r
-  \r
-**/ \r
+\r
+**/\r
 CHAR8 *\r
 EFIAPI\r
 NetLibCreateDnsQName (\r
@@ -3361,16 +3351,18 @@ NetLibCreateDnsQName (
   UINTN                 Len;\r
   UINTN                 Index;\r
 \r
+  ASSERT (DomainName != NULL);\r
+\r
   QueryName     = NULL;\r
   QueryNameSize = 0;\r
   Header        = NULL;\r
   Tail          = NULL;\r
 \r
   //\r
-  // One byte for first label length, one byte for terminated length zero. \r
+  // One byte for first label length, one byte for terminated length zero.\r
   //\r
   QueryNameSize = StrLen (DomainName) + 2;\r
-  \r
+\r
   if (QueryNameSize > DNS_MAX_NAME_SIZE) {\r
     return NULL;\r
   }\r
@@ -3379,7 +3371,7 @@ NetLibCreateDnsQName (
   if (QueryName == NULL) {\r
     return NULL;\r
   }\r
-  \r
+\r
   Header = QueryName;\r
   Tail = Header + 1;\r
   Len = 0;\r