]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Library/DxeIpIoLib/DxeIpIoLib.c
MdeModulePkg: Clean up source files
[mirror_edk2.git] / MdeModulePkg / Library / DxeIpIoLib / DxeIpIoLib.c
index 9a70e9075d9abe43d2b3691bc36b344aadf838b7..fafc26894dba2184f25d8776468ac700625ebbe5 100644 (file)
@@ -2,7 +2,7 @@
   IpIo Library.\r
 \r
 (C) Copyright 2014 Hewlett-Packard Development Company, L.P.<BR>\r
-Copyright (c) 2005 - 2016, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2005 - 2018, Intel Corporation. All rights reserved.<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
@@ -78,11 +78,11 @@ GLOBAL_REMOVE_IF_UNREFERENCED ICMP_ERROR_INFO  mIcmp6ErrMap[10] = {
   {FALSE, TRUE}, // ICMP6_ERR_UNREACH_NET\r
   {FALSE, TRUE}, // ICMP6_ERR_UNREACH_HOST\r
   {TRUE,  TRUE}, // ICMP6_ERR_UNREACH_PROTOCOL\r
-  {TRUE,  TRUE}, // ICMP6_ERR_UNREACH_PORT    \r
+  {TRUE,  TRUE}, // ICMP6_ERR_UNREACH_PORT\r
   {TRUE,  TRUE}, // ICMP6_ERR_PACKAGE_TOOBIG\r
   {FALSE, TRUE}, // ICMP6_ERR_TIMXCEED_HOPLIMIT\r
   {FALSE, TRUE}, // ICMP6_ERR_TIMXCEED_REASS\r
-  {FALSE, TRUE}, // ICMP6_ERR_PARAMPROB_HEADER    \r
+  {FALSE, TRUE}, // ICMP6_ERR_PARAMPROB_HEADER\r
   {FALSE, TRUE}, // ICMP6_ERR_PARAMPROB_NEXHEADER\r
   {FALSE, TRUE}  // ICMP6_ERR_PARAMPROB_IPV6OPTION\r
 };\r
@@ -129,6 +129,7 @@ IpIoTransmitHandler (
 \r
   @retval       EFI_SUCCESS        The IP child is created and the IP protocol\r
                                    interface is retrieved.\r
+  @retval       EFI_UNSUPPORTED    Upsupported IpVersion.\r
   @retval       Others             The required operation failed.\r
 \r
 **/\r
@@ -206,6 +207,7 @@ IpIoCreateIpChildOpenProtocol (
 \r
   @retval     EFI_SUCCESS         The IP protocol is closed and the relevant IP child\r
                                   is destroyed.\r
+  @retval     EFI_UNSUPPORTED     Upsupported IpVersion.\r
   @retval     Others              The required operation failed.\r
 \r
 **/\r
@@ -234,28 +236,29 @@ IpIoCloseProtocolDestroyIpChild (
   //\r
   // Close the previously openned IP protocol.\r
   //\r
-  gBS->CloseProtocol (\r
-         ChildHandle,\r
-         IpProtocolGuid,\r
-         ImageHandle,\r
-         ControllerHandle\r
-         );\r
+  Status = gBS->CloseProtocol (\r
+                  ChildHandle,\r
+                  IpProtocolGuid,\r
+                  ImageHandle,\r
+                  ControllerHandle\r
+                  );\r
+  if (EFI_ERROR (Status)) {\r
+    return Status;\r
+  }\r
 \r
   //\r
   // Destroy the IP child.\r
   //\r
-  Status = NetLibDestroyServiceChild (\r
-             ControllerHandle,\r
-             ImageHandle,\r
-             ServiceBindingGuid,\r
-             ChildHandle\r
-             );\r
-\r
-  return Status;\r
+  return NetLibDestroyServiceChild (\r
+           ControllerHandle,\r
+           ImageHandle,\r
+           ServiceBindingGuid,\r
+           ChildHandle\r
+           );\r
 }\r
 \r
 /**\r
-  This function handles ICMPv4 packets. It is the worker function of \r
+  This function handles ICMPv4 packets. It is the worker function of\r
   IpIoIcmpHandler.\r
 \r
   @param[in]       IpIo            Pointer to the IP_IO instance.\r
@@ -281,14 +284,21 @@ IpIoIcmpv4Handler (
   UINT8                Code;\r
   UINT32               TrimBytes;\r
 \r
+  ASSERT (IpIo != NULL);\r
+  ASSERT (Pkt != NULL);\r
+  ASSERT (Session != NULL);\r
   ASSERT (IpIo->IpVersion == IP_VERSION_4);\r
 \r
-  IcmpHdr = NET_PROTO_HDR (Pkt, IP4_ICMP_ERROR_HEAD);\r
-  IpHdr   = (EFI_IP4_HEADER *) (&IcmpHdr->IpHead);\r
-\r
   //\r
   // Check the ICMP packet length.\r
   //\r
+  if (Pkt->TotalSize < sizeof (IP4_ICMP_ERROR_HEAD)) {\r
+    return EFI_ABORTED;\r
+  }\r
+\r
+  IcmpHdr = NET_PROTO_HDR (Pkt, IP4_ICMP_ERROR_HEAD);\r
+  IpHdr   = (EFI_IP4_HEADER *) (&IcmpHdr->IpHead);\r
+\r
   if (Pkt->TotalSize < ICMP_ERRLEN (IpHdr)) {\r
 \r
     return EFI_ABORTED;\r
@@ -378,13 +388,19 @@ IpIoIcmpv4Handler (
 \r
   NetbufTrim (Pkt, TrimBytes, TRUE);\r
 \r
-  IpIo->PktRcvdNotify (EFI_ICMP_ERROR, IcmpErr, Session, Pkt, IpIo->RcvdContext);\r
+  //\r
+  // If the input packet has invalid format, and TrimBytes is larger than\r
+  // the packet size, the NetbufTrim might trim the packet to zero.\r
+  //\r
+  if (Pkt->TotalSize != 0) {\r
+    IpIo->PktRcvdNotify (EFI_ICMP_ERROR, IcmpErr, Session, Pkt, IpIo->RcvdContext);\r
+  }\r
 \r
-  return EFI_SUCCESS;  \r
+  return EFI_SUCCESS;\r
 }\r
 \r
 /**\r
-  This function handles ICMPv6 packets. It is the worker function of \r
+  This function handles ICMPv6 packets. It is the worker function of\r
   IpIoIcmpHandler.\r
 \r
   @param[in]       IpIo            Pointer to the IP_IO instance.\r
@@ -412,6 +428,9 @@ IpIoIcmpv6Handler (
   UINT32               TrimBytes;\r
   BOOLEAN              Flag;\r
 \r
+  ASSERT (IpIo != NULL);\r
+  ASSERT (Pkt != NULL);\r
+  ASSERT (Session != NULL);\r
   ASSERT (IpIo->IpVersion == IP_VERSION_6);\r
 \r
   //\r
@@ -428,7 +447,7 @@ IpIoIcmpv6Handler (
 \r
   //\r
   // Analyze the ICMPv6 Error in this ICMPv6 packet\r
-  // \r
+  //\r
   switch (Type) {\r
   case ICMP_V6_DEST_UNREACHABLE:\r
     switch (Code) {\r
@@ -487,7 +506,7 @@ IpIoIcmpv6Handler (
    default:\r
 \r
      return EFI_ABORTED;\r
-   } \r
+   }\r
 \r
   //\r
   // Notify user the ICMPv6 packet only containing payload except\r
@@ -498,7 +517,7 @@ IpIoIcmpv6Handler (
   NextHeader = IpHdr->NextHeader;\r
   PayLoadHdr = (UINT8 *) ((UINT8 *) IcmpHdr + sizeof (IP6_ICMP_ERROR_HEAD));\r
   Flag       = TRUE;\r
-  \r
+\r
   do {\r
     switch (NextHeader) {\r
     case EFI_IP_PROTO_UDP:\r
@@ -536,10 +555,16 @@ IpIoIcmpv6Handler (
   } while (Flag);\r
 \r
   TrimBytes = (UINT32) (PayLoadHdr - (UINT8 *) IcmpHdr);\r
-  \r
+\r
   NetbufTrim (Pkt, TrimBytes, TRUE);\r
 \r
-  IpIo->PktRcvdNotify (EFI_ICMP_ERROR, IcmpErr, Session, Pkt, IpIo->RcvdContext);\r
+  //\r
+  // If the input packet has invalid format, and TrimBytes is larger than\r
+  // the packet size, the NetbufTrim might trim the packet to zero.\r
+  //\r
+  if (Pkt->TotalSize != 0) {\r
+    IpIo->PktRcvdNotify (EFI_ICMP_ERROR, IcmpErr, Session, Pkt, IpIo->RcvdContext);\r
+  }\r
 \r
   return EFI_SUCCESS;\r
 }\r
@@ -609,8 +634,8 @@ IpIoExtFree (
   @param[in]       Dest                 Pointer to the destination IP address.\r
   @param[in]       Override             Pointer to the overriden IP_IO data.\r
 \r
-  @return Pointer to the data structure created to wrap the packet. If NULL,\r
-  @return resource limit occurred.\r
+  @return Pointer to the data structure created to wrap the packet. If any error occurs,\r
+          then return NULL.\r
 \r
 **/\r
 IP_IO_SEND_ENTRY *\r
@@ -625,7 +650,7 @@ IpIoCreateSndEntry (
   )\r
 {\r
   IP_IO_SEND_ENTRY          *SndEntry;\r
-  EFI_EVENT                 Event;  \r
+  EFI_EVENT                 Event;\r
   EFI_STATUS                Status;\r
   NET_FRAGMENT              *ExtFragment;\r
   UINT32                    FragmentCount;\r
@@ -675,7 +700,7 @@ IpIoCreateSndEntry (
   }\r
 \r
   //\r
-  // Build a fragment table to contain the fragments in the packet. \r
+  // Build a fragment table to contain the fragments in the packet.\r
   //\r
   if (IpIo->IpVersion == IP_VERSION_4) {\r
     ExtFragment = (NET_FRAGMENT *) TxData->Ip4TxData.FragmentTable;\r
@@ -732,7 +757,7 @@ IpIoCreateSndEntry (
     Ip6TxData->FragmentCount = FragmentCount;\r
     Ip6TxData->ExtHdrsLength = 0;\r
     Ip6TxData->ExtHdrs       = NULL;\r
-    \r
+\r
     //\r
     // Set the fields of SndToken\r
     //\r
@@ -779,7 +804,7 @@ ON_ERROR:
 \r
 /**\r
   Destroy the SndEntry.\r
-  \r
+\r
   This function pairs with IpIoCreateSndEntry().\r
 \r
   @param[in]  SndEntry              Pointer to the send entry to be destroyed.\r
@@ -1014,12 +1039,22 @@ IpIoListenHandlerDpc (
     return;\r
   }\r
 \r
-  if (((EFI_SUCCESS != Status) && (EFI_ICMP_ERROR != Status)) || (NULL == RxData)) {\r
+  if ((EFI_SUCCESS != Status) && (EFI_ICMP_ERROR != Status)) {\r
     //\r
-    // @bug Only process the normal packets and the icmp error packets, if RxData is NULL\r
-    // @bug with Status == EFI_SUCCESS or EFI_ICMP_ERROR, just resume the receive although\r
-    // @bug this should be a bug of the low layer (IP).\r
+    // Only process the normal packets and the icmp error packets.\r
     //\r
+    if (RxData != NULL) {\r
+      goto CleanUp;\r
+    } else {\r
+      goto Resume;\r
+    }\r
+  }\r
+\r
+  //\r
+  // if RxData is NULL with Status == EFI_SUCCESS or EFI_ICMP_ERROR, this should be a code issue in the low layer (IP).\r
+  //\r
+  ASSERT (RxData != NULL);\r
+  if (RxData == NULL) {\r
     goto Resume;\r
   }\r
 \r
@@ -1028,12 +1063,19 @@ IpIoListenHandlerDpc (
   }\r
 \r
   if (IpIo->IpVersion == IP_VERSION_4) {\r
+    ASSERT (RxData->Ip4RxData.Header != NULL);\r
+    if (IP4_IS_LOCAL_BROADCAST (EFI_IP4 (RxData->Ip4RxData.Header->SourceAddress))) {\r
+      //\r
+      // The source address is a broadcast address, discard it.\r
+      //\r
+      goto CleanUp;\r
+    }\r
     if ((EFI_IP4 (RxData->Ip4RxData.Header->SourceAddress) != 0) &&\r
         (IpIo->SubnetMask != 0) &&\r
         IP4_NET_EQUAL (IpIo->StationIp, EFI_NTOHL (((EFI_IP4_RECEIVE_DATA *) RxData)->Header->SourceAddress), IpIo->SubnetMask) &&\r
         !NetIp4IsUnicast (EFI_NTOHL (((EFI_IP4_RECEIVE_DATA *) RxData)->Header->SourceAddress), IpIo->SubnetMask)) {\r
       //\r
-      // The source address is not zero and it's not a unicast IP address, discard it.\r
+      // The source address doesn't match StationIp and it's not a unicast IP address, discard it.\r
       //\r
       goto CleanUp;\r
     }\r
@@ -1045,6 +1087,11 @@ IpIoListenHandlerDpc (
       goto CleanUp;\r
     }\r
 \r
+    //\r
+    // The fragment should always be valid for non-zero length packet.\r
+    //\r
+    ASSERT (RxData->Ip4RxData.FragmentCount != 0);\r
+\r
     //\r
     // Create a netbuffer representing IPv4 packet\r
     //\r
@@ -1069,18 +1116,23 @@ IpIoListenHandlerDpc (
     Session.IpHdrLen       = RxData->Ip4RxData.HeaderLength;\r
     Session.IpVersion      = IP_VERSION_4;\r
   } else {\r
-\r
+    ASSERT (RxData->Ip6RxData.Header != NULL);\r
     if (!NetIp6IsValidUnicast(&RxData->Ip6RxData.Header->SourceAddress)) {\r
       goto CleanUp;\r
     }\r
-    \r
+\r
     if (RxData->Ip6RxData.DataLength == 0) {\r
       //\r
       // Discard zero length data payload packet.\r
       //\r
       goto CleanUp;\r
     }\r
-    \r
+\r
+    //\r
+    // The fragment should always be valid for non-zero length packet.\r
+    //\r
+    ASSERT (RxData->Ip6RxData.FragmentCount != 0);\r
+\r
     //\r
     // Create a netbuffer representing IPv6 packet\r
     //\r
@@ -1100,19 +1152,19 @@ IpIoListenHandlerDpc (
     // Create a net session\r
     //\r
     CopyMem (\r
-      &Session.Source, \r
+      &Session.Source,\r
       &RxData->Ip6RxData.Header->SourceAddress,\r
       sizeof(EFI_IPv6_ADDRESS)\r
       );\r
     CopyMem (\r
-      &Session.Dest, \r
-      &RxData->Ip6RxData.Header->DestinationAddress, \r
+      &Session.Dest,\r
+      &RxData->Ip6RxData.Header->DestinationAddress,\r
       sizeof(EFI_IPv6_ADDRESS)\r
       );\r
     Session.IpHdr.Ip6Hdr = RxData->Ip6RxData.Header;\r
     Session.IpHdrLen     = RxData->Ip6RxData.HeaderLength;\r
     Session.IpVersion    = IP_VERSION_6;\r
-  } \r
+  }\r
 \r
   if (EFI_SUCCESS == Status) {\r
 \r
@@ -1134,7 +1186,7 @@ CleanUp:
   if (IpIo->IpVersion == IP_VERSION_4){\r
     gBS->SignalEvent (RxData->Ip4RxData.RecycleSignal);\r
   } else {\r
-    gBS->SignalEvent (RxData->Ip6RxData.RecycleSignal); \r
+    gBS->SignalEvent (RxData->Ip6RxData.RecycleSignal);\r
   }\r
 \r
 Resume:\r
@@ -1169,7 +1221,9 @@ IpIoListenHandler (
 \r
 /**\r
   Create a new IP_IO instance.\r
-  \r
+\r
+  If IpVersion is not IP_VERSION_4 or IP_VERSION_6, then ASSERT().\r
+\r
   This function uses IP4/IP6 service binding protocol in Controller to create\r
   an IP4/IP6 child (aka IP4/IP6 instance).\r
 \r
@@ -1178,7 +1232,7 @@ IpIoListenHandler (
   @param[in]  Controller        The controller handle that has IP4 or IP6 service\r
                                 binding protocol installed.\r
   @param[in]  IpVersion         The version of the IP protocol to use, either\r
-                                IPv4 or IPv6.                            \r
+                                IPv4 or IPv6.\r
 \r
   @return Pointer to a newly created IP_IO instance, or NULL if failed.\r
 \r
@@ -1233,8 +1287,8 @@ IpIoCreate (
              Controller,\r
              Image,\r
              &IpIo->ChildHandle,\r
-             IpVersion,             \r
-             (VOID **)&(IpIo->Ip)\r
+             IpVersion,\r
+             (VOID **) & (IpIo->Ip)\r
              );\r
   if (EFI_ERROR (Status)) {\r
     goto ReleaseIpIo;\r
@@ -1256,7 +1310,9 @@ ReleaseIpIo:
 \r
 /**\r
   Open an IP_IO instance for use.\r
-  \r
+\r
+  If Ip version is not IP_VERSION_4 or IP_VERSION_6, then ASSERT().\r
+\r
   This function is called after IpIoCreate(). It is used for configuring the IP\r
   instance and register the callbacks and their context data for sending and\r
   receiving IP packets.\r
@@ -1266,11 +1322,13 @@ ReleaseIpIo:
   @param[in]       OpenData           The configuration data and callbacks for\r
                                       the IP_IO instance.\r
 \r
-  @retval          EFI_SUCCESS        The IP_IO instance opened with OpenData\r
-                                      successfully.\r
-  @retval          EFI_ACCESS_DENIED  The IP_IO instance is configured, avoid to \r
-                                      reopen it.\r
-  @retval          Others             Error condition occurred.\r
+  @retval          EFI_SUCCESS            The IP_IO instance opened with OpenData\r
+                                          successfully.\r
+  @retval          EFI_ACCESS_DENIED      The IP_IO instance is configured, avoid to\r
+                                          reopen it.\r
+  @retval          EFI_UNSUPPORTED        IPv4 RawData mode is no supported.\r
+  @retval          EFI_INVALID_PARAMETER  Invalid input parameter.\r
+  @retval          Others                 Error condition occurred.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -1283,6 +1341,10 @@ IpIoOpen (
   EFI_STATUS        Status;\r
   UINT8             IpVersion;\r
 \r
+  if (IpIo == NULL || OpenData == NULL) {\r
+    return EFI_INVALID_PARAMETER;\r
+  }\r
+\r
   if (IpIo->IsConfigured) {\r
     return EFI_ACCESS_DENIED;\r
   }\r
@@ -1307,7 +1369,7 @@ IpIoOpen (
       IpIo->StationIp = EFI_NTOHL (OpenData->IpConfigData.Ip4CfgData.StationAddress);\r
       IpIo->SubnetMask = EFI_NTOHL (OpenData->IpConfigData.Ip4CfgData.SubnetMask);\r
     }\r
-    \r
+\r
     Status = IpIo->Ip.Ip4->Configure (\r
                              IpIo->Ip.Ip4,\r
                              &OpenData->IpConfigData.Ip4CfgData\r
@@ -1315,7 +1377,7 @@ IpIoOpen (
   } else {\r
 \r
     Status = IpIo->Ip.Ip6->Configure (\r
-                             IpIo->Ip.Ip6, \r
+                             IpIo->Ip.Ip6,\r
                              &OpenData->IpConfigData.Ip6CfgData\r
                              );\r
   }\r
@@ -1361,7 +1423,7 @@ IpIoOpen (
                              );\r
     if (EFI_ERROR (Status)) {\r
       IpIo->Ip.Ip4->Configure (IpIo->Ip.Ip4, NULL);\r
-      goto ErrorExit;\r
+      return Status;\r
     }\r
 \r
   } else {\r
@@ -1373,29 +1435,30 @@ IpIoOpen (
                              );\r
     if (EFI_ERROR (Status)) {\r
       IpIo->Ip.Ip6->Configure (IpIo->Ip.Ip6, NULL);\r
-      goto ErrorExit;\r
+      return Status;\r
     }\r
   }\r
 \r
   IpIo->IsConfigured = TRUE;\r
   InsertTailList (&mActiveIpIoList, &IpIo->Entry);\r
 \r
-ErrorExit:\r
-\r
   return Status;\r
 }\r
 \r
 \r
 /**\r
   Stop an IP_IO instance.\r
-  \r
+\r
+  If Ip version is not IP_VERSION_4 or IP_VERSION_6, then ASSERT().\r
+\r
   This function is paired with IpIoOpen(). The IP_IO will be unconfigured and all\r
   the pending send/receive tokens will be canceled.\r
 \r
   @param[in, out]  IpIo            Pointer to the IP_IO instance that needs to stop.\r
 \r
-  @retval          EFI_SUCCESS     The IP_IO instance stopped successfully.\r
-  @retval          Others          Error condition occurred.\r
+  @retval          EFI_SUCCESS            The IP_IO instance stopped successfully.\r
+  @retval          EFI_INVALID_PARAMETER  Invalid input parameter.\r
+  @retval          Others                 Error condition occurred.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -1408,6 +1471,10 @@ IpIoStop (
   IP_IO_IP_INFO     *IpInfo;\r
   UINT8             IpVersion;\r
 \r
+  if (IpIo == NULL) {\r
+    return EFI_INVALID_PARAMETER;\r
+  }\r
+\r
   if (!IpIo->IsConfigured) {\r
     return EFI_SUCCESS;\r
   }\r
@@ -1465,7 +1532,7 @@ IpIoStop (
 \r
 /**\r
   Destroy an IP_IO instance.\r
-  \r
+\r
   This function is paired with IpIoCreate(). The IP_IO will be closed first.\r
   Resource will be freed afterwards. See IpIoCloseProtocolDestroyIpChild().\r
 \r
@@ -1482,20 +1549,28 @@ IpIoDestroy (
   IN OUT IP_IO *IpIo\r
   )\r
 {\r
+  EFI_STATUS    Status;\r
+\r
   //\r
   // Stop the IpIo.\r
   //\r
-  IpIoStop (IpIo);\r
+  Status = IpIoStop (IpIo);\r
+  if (EFI_ERROR (Status)) {\r
+    return Status;\r
+  }\r
 \r
   //\r
   // Close the IP protocol and destroy the child.\r
   //\r
-  IpIoCloseProtocolDestroyIpChild (\r
-    IpIo->Controller,\r
-    IpIo->Image,\r
-    IpIo->ChildHandle,\r
-    IpIo->IpVersion\r
-    );\r
+  Status = IpIoCloseProtocolDestroyIpChild (\r
+             IpIo->Controller,\r
+             IpIo->Image,\r
+             IpIo->ChildHandle,\r
+             IpIo->IpVersion\r
+             );\r
+  if (EFI_ERROR (Status)) {\r
+    return Status;\r
+  }\r
 \r
   gBS->FreePool (IpIo);\r
 \r
@@ -1505,8 +1580,8 @@ IpIoDestroy (
 \r
 /**\r
   Send out an IP packet.\r
-  \r
-  This function is called after IpIoOpen(). The data to be sent are wrapped in\r
+\r
+  This function is called after IpIoOpen(). The data to be sent is wrapped in\r
   Pkt. The IP instance wrapped in IpIo is used for sending by default but can be\r
   overriden by Sender. Other sending configs, like source address and gateway\r
   address etc., are specified in OverrideData.\r
@@ -1518,12 +1593,15 @@ IpIoDestroy (
   @param[in]       Context               Optional context data.\r
   @param[in]       NotifyData            Optional notify data.\r
   @param[in]       Dest                  The destination IP address to send this packet to.\r
+                                         This parameter is optional when using IPv6.\r
   @param[in]       OverrideData          The data to override some configuration of the IP\r
                                          instance used for sending.\r
 \r
   @retval          EFI_SUCCESS           The operation is completed successfully.\r
+  @retval          EFI_INVALID_PARAMETER The input parameter is not correct.\r
   @retval          EFI_NOT_STARTED       The IpIo is not configured.\r
   @retval          EFI_OUT_OF_RESOURCES  Failed due to resource limit.\r
+  @retval          Others                Error condition occurred.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -1534,7 +1612,7 @@ IpIoSend (
   IN     IP_IO_IP_INFO  *Sender        OPTIONAL,\r
   IN     VOID           *Context       OPTIONAL,\r
   IN     VOID           *NotifyData    OPTIONAL,\r
-  IN     EFI_IP_ADDRESS *Dest,\r
+  IN     EFI_IP_ADDRESS *Dest          OPTIONAL,\r
   IN     IP_IO_OVERRIDE *OverrideData  OPTIONAL\r
   )\r
 {\r
@@ -1542,7 +1620,13 @@ IpIoSend (
   IP_IO_IP_PROTOCOL Ip;\r
   IP_IO_SEND_ENTRY  *SndEntry;\r
 \r
-  ASSERT ((IpIo->IpVersion != IP_VERSION_4) || (Dest != NULL));\r
+  if ((IpIo == NULL) || (Pkt == NULL)) {\r
+    return EFI_INVALID_PARAMETER;\r
+  }\r
+\r
+  if ((IpIo->IpVersion == IP_VERSION_4) && (Dest == NULL)) {\r
+    return EFI_INVALID_PARAMETER;\r
+  }\r
 \r
   if (!IpIo->IsConfigured) {\r
     return EFI_NOT_STARTED;\r
@@ -1584,6 +1668,9 @@ IpIoSend (
 /**\r
   Cancel the IP transmit token which wraps this Packet.\r
 \r
+  If IpIo is NULL, then ASSERT().\r
+  If Packet is NULL, then ASSERT().\r
+\r
   @param[in]  IpIo                  Pointer to the IP_IO instance.\r
   @param[in]  Packet                Pointer to the packet of NET_BUF to cancel.\r
 \r
@@ -1630,7 +1717,10 @@ IpIoCancelTxToken (
 \r
 /**\r
   Add a new IP instance for sending data.\r
-  \r
+\r
+  If IpIo is NULL, then ASSERT().\r
+  If Ip version is not IP_VERSION_4 or IP_VERSION_6, then ASSERT().\r
+\r
   The function is used to add the IP_IO to the IP_IO sending list. The caller\r
   can later use IpIoFindSender() to get the IP_IO and call IpIoSend() to send\r
   data.\r
@@ -1652,6 +1742,7 @@ IpIoAddIp (
   EFI_EVENT      Event;\r
 \r
   ASSERT (IpIo != NULL);\r
+  ASSERT ((IpIo->IpVersion == IP_VERSION_4) || (IpIo->IpVersion == IP_VERSION_6));\r
 \r
   IpInfo = AllocatePool (sizeof (IP_IO_IP_INFO));\r
   if (IpInfo == NULL) {\r
@@ -1732,6 +1823,9 @@ ReleaseIpInfo:
   Configure the IP instance of this IpInfo and start the receiving if IpConfigData\r
   is not NULL.\r
 \r
+  If IpInfo is NULL, then ASSERT().\r
+  If Ip version is not IP_VERSION_4 or IP_VERSION_6, then ASSERT().\r
+\r
   @param[in, out]  IpInfo          Pointer to the IP_IO_IP_INFO instance.\r
   @param[in, out]  IpConfigData    The IP configure data used to configure the IP\r
                                    instance, if NULL the IP instance is reset. If\r
@@ -1781,31 +1875,35 @@ IpIoConfigIp (
   }\r
 \r
   if (EFI_ERROR (Status)) {\r
-    goto OnExit;\r
+    return Status;\r
   }\r
 \r
   if (IpConfigData != NULL) {\r
-    if (IpInfo->IpVersion == IP_VERSION_4){\r
+    if (IpInfo->IpVersion == IP_VERSION_4) {\r
 \r
       if (((EFI_IP4_CONFIG_DATA *) IpConfigData)->UseDefaultAddress) {\r
-        Ip.Ip4->GetModeData (\r
-                  Ip.Ip4, \r
-                  &Ip4ModeData, \r
-                  NULL, \r
-                  NULL\r
-                  );\r
+        Status = Ip.Ip4->GetModeData (\r
+                           Ip.Ip4,\r
+                           &Ip4ModeData,\r
+                           NULL,\r
+                           NULL\r
+                           );\r
+        if (EFI_ERROR (Status)) {\r
+          Ip.Ip4->Configure (Ip.Ip4, NULL);\r
+          return Status;\r
+        }\r
 \r
         IP4_COPY_ADDRESS (&((EFI_IP4_CONFIG_DATA*) IpConfigData)->StationAddress, &Ip4ModeData.ConfigData.StationAddress);\r
         IP4_COPY_ADDRESS (&((EFI_IP4_CONFIG_DATA*) IpConfigData)->SubnetMask, &Ip4ModeData.ConfigData.SubnetMask);\r
-    }\r
+      }\r
 \r
       CopyMem (\r
-        &IpInfo->Addr.Addr, \r
-        &((EFI_IP4_CONFIG_DATA *) IpConfigData)->StationAddress, \r
+        &IpInfo->Addr.Addr,\r
+        &((EFI_IP4_CONFIG_DATA *) IpConfigData)->StationAddress,\r
         sizeof (IP4_ADDR)\r
         );\r
       CopyMem (\r
-        &IpInfo->PreMask.SubnetMask, \r
+        &IpInfo->PreMask.SubnetMask,\r
         &((EFI_IP4_CONFIG_DATA *) IpConfigData)->SubnetMask,\r
         sizeof (IP4_ADDR)\r
         );\r
@@ -1814,16 +1912,20 @@ IpIoConfigIp (
                          Ip.Ip4,\r
                          &IpInfo->DummyRcvToken.Ip4Token\r
                          );\r
-    if (EFI_ERROR (Status)) {\r
-      Ip.Ip4->Configure (Ip.Ip4, NULL);\r
-    }\r
-  } else {\r
-    Ip.Ip6->GetModeData (\r
-              Ip.Ip6,\r
-              &Ip6ModeData,\r
-              NULL,\r
-              NULL\r
-              );\r
+      if (EFI_ERROR (Status)) {\r
+        Ip.Ip4->Configure (Ip.Ip4, NULL);\r
+      }\r
+    } else {\r
+      Status = Ip.Ip6->GetModeData (\r
+                         Ip.Ip6,\r
+                         &Ip6ModeData,\r
+                         NULL,\r
+                         NULL\r
+                         );\r
+      if (EFI_ERROR (Status)) {\r
+        Ip.Ip6->Configure (Ip.Ip6, NULL);\r
+        return Status;\r
+      }\r
 \r
       if (Ip6ModeData.IsConfigured) {\r
         CopyMem (\r
@@ -1858,12 +1960,12 @@ IpIoConfigIp (
 \r
       } else {\r
         Status = EFI_NO_MAPPING;\r
-        goto OnExit;\r
-      } \r
+        return Status;\r
+      }\r
 \r
       CopyMem (\r
-        &IpInfo->Addr, \r
-        &Ip6ModeData.ConfigData.StationAddress, \r
+        &IpInfo->Addr,\r
+        &Ip6ModeData.ConfigData.StationAddress,\r
         sizeof (EFI_IPv6_ADDRESS)\r
         );\r
 \r
@@ -1874,7 +1976,7 @@ IpIoConfigIp (
       if (EFI_ERROR (Status)) {\r
         Ip.Ip6->Configure (Ip.Ip6, NULL);\r
       }\r
-    }  \r
+    }\r
   } else {\r
     //\r
     // The IP instance is reset, set the stored Addr and SubnetMask to zero.\r
@@ -1883,8 +1985,6 @@ IpIoConfigIp (
     ZeroMem (&IpInfo->PreMask, sizeof (IpInfo->PreMask));\r
   }\r
 \r
-OnExit:\r
-\r
   return Status;\r
 }\r
 \r
@@ -1892,7 +1992,9 @@ OnExit:
 /**\r
   Destroy an IP instance maintained in IpIo->IpList for\r
   sending purpose.\r
-  \r
+\r
+  If Ip version is not IP_VERSION_4 or IP_VERSION_6, then ASSERT().\r
+\r
   This function pairs with IpIoAddIp(). The IpInfo is previously created by\r
   IpIoAddIp(). The IP_IO_IP_INFO::RefCnt is decremented and the IP instance\r
   will be dstroyed if the RefCnt is zero.\r
@@ -1911,6 +2013,10 @@ IpIoRemoveIp (
 \r
   UINT8               IpVersion;\r
 \r
+  if (IpIo == NULL || IpInfo == NULL) {\r
+    return;\r
+  }\r
+\r
   ASSERT (IpInfo->RefCnt > 0);\r
 \r
   NET_PUT_REF (IpInfo);\r
@@ -1964,7 +2070,7 @@ IpIoRemoveIp (
 /**\r
   Find the first IP protocol maintained in IpIo whose local\r
   address is the same as Src.\r
-  \r
+\r
   This function is called when the caller needs the IpIo to send data to the\r
   specified Src. The IpIo was added previously by IpIoAddIp().\r
 \r
@@ -1974,7 +2080,7 @@ IpIoRemoveIp (
   @param[in]       Src               The local IP address.\r
 \r
   @return Pointer to the IP protocol can be used for sending purpose and its local\r
-          address is the same with Src.\r
+          address is the same with Src. NULL if failed.\r
 \r
 **/\r
 IP_IO_IP_INFO *\r
@@ -1990,7 +2096,13 @@ IpIoFindSender (
   LIST_ENTRY      *IpInfoEntry;\r
   IP_IO_IP_INFO   *IpInfo;\r
 \r
-  ASSERT ((IpVersion == IP_VERSION_4) || (IpVersion == IP_VERSION_6));  \r
+  if (IpIo == NULL || Src == NULL) {\r
+    return NULL;\r
+  }\r
+\r
+  if ((IpVersion != IP_VERSION_4) && (IpVersion != IP_VERSION_6)) {\r
+    return NULL;\r
+  }\r
 \r
   NET_LIST_FOR_EACH (IpIoEntry, &mActiveIpIoList) {\r
     IpIoPtr = NET_LIST_USER_STRUCT (IpIoEntry, IP_IO, Entry);\r
@@ -2012,10 +2124,9 @@ IpIoFindSender (
 \r
         if (EFI_IP6_EQUAL (&IpInfo->Addr.v6, &Src->v6)) {\r
           *IpIo = IpIoPtr;\r
-          return IpInfo;       \r
+          return IpInfo;\r
         }\r
-      }      \r
-\r
+      }\r
     }\r
   }\r
 \r
@@ -2028,16 +2139,17 @@ IpIoFindSender (
 \r
 /**\r
   Get the ICMP error map information.\r
-  \r
+\r
   The ErrorStatus will be returned. The IsHard and Notify are optional. If they\r
   are not NULL, this routine will fill them.\r
 \r
   @param[in]   IcmpError             IcmpError Type.\r
   @param[in]   IpVersion             The version of the IP protocol to use,\r
-                                     either IPv4 or IPv6. \r
+                                     either IPv4 or IPv6.\r
   @param[out]  IsHard                If TRUE, indicates that it is a hard error.\r
   @param[out]  Notify                If TRUE, SockError needs to be notified.\r
 \r
+  @retval EFI_UNSUPPORTED            Unrecognizable ICMP error code.\r
   @return ICMP Error Status, such as EFI_NETWORK_UNREACHABLE.\r
 \r
 **/\r
@@ -2105,12 +2217,12 @@ IpIoGetIcmpErrStatus (
 \r
     case ICMP6_ERR_UNREACH_HOST:\r
     case ICMP6_ERR_TIMXCEED_HOPLIMIT:\r
-    case ICMP6_ERR_TIMXCEED_REASS:       \r
+    case ICMP6_ERR_TIMXCEED_REASS:\r
       return EFI_HOST_UNREACHABLE;\r
 \r
     case ICMP6_ERR_UNREACH_PROTOCOL:\r
       return EFI_PROTOCOL_UNREACHABLE;\r
-      \r
+\r
     case ICMP6_ERR_UNREACH_PORT:\r
       return EFI_PORT_UNREACHABLE;\r
 \r
@@ -2139,39 +2251,45 @@ IpIoGetIcmpErrStatus (
   Refresh the remote peer's Neighbor Cache entries.\r
 \r
   This function is called when the caller needs the IpIo to refresh the existing\r
-  IPv6 neighbor cache entries since the neighbor is considered reachable by the \r
-  node has recently received a confirmation that packets sent recently to the \r
-  neighbor were received by its IP layer. \r
+  IPv6 neighbor cache entries since the neighbor is considered reachable by the\r
+  node has recently received a confirmation that packets sent recently to the\r
+  neighbor were received by its IP layer.\r
 \r
   @param[in]   IpIo                  Pointer to an IP_IO instance\r
   @param[in]   Neighbor              The IP address of the neighbor\r
   @param[in]   Timeout               Time in 100-ns units that this entry will\r
-                                     remain in the neighbor cache. A value of \r
-                                     zero means that the entry is permanent. \r
-                                     A value of non-zero means that the entry is \r
+                                     remain in the neighbor cache. A value of\r
+                                     zero means that the entry is permanent.\r
+                                     A value of non-zero means that the entry is\r
                                      dynamic and will be deleted after Timeout.\r
 \r
   @retval      EFI_SUCCESS           The operation is completed successfully.\r
   @retval      EFI_NOT_STARTED       The IpIo is not configured.\r
   @retval      EFI_INVALID_PARAMETER Neighbor Address is invalid.\r
-  @retval      EFI_NOT_FOUND         The neighbor cache entry is not in the \r
-                                     neighbor table.  \r
+  @retval      EFI_NOT_FOUND         The neighbor cache entry is not in the\r
+                                     neighbor table.\r
+  @retval      EFI_UNSUPPORTED       IP version is IPv4, which doesn't support neighbor cache refresh.\r
   @retval      EFI_OUT_OF_RESOURCES  Failed due to resource limit.\r
 \r
 **/\r
 EFI_STATUS\r
+EFIAPI\r
 IpIoRefreshNeighbor (\r
   IN IP_IO           *IpIo,\r
   IN EFI_IP_ADDRESS  *Neighbor,\r
-  IN UINT32          Timeout  \r
+  IN UINT32          Timeout\r
   )\r
 {\r
   EFI_IP6_PROTOCOL  *Ip;\r
 \r
-  if (!IpIo->IsConfigured || IpIo->IpVersion != IP_VERSION_6) {\r
+  if (!IpIo->IsConfigured) {\r
     return EFI_NOT_STARTED;\r
   }\r
 \r
+  if (IpIo->IpVersion != IP_VERSION_6) {\r
+    return EFI_UNSUPPORTED;\r
+  }\r
+\r
   Ip = IpIo->Ip.Ip6;\r
 \r
   return Ip->Neighbors (Ip, FALSE, &Neighbor->v6, NULL, Timeout, TRUE);\r