]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Library/DxeNetLib/DxeNetLib.c
Code clean up in NetLib:
[mirror_edk2.git] / MdeModulePkg / Library / DxeNetLib / DxeNetLib.c
index b42054f544777b46e144acc824eb7ee8a8c1194d..ffcd70ca844109d3171ef01b10e11d5d2bf4efdd 100644 (file)
@@ -1,6 +1,6 @@
 /** @file\r
   Network library.\r
-  \r
+\r
 Copyright (c) 2005 - 2009, Intel Corporation.<BR>\r
 All rights reserved. This program and the accompanying materials\r
 are licensed and made available under the terms and conditions of the BSD License\r
@@ -16,6 +16,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/ManagedNetwork.h>\r
 #include <Protocol/HiiConfigRouting.h>\r
 #include <Protocol/ComponentName.h>\r
 #include <Protocol/ComponentName2.h>\r
@@ -33,14 +34,12 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #include <Library/HiiLib.h>\r
 #include <Library/PrintLib.h>\r
 \r
-GLOBAL_REMOVE_IF_UNREFERENCED CONST CHAR8 mNetLibHexStr[] = {'0','1','2','3','4','5','6','7','8','9','A','B','C','D','E','F'};\r
-\r
 #define NIC_ITEM_CONFIG_SIZE   sizeof (NIC_IP4_CONFIG_INFO) + sizeof (EFI_IP4_ROUTE_TABLE) * MAX_IP4_CONFIG_IN_VARIABLE\r
 \r
 //\r
 // All the supported IP4 maskes in host byte order.\r
 //\r
-IP4_ADDR  gIp4AllMasks[IP4_MASK_NUM] = {\r
+GLOBAL_REMOVE_IF_UNREFERENCED IP4_ADDR  gIp4AllMasks[IP4_MASK_NUM] = {\r
   0x00000000,\r
   0x80000000,\r
   0xC0000000,\r
@@ -79,26 +78,25 @@ IP4_ADDR  gIp4AllMasks[IP4_MASK_NUM] = {
   0xFFFFFFFF,\r
 };\r
 \r
-EFI_IPv4_ADDRESS  mZeroIp4Addr = {{0, 0, 0, 0}};\r
+GLOBAL_REMOVE_IF_UNREFERENCED EFI_IPv4_ADDRESS  mZeroIp4Addr = {{0, 0, 0, 0}};\r
 \r
 //\r
-// Any error level digitally larger than mNetDebugLevelMax \r
+// Any error level digitally larger than mNetDebugLevelMax\r
 // will be silently discarded.\r
 //\r
-UINTN  mNetDebugLevelMax = NETDEBUG_LEVEL_ERROR;\r
-UINT32 mSyslogPacketSeq  = 0xDEADBEEF;\r
+GLOBAL_REMOVE_IF_UNREFERENCED UINTN  mNetDebugLevelMax = NETDEBUG_LEVEL_ERROR;\r
+GLOBAL_REMOVE_IF_UNREFERENCED UINT32 mSyslogPacketSeq  = 0xDEADBEEF;\r
 \r
-// \r
-// You can change mSyslogDstMac mSyslogDstIp and mSyslogSrcIp \r
-// here to direct the syslog packets to the syslog deamon. The \r
-// default is broadcast to both the ethernet and IP. \r
 //\r
-UINT8  mSyslogDstMac[NET_ETHER_ADDR_LEN] = {0xff, 0xff, 0xff, 0xff, 0xff, 0xff};\r
-UINT32 mSyslogDstIp                      = 0xffffffff;\r
-UINT32 mSyslogSrcIp                      = 0;\r
+// You can change mSyslogDstMac mSyslogDstIp and mSyslogSrcIp\r
+// here to direct the syslog packets to the syslog deamon. The\r
+// default is broadcast to both the ethernet and IP.\r
+//\r
+GLOBAL_REMOVE_IF_UNREFERENCED UINT8  mSyslogDstMac[NET_ETHER_ADDR_LEN] = {0xff, 0xff, 0xff, 0xff, 0xff, 0xff};\r
+GLOBAL_REMOVE_IF_UNREFERENCED UINT32 mSyslogDstIp                      = 0xffffffff;\r
+GLOBAL_REMOVE_IF_UNREFERENCED UINT32 mSyslogSrcIp                      = 0;\r
 \r
-CHAR8 *\r
-mMonthName[] = {\r
+GLOBAL_REMOVE_IF_UNREFERENCED CHAR8 *mMonthName[] = {\r
   "Jan",\r
   "Feb",\r
   "Mar",\r
@@ -114,7 +112,7 @@ mMonthName[] = {
 };\r
 \r
 /**\r
-  Locate the handles that support SNP, then open one of them \r
+  Locate the handles that support SNP, then open one of them\r
   to send the syslog packets. The caller isn't required to close\r
   the SNP after use because the SNP is opened by HandleProtocol.\r
 \r
@@ -147,12 +145,12 @@ SyslogLocateSnp (
   if (EFI_ERROR (Status) || (HandleCount == 0)) {\r
     return NULL;\r
   }\r
-  \r
+\r
   //\r
   // Try to open one of the ethernet SNP protocol to send packet\r
   //\r
   Snp = NULL;\r
-  \r
+\r
   for (Index = 0; Index < HandleCount; Index++) {\r
     Status = gBS->HandleProtocol (\r
                     Handles[Index],\r
@@ -160,10 +158,10 @@ SyslogLocateSnp (
                     (VOID **) &Snp\r
                     );\r
 \r
-    if ((Status == EFI_SUCCESS) && (Snp != NULL) && \r
+    if ((Status == EFI_SUCCESS) && (Snp != NULL) &&\r
         (Snp->Mode->IfType == NET_IFTYPE_ETHERNET) &&\r
         (Snp->Mode->MaxPacketSize >= NET_SYSLOG_PACKET_LEN)) {\r
-        \r
+\r
       break;\r
     }\r
 \r
@@ -176,18 +174,18 @@ SyslogLocateSnp (
 \r
 /**\r
   Transmit a syslog packet synchronously through SNP. The Packet\r
-  already has the ethernet header prepended. This function should \r
+  already has the ethernet header prepended. This function should\r
   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
+  This code snip is copied from MNP.\r
+\r
+  @param[in] Packet          The Syslog packet\r
+  @param[in] Length          The length of the packet\r
 \r
-  @param[in] Packet  - The Syslog packet \r
-  @param[in] Length  - The length of the packet\r
+  @retval EFI_DEVICE_ERROR   Failed to locate a usable SNP protocol\r
+  @retval EFI_TIMEOUT        Timeout happened to send the packet.\r
+  @retval EFI_SUCCESS        Packet is sent.\r
 \r
-  @retval EFI_DEVICE_ERROR - Failed to locate a usable SNP protocol\r
-  @retval EFI_TIMEOUT      - Timeout happened to send the packet.\r
-  @retval EFI_SUCCESS      - Packet is sent.\r
-  \r
 **/\r
 EFI_STATUS\r
 SyslogSendPacket (\r
@@ -241,7 +239,7 @@ SyslogSendPacket (
       Status = EFI_DEVICE_ERROR;\r
       break;\r
     }\r
-    \r
+\r
     //\r
     // If Status is EFI_SUCCESS, the packet is put in the transmit queue.\r
     // if Status is EFI_NOT_READY, the transmit engine of the network\r
@@ -265,7 +263,7 @@ SyslogSendPacket (
     if ((Status == EFI_SUCCESS) || (Status == EFI_TIMEOUT)) {\r
       break;\r
     }\r
-    \r
+\r
     //\r
     // Status is EFI_NOT_READY. Restart the timer event and\r
     // call Snp->Transmit again.\r
@@ -281,20 +279,18 @@ ON_EXIT:
 }\r
 \r
 /**\r
-  Build a syslog packet, including the Ethernet/Ip/Udp headers \r
-  and user's message. \r
-  \r
-  @param[in]  Level   - Syslog servity level\r
-  @param[in]  Module  - The module that generates the log\r
-  @param[in]  File    - The file that contains the current log\r
-  @param[in]  Line    - The line of code in the File that contains the current log\r
-  @param[in]  Message - The log message\r
-  @param[in]  BufLen  - The lenght of the Buf\r
-  @param[out] Buf     - The buffer to put the packet data\r
+  Build a syslog packet, including the Ethernet/Ip/Udp headers\r
+  and user's message.\r
 \r
-Returns:\r
+  @param[in]  Level     Syslog servity level\r
+  @param[in]  Module    The module that generates the log\r
+  @param[in]  File      The file that contains the current log\r
+  @param[in]  Line      The line of code in the File that contains the current log\r
+  @param[in]  Message   The log message\r
+  @param[in]  BufLen    The lenght of the Buf\r
+  @param[out] Buf       The buffer to put the packet data\r
 \r
-  The length of the syslog packet built.\r
+  @return The length of the syslog packet built.\r
 \r
 **/\r
 UINT32\r
@@ -305,7 +301,7 @@ SyslogBuildPacket (
   IN  UINT32                Line,\r
   IN  UINT8                 *Message,\r
   IN  UINT32                BufLen,\r
-  OUT CHAR8                 *Buf  \r
+  OUT CHAR8                 *Buf\r
   )\r
 {\r
   ETHER_HEAD                *Ether;\r
@@ -316,7 +312,7 @@ SyslogBuildPacket (
   UINT32                    Len;\r
 \r
   //\r
-  // Fill in the Ethernet header. Leave alone the source MAC. \r
+  // Fill in the Ethernet header. Leave alone the source MAC.\r
   // SyslogSendPacket will fill in the address for us.\r
   //\r
   Ether = (ETHER_HEAD *) Buf;\r
@@ -374,7 +370,7 @@ SyslogBuildPacket (
                     BufLen,\r
                     "<%d> %a %d %d:%d:%d ",\r
                     Pri,\r
-                    mMonthName [Time.Month-1], \r
+                    mMonthName [Time.Month-1],\r
                     Time.Day,\r
                     Time.Hour,\r
                     Time.Minute,\r
@@ -383,9 +379,9 @@ SyslogBuildPacket (
   Len--;\r
 \r
   Len += (UINT32) AsciiSPrint (\r
-                    Buf + Len, \r
-                    BufLen - Len, \r
-                    "Tiano %a: %a (Line: %d File: %a)", \r
+                    Buf + Len,\r
+                    BufLen - Len,\r
+                    "Tiano %a: %a (Line: %d File: %a)",\r
                     Module,\r
                     Message,\r
                     Line,\r
@@ -407,23 +403,23 @@ SyslogBuildPacket (
 }\r
 \r
 /**\r
-  Allocate a buffer, then format the message to it. This is a \r
-  help function for the NET_DEBUG_XXX macros. The PrintArg of \r
-  these macros treats the variable length print parameters as a \r
+  Allocate a buffer, then format the message to it. This is a\r
+  help function for the NET_DEBUG_XXX macros. The PrintArg of\r
+  these macros treats the variable length print parameters as a\r
   single parameter, and pass it to the NetDebugASPrint. For\r
   example, NET_DEBUG_TRACE ("Tcp", ("State transit to %a\n", Name))\r
-  if extracted to:   \r
-  \r
+  if extracted to:\r
+\r
          NetDebugOutput (\r
-           NETDEBUG_LEVEL_TRACE, \r
-           "Tcp", \r
+           NETDEBUG_LEVEL_TRACE,\r
+           "Tcp",\r
            __FILE__,\r
            __LINE__,\r
-           NetDebugASPrint ("State transit to %a\n", Name) \r
-         )  \r
\r
+           NetDebugASPrint ("State transit to %a\n", Name)\r
+         )\r
+\r
   @param Format  The ASCII format string.\r
-  @param ...     The variable length parameter whose format is determined \r
+  @param ...     The variable length parameter whose format is determined\r
                  by the Format string.\r
 \r
   @return        The buffer containing the formatted message,\r
@@ -466,12 +462,12 @@ NetDebugASPrint (
 \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_SUCCESS           The log is discard because that it is more verbose \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
+**/\r
 EFI_STATUS\r
 NetDebugOutput (\r
-  IN UINT32                    Level, \r
+  IN UINT32                    Level,\r
   IN UINT8                     *Module,\r
   IN UINT8                     *File,\r
   IN UINT32                    Line,\r
@@ -493,7 +489,7 @@ NetDebugOutput (
     Status = EFI_SUCCESS;\r
     goto ON_EXIT;\r
   }\r
-  \r
+\r
   //\r
   // Allocate a maxium of 1024 bytes, the caller should ensure\r
   // that the message plus the ethernet/ip/udp header is shorter\r
@@ -505,7 +501,7 @@ NetDebugOutput (
     Status = EFI_OUT_OF_RESOURCES;\r
     goto ON_EXIT;\r
   }\r
-  \r
+\r
   //\r
   // Build the message: Ethernet header + IP header + Udp Header + user data\r
   //\r
@@ -528,8 +524,8 @@ ON_EXIT:
   return Status;\r
 }\r
 /**\r
-  Return the length of the mask. \r
-  \r
+  Return the length of the mask.\r
+\r
   Return the length of the mask, the correct value is from 0 to 32.\r
   If the mask is invalid, return the invalid length 33, which is IP4_MASK_NUM.\r
   NetMask is in the host byte order.\r
@@ -537,7 +533,7 @@ ON_EXIT:
   @param[in]  NetMask              The netmask to get the length from.\r
 \r
   @return The length of the netmask, IP4_MASK_NUM if the mask is invalid.\r
-  \r
+\r
 **/\r
 INTN\r
 EFIAPI\r
@@ -561,19 +557,19 @@ NetGetMaskLength (
 /**\r
   Return the class of the IP address, such as class A, B, C.\r
   Addr is in host byte order.\r
-  \r
+\r
   The address of class A  starts with 0.\r
   If the address belong to class A, return IP4_ADDR_CLASSA.\r
-  The address of class B  starts with 10. \r
+  The address of class B  starts with 10.\r
   If the address belong to class B, return IP4_ADDR_CLASSB.\r
-  The address of class C  starts with 110. \r
+  The address of class C  starts with 110.\r
   If the address belong to class C, return IP4_ADDR_CLASSC.\r
-  The address of class D  starts with 1110. \r
+  The address of class D  starts with 1110.\r
   If the address belong to class D, return IP4_ADDR_CLASSD.\r
   The address of class E  starts with 1111.\r
   If the address belong to class E, return IP4_ADDR_CLASSE.\r
 \r
-  \r
+\r
   @param[in]   Addr                  The address to get the class from.\r
 \r
   @return IP address class, such as IP4_ADDR_CLASSA.\r
@@ -611,10 +607,10 @@ NetGetIpClass (
 /**\r
   Check whether the IP is a valid unicast address according to\r
   the netmask. If NetMask is zero, use the IP address's class to get the default mask.\r
-  \r
+\r
   If Ip is 0, IP is not a valid unicast address.\r
   Class D address is used for multicasting and class E address is reserved for future. If Ip\r
-  belongs to class D or class E, IP is not a valid unicast address. \r
+  belongs to class D or class E, IP is not a valid unicast address.\r
   If all bits of the host address of IP are 0 or 1, IP is also not a valid unicast address.\r
 \r
   @param[in]  Ip                    The IP to check against.\r
@@ -656,21 +652,21 @@ NetIp4IsUnicast (
   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
   ::1, it is also not a valid unicast address to be assigned to any physical\r
-  interface. \r
+  interface.\r
 \r
   @param[in]  Ip6                   The IPv6 address to check against.\r
 \r
   @return TRUE if Ip6 is a valid unicast address on the network, otherwise FALSE.\r
 \r
-**/ \r
+**/\r
 BOOLEAN\r
 NetIp6IsValidUnicast (\r
   IN EFI_IPv6_ADDRESS       *Ip6\r
-  ) \r
+  )\r
 {\r
   UINT8 Byte;\r
   UINT8 Index;\r
-  \r
+\r
   if (Ip6->Addr[0] == 0xFF) {\r
     return FALSE;\r
   }\r
@@ -687,7 +683,7 @@ NetIp6IsValidUnicast (
     return FALSE;\r
   }\r
 \r
-  return TRUE;  \r
+  return TRUE;\r
 }\r
 \r
 /**\r
@@ -697,7 +693,7 @@ NetIp6IsValidUnicast (
 \r
   @retval TRUE     - Yes, unspecified\r
   @retval FALSE    - No\r
-  \r
+\r
 **/\r
 BOOLEAN\r
 NetIp6IsUnspecifiedAddr (\r
@@ -722,7 +718,7 @@ NetIp6IsUnspecifiedAddr (
 \r
   @retval TRUE  - Yes, link-local address\r
   @retval FALSE - No\r
-  \r
+\r
 **/\r
 BOOLEAN\r
 NetIp6IsLinkLocalAddr (\r
@@ -730,13 +726,13 @@ NetIp6IsLinkLocalAddr (
   )\r
 {\r
   UINT8 Index;\r
-  \r
+\r
   ASSERT (Ip6 != NULL);\r
 \r
   if (Ip6->Addr[0] != 0xFE) {\r
     return FALSE;\r
   }\r
-  \r
+\r
   if (Ip6->Addr[1] != 0x80) {\r
     return FALSE;\r
   }\r
@@ -759,7 +755,7 @@ NetIp6IsLinkLocalAddr (
 \r
   @retval TRUE            - Yes, connected.\r
   @retval FALSE           - No.\r
-  \r
+\r
 **/\r
 BOOLEAN\r
 NetIp6IsNetEqual (\r
@@ -773,14 +769,14 @@ NetIp6IsNetEqual (
   UINT8 Mask;\r
 \r
   ASSERT (Ip1 != NULL && Ip2 != NULL);\r
-  \r
+\r
   if (PrefixLength == 0) {\r
     return TRUE;\r
   }\r
 \r
   Byte = (UINT8) (PrefixLength / 8);\r
   Bit  = (UINT8) (PrefixLength % 8);\r
-  \r
+\r
   if (CompareMem (Ip1, Ip2, Byte) != 0) {\r
     return FALSE;\r
   }\r
@@ -790,9 +786,9 @@ NetIp6IsNetEqual (
 \r
     if ((Ip1->Addr[Byte] & Mask) != (Ip2->Addr[Byte] & Mask)) {\r
       return FALSE;\r
-    }    \r
+    }\r
   }\r
-  \r
+\r
   return TRUE;\r
 }\r
 \r
@@ -831,11 +827,11 @@ Ip6Swap128 (
 \r
 /**\r
   Initialize a random seed using current time.\r
-  \r
-  Get current time first. Then initialize a random seed based on some basic \r
-  mathematics operation on the hour, day, minute, second, nanosecond and year \r
+\r
+  Get current time first. Then initialize a random seed based on some basic\r
+  mathematics operation on the hour, day, minute, second, nanosecond and year\r
   of the current time.\r
-  \r
+\r
   @return The random seed initialized with current time.\r
 \r
 **/\r
@@ -859,8 +855,8 @@ NetRandomInitSeed (
 \r
 /**\r
   Extract a UINT32 from a byte stream.\r
-  \r
-  Copy a UINT32 from a byte stream, then converts it from Network \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
   @param[in]  Buf                 The buffer to extract the UINT32.\r
@@ -882,14 +878,14 @@ NetGetUint32 (
 \r
 \r
 /**\r
-  Put a UINT32 to the byte stream in network byte order. \r
-  \r
-  Converts a UINT32 from host byte order to network byte order. Then copy it to the \r
+  Put a UINT32 to the byte stream in network byte order.\r
+\r
+  Converts a UINT32 from host byte order to network byte order. Then copy it to the\r
   byte stream.\r
 \r
   @param[in, out]  Buf          The buffer to put the UINT32.\r
   @param[in]      Data          The data to put.\r
-  \r
+\r
 **/\r
 VOID\r
 EFIAPI\r
@@ -905,16 +901,16 @@ NetPutUint32 (
 \r
 /**\r
   Remove the first node entry on the list, and return the removed node entry.\r
-  \r
+\r
   Removes the first node Entry from a doubly linked list. It is up to the caller of\r
   this function to release the memory used by the first node if that is required. On\r
-  exit, the removed node is returned. \r
+  exit, the removed node is returned.\r
 \r
   If Head is NULL, then ASSERT().\r
   If Head was not initialized, then ASSERT().\r
   If PcdMaximumLinkedListLength is not zero, and the number of nodes in the\r
   linked list including the head node is greater than or equal to PcdMaximumLinkedListLength,\r
-  then ASSERT().    \r
+  then ASSERT().\r
 \r
   @param[in, out]  Head                  The list header.\r
 \r
@@ -953,14 +949,14 @@ NetListRemoveHead (
 \r
   Removes the last node entry from a doubly linked list. It is up to the caller of\r
   this function to release the memory used by the first node if that is required. On\r
-  exit, the removed node is returned. \r
+  exit, the removed node is returned.\r
 \r
   If Head is NULL, then ASSERT().\r
   If Head was not initialized, then ASSERT().\r
   If PcdMaximumLinkedListLength is not zero, and the number of nodes in the\r
   linked list including the head node is greater than or equal to PcdMaximumLinkedListLength,\r
-  then ASSERT(). \r
-  \r
+  then ASSERT().\r
+\r
   @param[in, out]  Head                  The list head.\r
 \r
   @return The last node entry that is removed from the list, NULL if the list is empty.\r
@@ -995,10 +991,10 @@ NetListRemoveTail (
 \r
 /**\r
   Insert a new node entry after a designated node entry of a doubly linked list.\r
-  \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
+\r
   @param[in, out]  PrevEntry             The previous entry to insert after.\r
   @param[in, out]  NewEntry              The new entry to insert.\r
 \r
@@ -1019,10 +1015,10 @@ NetListInsertAfter (
 \r
 /**\r
   Insert a new node entry before a designated node entry of a doubly linked list.\r
-  \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
+\r
   @param[in, out]  PostEntry             The entry to insert before.\r
   @param[in, out]  NewEntry              The new entry to insert.\r
 \r
@@ -1043,15 +1039,15 @@ NetListInsertBefore (
 \r
 /**\r
   Initialize the netmap. Netmap is a reposity to keep the <Key, Value> pairs.\r
\r
-  Initialize the forward and backward links of two head nodes donated by Map->Used \r
+\r
+  Initialize the forward and backward links of two head nodes donated by Map->Used\r
   and Map->Recycled of two doubly linked lists.\r
   Initializes the count of the <Key, Value> pairs in the netmap to zero.\r
-   \r
+\r
   If Map is NULL, then ASSERT().\r
   If the address of Map->Used is NULL, then ASSERT().\r
   If the address of Map->Recycled is NULl, then ASSERT().\r
\r
+\r
   @param[in, out]  Map                   The netmap to initialize.\r
 \r
 **/\r
@@ -1071,13 +1067,13 @@ NetMapInit (
 \r
 /**\r
   To clean up the netmap, that is, release allocated memories.\r
-  \r
+\r
   Removes all nodes of the Used doubly linked list and free memory of all related netmap items.\r
   Removes all nodes of the Recycled doubly linked list and free memory of all related netmap items.\r
   The number of the <Key, Value> pairs in the netmap is set to be zero.\r
-  \r
+\r
   If Map is NULL, then ASSERT().\r
-  \r
+\r
   @param[in, out]  Map                   The netmap to clean up.\r
 \r
 **/\r
@@ -1117,12 +1113,12 @@ NetMapClean (
 \r
 /**\r
   Test whether the netmap is empty and return true if it is.\r
-  \r
+\r
   If the number of the <Key, Value> pairs in the netmap is zero, return TRUE.\r
-   \r
+\r
   If Map is NULL, then ASSERT().\r
\r
-  \r
+\r
+\r
   @param[in]  Map                   The net map to test.\r
 \r
   @return TRUE if the netmap is empty, otherwise FALSE.\r
@@ -1158,15 +1154,15 @@ NetMapGetCount (
 \r
 \r
 /**\r
-  Return one allocated item. \r
-  \r
-  If the Recycled doubly linked list of the netmap is empty, it will try to allocate \r
+  Return one allocated item.\r
+\r
+  If the Recycled doubly linked list of the netmap is empty, it will try to allocate\r
   a batch of items if there are enough resources and add corresponding nodes to the begining\r
   of the Recycled doubly linked list of the netmap. Otherwise, it will directly remove\r
   the fist node entry of the Recycled doubly linked list and return the corresponding item.\r
-  \r
+\r
   If Map is NULL, then ASSERT().\r
-  \r
+\r
   @param[in, out]  Map          The netmap to allocate item for.\r
 \r
   @return                       The allocated item. If NULL, the\r
@@ -1211,13 +1207,13 @@ NetMapAllocItem (
 \r
 /**\r
   Allocate an item to save the <Key, Value> pair to the head of the netmap.\r
-  \r
+\r
   Allocate an item to save the <Key, Value> pair and add corresponding node entry\r
-  to the beginning of the Used doubly linked list. The number of the <Key, Value> \r
+  to the beginning of the Used doubly linked list. The number of the <Key, Value>\r
   pairs in the netmap increase by 1.\r
 \r
   If Map is NULL, then ASSERT().\r
-  \r
+\r
   @param[in, out]  Map                   The netmap to insert into.\r
   @param[in]       Key                   The user's key.\r
   @param[in]       Value                 The user's value for the key.\r
@@ -1257,11 +1253,11 @@ NetMapInsertHead (
   Allocate an item to save the <Key, Value> pair to the tail of the netmap.\r
 \r
   Allocate an item to save the <Key, Value> pair and add corresponding node entry\r
-  to the tail of the Used doubly linked list. The number of the <Key, Value> \r
+  to the tail of the Used doubly linked list. The number of the <Key, Value>\r
   pairs in the netmap increase by 1.\r
 \r
   If Map is NULL, then ASSERT().\r
-  \r
+\r
   @param[in, out]  Map                   The netmap to insert into.\r
   @param[in]       Key                   The user's key.\r
   @param[in]       Value                 The user's value for the key.\r
@@ -1327,12 +1323,12 @@ NetItemInMap (
 \r
 /**\r
   Find the key in the netmap and returns the point to the item contains the Key.\r
-  \r
-  Iterate the Used doubly linked list of the netmap to get every item. Compare the key of every \r
+\r
+  Iterate the Used doubly linked list of the netmap to get every item. Compare the key of every\r
   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
-  \r
+\r
   @param[in]  Map                   The netmap to search within.\r
   @param[in]  Key                   The key to search.\r
 \r
@@ -1365,16 +1361,16 @@ NetMapFindKey (
 \r
 /**\r
   Remove the node entry of the item from the netmap and return the key of the removed item.\r
-  \r
-  Remove the node entry of the item from the Used doubly linked list of the netmap. \r
-  The number of the <Key, Value> pairs in the netmap decrease by 1. Then add the node \r
+\r
+  Remove the node entry of the item from the Used doubly linked list of the netmap.\r
+  The number of the <Key, Value> pairs in the netmap decrease by 1. Then add the node\r
   entry of the item to the Recycled doubly linked list of the netmap. If Value is not NULL,\r
   Value will point to the value of the item. It returns the key of the removed item.\r
-  \r
+\r
   If Map is NULL, then ASSERT().\r
   If Item is NULL, then ASSERT().\r
   if item in not in the netmap, then ASSERT().\r
-  \r
+\r
   @param[in, out]  Map                   The netmap to remove the item from.\r
   @param[in, out]  Item                  The item to remove.\r
   @param[out]      Value                 The variable to receive the value if not NULL.\r
@@ -1408,14 +1404,14 @@ NetMapRemoveItem (
 /**\r
   Remove the first node entry on the netmap and return the key of the removed item.\r
 \r
-  Remove the first node entry from the Used doubly linked list of the netmap. \r
-  The number of the <Key, Value> pairs in the netmap decrease by 1. Then add the node \r
+  Remove the first node entry from the Used doubly linked list of the netmap.\r
+  The number of the <Key, Value> pairs in the netmap decrease by 1. Then add the node\r
   entry to the Recycled doubly linked list of the netmap. If parameter Value is not NULL,\r
   parameter Value will point to the value of the item. It returns the key of the removed item.\r
-  \r
+\r
   If Map is NULL, then ASSERT().\r
   If the Used doubly linked list is empty, then ASSERT().\r
-  \r
+\r
   @param[in, out]  Map                   The netmap to remove the head from.\r
   @param[out]      Value                 The variable to receive the value if not NULL.\r
 \r
@@ -1453,14 +1449,14 @@ NetMapRemoveHead (
 /**\r
   Remove the last node entry on the netmap and return the key of the removed item.\r
 \r
-  Remove the last node entry from the Used doubly linked list of the netmap. \r
-  The number of the <Key, Value> pairs in the netmap decrease by 1. Then add the node \r
+  Remove the last node entry from the Used doubly linked list of the netmap.\r
+  The number of the <Key, Value> pairs in the netmap decrease by 1. Then add the node\r
   entry to the Recycled doubly linked list of the netmap. If parameter Value is not NULL,\r
   parameter Value will point to the value of the item. It returns the key of the removed item.\r
-  \r
+\r
   If Map is NULL, then ASSERT().\r
   If the Used doubly linked list is empty, then ASSERT().\r
-  \r
+\r
   @param[in, out]  Map                   The netmap to remove the tail from.\r
   @param[out]      Value                 The variable to receive the value if not NULL.\r
 \r
@@ -1497,14 +1493,14 @@ NetMapRemoveTail (
 \r
 /**\r
   Iterate through the netmap and call CallBack for each item.\r
-  \r
+\r
   It will contiue the traverse if CallBack returns EFI_SUCCESS, otherwise, break\r
-  from the loop. It returns the CallBack's last return value. This function is \r
+  from the loop. It returns the CallBack's last return value. This function is\r
   delete safe for the current item.\r
 \r
   If Map is NULL, then ASSERT().\r
   If CallBack is NULL, then ASSERT().\r
-  \r
+\r
   @param[in]  Map                   The Map to iterate through.\r
   @param[in]  CallBack              The callback function to call for each item.\r
   @param[in]  Arg                   The opaque parameter to the callback.\r
@@ -1555,7 +1551,7 @@ NetMapIterate (
 \r
   Disconnect the driver specified by ImageHandle from all the devices in the handle database.\r
   Uninstall all the protocols installed in the driver entry point.\r
-  \r
+\r
   @param[in]  ImageHandle       The drivers' driver image.\r
 \r
   @retval EFI_SUCCESS           The image is unloaded.\r
@@ -1669,12 +1665,12 @@ NetLibDefaultUnload (
 \r
 /**\r
   Create a child of the service that is identified by ServiceBindingGuid.\r
-  \r
+\r
   Get the ServiceBinding Protocol first, then use it to create a child.\r
 \r
   If ServiceBindingGuid is NULL, then ASSERT().\r
   If ChildHandle is NULL, then ASSERT().\r
-  \r
+\r
   @param[in]       Controller            The controller which has the service installed.\r
   @param[in]       Image                 The image handle used to open service.\r
   @param[in]       ServiceBindingGuid    The service's Guid.\r
@@ -1725,11 +1721,11 @@ NetLibCreateServiceChild (
 \r
 /**\r
   Destory a child of the service that is identified by ServiceBindingGuid.\r
-  \r
+\r
   Get the ServiceBinding Protocol first, then use it to destroy a child.\r
-  \r
+\r
   If ServiceBindingGuid is NULL, then ASSERT().\r
-  \r
+\r
   @param[in]   Controller            The controller which has the service installed.\r
   @param[in]   Image                 The image handle used to open service.\r
   @param[in]   ServiceBindingGuid    The service's Guid.\r
@@ -1793,7 +1789,7 @@ NetLibDestroyServiceChild (
                                      get the simple network protocol.\r
   @param[out]  MacString             The pointer to store the address of the string\r
                                      representation of  the mac address.\r
-  \r
+\r
   @retval EFI_SUCCESS           Convert the mac address a unicode string successfully.\r
   @retval EFI_OUT_OF_RESOURCES  There are not enough memory resource.\r
   @retval Others                Failed to open the simple network protocol.\r
@@ -1811,6 +1807,7 @@ NetLibGetMacString (
   EFI_SIMPLE_NETWORK_PROTOCOL  *Snp;\r
   EFI_SIMPLE_NETWORK_MODE      *Mode;\r
   CHAR16                       *MacAddress;\r
+  UINT8                        *HwAddress;\r
   UINTN                        Index;\r
 \r
   *MacString = NULL;\r
@@ -1840,18 +1837,18 @@ NetLibGetMacString (
   if (MacAddress == NULL) {\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
+  *MacString = MacAddress;\r
 \r
   //\r
   // Convert the mac address into a unicode string.\r
   //\r
+  HwAddress = Mode->CurrentAddress.Addr;\r
   for (Index = 0; Index < Mode->HwAddressSize; Index++) {\r
-    MacAddress[Index * 2]     = (CHAR16) mNetLibHexStr[(Mode->CurrentAddress.Addr[Index] >> 4) & 0x0F];\r
-    MacAddress[Index * 2 + 1] = (CHAR16) mNetLibHexStr[Mode->CurrentAddress.Addr[Index] & 0x0F];\r
+    MacAddress += UnicodeValueToString (MacAddress, PREFIX_ZERO | RADIX_HEX, *(HwAddress++), 2);\r
   }\r
 \r
   MacAddress[Mode->HwAddressSize * 2] = L'\0';\r
 \r
-  *MacString = MacAddress;\r
 \r
   return EFI_SUCCESS;\r
 }\r
@@ -1860,11 +1857,11 @@ NetLibGetMacString (
   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 NIC Ip4 Config Protocol installed, the \r
+  If the controller handle does not have the NIC Ip4 Config Protocol installed, the\r
   default address is static. If the EFI variable to save the configuration is not found,\r
-  the default address is static. Otherwise, get the result from the EFI variable which \r
+  the default address is static. Otherwise, get the result from the EFI variable which\r
   saving the configuration.\r
-   \r
+\r
   @param[in]   Controller     The controller handle which has the NIC Ip4 Config Protocol\r
                               relative with the default address to judge.\r
 \r
@@ -1911,7 +1908,7 @@ NetLibDefaultAddressIsStatic (
   if (ConfigHdr == NULL) {\r
     return TRUE;\r
   }\r
-  \r
+\r
   Len = StrLen (ConfigHdr);\r
   ConfigResp = AllocateZeroPool ((Len + NIC_ITEM_CONFIG_SIZE * 2 + 100) * sizeof (CHAR16));\r
   if (ConfigResp == NULL) {\r
@@ -1921,10 +1918,10 @@ NetLibDefaultAddressIsStatic (
 \r
   String = ConfigResp + Len;\r
   UnicodeSPrint (\r
-    String, \r
-    (8 + 4 + 7 + 4 + 1) * sizeof (CHAR16), \r
-    L"&OFFSET=%04X&WIDTH=%04X", \r
-    OFFSET_OF (NIC_IP4_CONFIG_INFO, Source), \r
+    String,\r
+    (8 + 4 + 7 + 4 + 1) * sizeof (CHAR16),\r
+    L"&OFFSET=%04X&WIDTH=%04X",\r
+    OFFSET_OF (NIC_IP4_CONFIG_INFO, Source),\r
     sizeof (UINT32)\r
     );\r
 \r
@@ -1957,7 +1954,7 @@ NetLibDefaultAddressIsStatic (
   }\r
 \r
   IsStatic = (BOOLEAN) (ConfigInfo->Source == IP4_CONFIG_SOURCE_STATIC);\r
\r
+\r
 ON_EXIT:\r
 \r
   if (AccessResults != NULL) {\r
@@ -1978,7 +1975,7 @@ ON_EXIT:
 \r
 /**\r
   Create an IPv4 device path node.\r
-  \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
   The length of the IPv4 device path node in bytes is 19.\r
@@ -2028,7 +2025,7 @@ NetLibCreateIPv4DPathNode (
 \r
 /**\r
   Create an IPv6 device path node.\r
-  \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
@@ -2070,7 +2067,7 @@ NetLibCreateIPv6DPathNode (
 \r
 /**\r
   Find the UNDI/SNP handle from controller and protocol GUID.\r
-  \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