]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Universal/Network/Ip4ConfigDxe/Ip4ConfigNv.c
Clean up the private GUID definition in module Level.
[mirror_edk2.git] / MdeModulePkg / Universal / Network / Ip4ConfigDxe / Ip4ConfigNv.c
index 88b3da6ed5926a8b7fd165a94848ce7edaa74883..59e5127b44a7d683b21d462d04a5cffe71f54c45 100644 (file)
@@ -1,8 +1,8 @@
 /** @file\r
   Helper functions for configuring or getting the parameters relating to Ip4.\r
 \r
-Copyright (c) 2009, Intel Corporation.<BR>\r
-All rights reserved. This program and the accompanying materials\r
+Copyright (c) 2009 - 2011, 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
 http://opensource.org/licenses/bsd-license.php\r
@@ -13,9 +13,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 **/\r
 \r
 #include "Ip4ConfigNv.h"\r
-\r
-EFI_GUID  mNicIp4ConfigNvDataGuid = EFI_NIC_IP4_CONFIG_NVDATA_GUID;\r
-\r
+#include "NicIp4Variable.h"\r
 \r
 /**\r
   Calculate the prefix length of the IPv4 subnet mask.\r
@@ -36,7 +34,7 @@ GetSubnetMaskPrefixLength (
   //\r
   // The SubnetMask is in network byte order.\r
   //\r
-  ReverseMask = (SubnetMask->Addr[0] << 24) | (SubnetMask->Addr[1] << 16) | (SubnetMask->Addr[2] << 8) | (SubnetMask->Addr[3]);\r
+  ReverseMask = SwapBytes32 (*(UINT32 *)&SubnetMask[0]);\r
 \r
   //\r
   // Reverse it.\r
@@ -61,14 +59,14 @@ GetSubnetMaskPrefixLength (
   Convert the decimal dotted IPv4 address into the binary IPv4 address.\r
 \r
   @param[in]   Str             The UNICODE string.\r
-  @param[out]  Ip              The storage to return the ASCII string.\r
+  @param[out]  Ip              The storage to return the IPv4 address.\r
 \r
   @retval EFI_SUCCESS           The binary IP address is returned in Ip.\r
   @retval EFI_INVALID_PARAMETER The IP string is malformatted.\r
 **/\r
 EFI_STATUS\r
-Ip4AsciiStrToIp (\r
-  IN  CHAR            *Str,\r
+Ip4StrToIp (\r
+  IN  CHAR16            *Str,\r
   OUT EFI_IPv4_ADDRESS  *Ip\r
   )\r
 {\r
@@ -77,15 +75,15 @@ Ip4AsciiStrToIp (
 \r
   Index = 0;\r
 \r
-  while (*Str != 0) {\r
+  while (*Str != L'\0') {\r
 \r
     if (Index > 3) {\r
       return EFI_INVALID_PARAMETER;\r
     }\r
 \r
     Number = 0;\r
-    while (NET_IS_DIGIT (*Str)) {\r
-      Number = Number * 10 + (*Str - '0');\r
+    while ((*Str >= L'0') && (*Str <= L'9')) {\r
+      Number = Number * 10 + (*Str - L'0');\r
       Str++;\r
     }\r
 \r
@@ -95,7 +93,7 @@ Ip4AsciiStrToIp (
 \r
     Ip->Addr[Index] = (UINT8) Number;\r
 \r
-    if ((*Str != '\0') && (*Str != '.')) {\r
+    if ((*Str != L'\0') && (*Str != L'.')) {\r
       //\r
       // The current character should be either the NULL terminator or\r
       // the dot delimiter.\r
@@ -103,7 +101,7 @@ Ip4AsciiStrToIp (
       return EFI_INVALID_PARAMETER;\r
     }\r
 \r
-    if (*Str == '.') {\r
+    if (*Str == L'.') {\r
       //\r
       // Skip the delimiter.\r
       //\r
@@ -148,41 +146,46 @@ Ip4ConfigConvertDeviceConfigDataToIfrNvData (
   OUT IP4_CONFIG_IFR_NVDATA     *IfrFormNvData\r
   )\r
 {\r
-  EFI_STATUS                    Status;\r
-  NIC_IP4_CONFIG_INFO           *NicConfig;\r
-  UINTN                         ConfigLen;\r
+  NIC_IP4_CONFIG_INFO  *NicConfig;\r
 \r
-  IfrFormNvData->DhcpEnable = 1;\r
-\r
-  ConfigLen = sizeof (NIC_IP4_CONFIG_INFO) + sizeof (EFI_IP4_ROUTE_TABLE) * 2;\r
-  NicConfig = AllocateZeroPool (ConfigLen);\r
-  ASSERT (NicConfig != NULL);\r
-  Status = EfiNicIp4ConfigGetInfo (Ip4ConfigInstance, &ConfigLen, NicConfig);\r
-  if (!EFI_ERROR (Status)) {\r
+  NicConfig = EfiNicIp4ConfigGetInfo (Ip4ConfigInstance);\r
+  if (NicConfig != NULL) {\r
+    IfrFormNvData->Configure = 1;\r
+    Ip4ConfigInstance->Ip4ConfigCallbackInfo.Configured = TRUE;\r
     if (NicConfig->Source == IP4_CONFIG_SOURCE_DHCP) {\r
       IfrFormNvData->DhcpEnable = 1;\r
+      Ip4ConfigInstance->Ip4ConfigCallbackInfo.DhcpEnabled = TRUE;\r
     } else {\r
       IfrFormNvData->DhcpEnable = 0;\r
       Ip4ConfigIpToStr (&NicConfig->Ip4Info.StationAddress, IfrFormNvData->StationAddress);\r
       Ip4ConfigIpToStr (&NicConfig->Ip4Info.SubnetMask, IfrFormNvData->SubnetMask);\r
       Ip4ConfigIpToStr (&NicConfig->Ip4Info.RouteTable[1].GatewayAddress, IfrFormNvData->GatewayAddress);\r
+\r
+      Ip4ConfigInstance->Ip4ConfigCallbackInfo.DhcpEnabled = FALSE;\r
+      CopyMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.LocalIp, &NicConfig->Ip4Info.StationAddress, sizeof (EFI_IPv4_ADDRESS));\r
+      CopyMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.SubnetMask, &NicConfig->Ip4Info.SubnetMask, sizeof (EFI_IPv4_ADDRESS));\r
+      CopyMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.Gateway, &NicConfig->Ip4Info.RouteTable[1].GatewayAddress, sizeof (EFI_IPv4_ADDRESS));\r
     }\r
+\r
+    FreePool (NicConfig);\r
+  } else {\r
+    IfrFormNvData->Configure = 0;\r
+    Ip4ConfigInstance->Ip4ConfigCallbackInfo.Configured = FALSE;\r
   }\r
-  FreePool (NicConfig);\r
 }\r
 \r
 /**\r
   Convert the IFR data into the network configuration data and set the IP\r
   configure parameters for the NIC.\r
 \r
-  @param[in]       IfrFormNvData     The IFR nv data.\r
+  @param[in]       IfrFormNvData     The IFR NV data.\r
   @param[in, out]  Ip4ConfigInstance The IP4Config instance.\r
-                                 \r
-  @retval EFI_SUCCESS            The configure parameter for this NIC was \r
+\r
+  @retval EFI_SUCCESS            The configure parameter for this NIC was\r
                                  set successfully.\r
   @retval EFI_ALREADY_STARTED    There is a pending auto configuration.\r
   @retval EFI_NOT_FOUND          No auto configure parameter is found.\r
-  \r
+\r
 **/\r
 EFI_STATUS\r
 Ip4ConfigConvertIfrNvDataToDeviceConfigData (\r
@@ -190,6 +193,7 @@ Ip4ConfigConvertIfrNvDataToDeviceConfigData (
   IN OUT IP4_CONFIG_INSTANCE       *Ip4ConfigInstance\r
   )\r
 {\r
+  EFI_STATUS                Status;\r
   EFI_IP_ADDRESS            HostIp;\r
   EFI_IP_ADDRESS            SubnetMask;\r
   EFI_IP_ADDRESS            Gateway;\r
@@ -197,16 +201,47 @@ Ip4ConfigConvertIfrNvDataToDeviceConfigData (
   NIC_IP4_CONFIG_INFO       *NicInfo;\r
   EFI_IP_ADDRESS            Ip;\r
 \r
+  ZeroMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo, sizeof (IP4_SETTING_INFO));\r
+\r
+  Ip4ConfigInstance->Ip4ConfigCallbackInfo.Configured = IfrFormNvData->Configure;\r
+  Ip4ConfigInstance->Ip4ConfigCallbackInfo.DhcpEnabled = IfrFormNvData->DhcpEnable;\r
+  Ip4StrToIp (IfrFormNvData->StationAddress, &Ip4ConfigInstance->Ip4ConfigCallbackInfo.LocalIp);\r
+  Ip4StrToIp (IfrFormNvData->SubnetMask, &Ip4ConfigInstance->Ip4ConfigCallbackInfo.SubnetMask);\r
+  Ip4StrToIp (IfrFormNvData->GatewayAddress, &Ip4ConfigInstance->Ip4ConfigCallbackInfo.Gateway);\r
+\r
+  if (!Ip4ConfigInstance->Ip4ConfigCallbackInfo.Configured) {\r
+    //\r
+    // Clear the variable\r
+    //\r
+    ZeroMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo, sizeof (IP4_SETTING_INFO));\r
+\r
+    Status = EfiNicIp4ConfigSetInfo (Ip4ConfigInstance, NULL, TRUE);\r
+    if (Status == EFI_NOT_FOUND) {\r
+      return EFI_SUCCESS;\r
+    }\r
+\r
+    return Status;\r
+  }\r
+\r
   NicInfo = AllocateZeroPool (sizeof (NIC_IP4_CONFIG_INFO) + 2 * sizeof (EFI_IP4_ROUTE_TABLE));\r
   ASSERT (NicInfo != NULL);\r
 \r
   NicInfo->Ip4Info.RouteTable = (EFI_IP4_ROUTE_TABLE *) (NicInfo + 1);\r
 \r
-  if (!Ip4ConfigInstance->Ip4ConfigCallbackInfo.Enabled) {\r
+  if (!Ip4ConfigInstance->Ip4ConfigCallbackInfo.DhcpEnabled) {\r
     CopyMem (&HostIp.v4, &Ip4ConfigInstance->Ip4ConfigCallbackInfo.LocalIp, sizeof (HostIp.v4));\r
     CopyMem (&SubnetMask.v4, &Ip4ConfigInstance->Ip4ConfigCallbackInfo.SubnetMask, sizeof (SubnetMask.v4));\r
     CopyMem (&Gateway.v4, &Ip4ConfigInstance->Ip4ConfigCallbackInfo.Gateway, sizeof (Gateway.v4));\r
 \r
+    if (!NetIp4IsUnicast (NTOHL (HostIp.Addr[0]), 0)) {\r
+      CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Invalid IP address!", NULL);\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+    if (EFI_IP4_EQUAL (&SubnetMask, &mZeroIp4Addr)) {\r
+      CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Invalid Subnet Mask!", NULL);\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+\r
     if ((Gateway.Addr[0] != 0)) {\r
       if (SubnetMask.Addr[0] == 0) {\r
         CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Gateway address is set but subnet mask is zero.", NULL);\r
@@ -228,8 +263,12 @@ Ip4ConfigConvertIfrNvDataToDeviceConfigData (
     CopyMem (&NicInfo->Ip4Info.RouteTable[0].SubnetAddress, &Ip.v4, sizeof (EFI_IPv4_ADDRESS));\r
     CopyMem (&NicInfo->Ip4Info.RouteTable[0].SubnetMask, &SubnetMask.v4, sizeof (EFI_IPv4_ADDRESS));\r
     CopyMem (&NicInfo->Ip4Info.RouteTable[1].GatewayAddress, &Gateway.v4, sizeof (EFI_IPv4_ADDRESS));\r
+\r
   } else {\r
     NicInfo->Source = IP4_CONFIG_SOURCE_DHCP;\r
+    ZeroMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.LocalIp, sizeof (EFI_IPv4_ADDRESS));\r
+    ZeroMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.SubnetMask, sizeof (EFI_IPv4_ADDRESS));\r
+    ZeroMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.Gateway, sizeof (EFI_IPv4_ADDRESS));\r
   }\r
 \r
   NicInfo->Perment = TRUE;\r
@@ -279,12 +318,6 @@ Ip4ConfigConvertIfrNvDataToDeviceConfigData (
                                   parts of the results that must be\r
                                   stored awaiting possible future\r
                                   protocols.\r
-  @retval EFI_INVALID_PARAMETER   For example, passing in a NULL\r
-                                  for the Request parameter\r
-                                  would result in this type of\r
-                                  error. In this case, the\r
-                                  Progress parameter would be\r
-                                  set to NULL.\r
   @retval EFI_NOT_FOUND           Routing data doesn't match any\r
                                   known driver. Progress set to the\r
                                   first character in the routing header.\r
@@ -310,76 +343,173 @@ Ip4DeviceExtractConfig (
   )\r
 {\r
   EFI_STATUS                       Status;\r
-  UINTN                            ConfigLen;\r
   NIC_IP4_CONFIG_INFO              *IfrDeviceNvData;\r
+  NIC_IP4_CONFIG_INFO              *NicConfig;\r
   IP4_CONFIG_INSTANCE              *Ip4ConfigInstance;\r
   IP4_CONFIG_IFR_NVDATA            *IfrFormNvData;\r
+  EFI_STRING                       ConfigRequestHdr;\r
+  EFI_STRING                       ConfigRequest;\r
+  EFI_STRING                       DeviceResult;\r
+  EFI_STRING                       FormResult;\r
+  CHAR16                           *StrPointer;\r
+  BOOLEAN                          AllocatedRequest;\r
+  UINTN                            Size;\r
+  UINTN                            BufferSize;\r
 \r
-  if (Request == NULL || Progress == NULL || Results == NULL) {\r
+  if (Progress == NULL || Results == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
-  *Progress = Request;\r
-\r
+  *Progress     = Request;\r
+  Size          = 0;\r
+  DeviceResult  = NULL;\r
+  FormResult    = NULL;\r
+  ConfigRequest = NULL;\r
+  Status        = EFI_SUCCESS;\r
+  AllocatedRequest  = FALSE;\r
   Ip4ConfigInstance = IP4_CONFIG_INSTANCE_FROM_CONFIG_ACCESS (This);\r
 \r
   //\r
   // Check Request data in <ConfigHdr>.\r
   //\r
-  if (HiiIsConfigHdrMatch (Request, &gEfiNicIp4ConfigVariableGuid, EFI_NIC_IP4_CONFIG_VARIABLE)) {\r
+  if ((Request == NULL) || HiiIsConfigHdrMatch (Request, &gEfiNicIp4ConfigVariableGuid, EFI_NIC_IP4_CONFIG_VARIABLE)) {\r
     IfrDeviceNvData = AllocateZeroPool (NIC_ITEM_CONFIG_SIZE);\r
     if (IfrDeviceNvData == NULL) {\r
       return EFI_OUT_OF_RESOURCES;\r
     }\r
 \r
-    ConfigLen = sizeof (NIC_IP4_CONFIG_INFO) + sizeof (EFI_IP4_ROUTE_TABLE) * 2;\r
-    Status = EfiNicIp4ConfigGetInfo (Ip4ConfigInstance, &ConfigLen, IfrDeviceNvData);\r
-    if (EFI_ERROR (Status)) {\r
-      FreePool (IfrDeviceNvData);\r
+    NicConfig = EfiNicIp4ConfigGetInfo (Ip4ConfigInstance);\r
+    if (NicConfig == NULL) {\r
       return EFI_NOT_FOUND;\r
     }\r
-  \r
+    CopyMem (IfrDeviceNvData, NicConfig, SIZEOF_NIC_IP4_CONFIG_INFO (NicConfig));\r
+    FreePool (NicConfig);\r
+\r
+    ConfigRequest = Request;\r
+    if ((Request == NULL) || (StrStr (Request, L"OFFSET") == NULL)) {\r
+      //\r
+      // Request has no request element, construct full request string.\r
+      // Allocate and fill a buffer large enough to hold the <ConfigHdr> template\r
+      // followed by "&OFFSET=0&WIDTH=WWWWWWWWWWWWWWWW" followed by a Null-terminator\r
+      //\r
+      ConfigRequestHdr = HiiConstructConfigHdr (&gEfiNicIp4ConfigVariableGuid, EFI_NIC_IP4_CONFIG_VARIABLE, Ip4ConfigInstance->ChildHandle);\r
+      Size = (StrLen (ConfigRequestHdr) + 32 + 1) * sizeof (CHAR16);\r
+      ConfigRequest = AllocateZeroPool (Size);\r
+      ASSERT (ConfigRequest != NULL);\r
+      AllocatedRequest = TRUE;\r
+      BufferSize = NIC_ITEM_CONFIG_SIZE;\r
+      UnicodeSPrint (ConfigRequest, Size, L"%s&OFFSET=0&WIDTH=%016LX", ConfigRequestHdr, (UINT64)BufferSize);\r
+      FreePool (ConfigRequestHdr);\r
+    }\r
+\r
     //\r
     // Convert buffer data to <ConfigResp> by helper function BlockToConfig()\r
     //\r
     Status = gHiiConfigRouting->BlockToConfig (\r
                                   gHiiConfigRouting,\r
-                                  Request,\r
+                                  ConfigRequest,\r
                                   (UINT8 *) IfrDeviceNvData,\r
                                   NIC_ITEM_CONFIG_SIZE,\r
-                                  Results,\r
+                                  &DeviceResult,\r
                                   Progress\r
                                   );\r
-  \r
+\r
     FreePool (IfrDeviceNvData);\r
+    //\r
+    // Free the allocated config request string.\r
+    //\r
+    if (AllocatedRequest) {\r
+      FreePool (ConfigRequest);\r
+      ConfigRequest = NULL;\r
+    }\r
+\r
+    if (EFI_ERROR (Status)) {\r
+      goto Failure;\r
+    }\r
+  }\r
 \r
-  } else if (HiiIsConfigHdrMatch (Request, &mNicIp4ConfigNvDataGuid, EFI_NIC_IP4_CONFIG_VARIABLE)) {\r
+  if ((Request == NULL) || HiiIsConfigHdrMatch (Request, &gNicIp4ConfigNvDataGuid, EFI_NIC_IP4_CONFIG_VARIABLE)) {\r
 \r
-    IfrFormNvData = AllocateZeroPool (NIC_ITEM_CONFIG_SIZE);\r
+    IfrFormNvData = AllocateZeroPool (sizeof (IP4_CONFIG_IFR_NVDATA));\r
     if (IfrFormNvData == NULL) {\r
       return EFI_OUT_OF_RESOURCES;\r
     }\r
-  \r
+\r
     Ip4ConfigConvertDeviceConfigDataToIfrNvData (Ip4ConfigInstance, IfrFormNvData);\r
-  \r
+\r
+    ConfigRequest = Request;\r
+    if ((Request == NULL) || (StrStr (Request, L"OFFSET") == NULL)) {\r
+      //\r
+      // Request has no request element, construct full request string.\r
+      // Allocate and fill a buffer large enough to hold the <ConfigHdr> template\r
+      // followed by "&OFFSET=0&WIDTH=WWWWWWWWWWWWWWWW" followed by a Null-terminator\r
+      //\r
+      ConfigRequestHdr = HiiConstructConfigHdr (&gNicIp4ConfigNvDataGuid, EFI_NIC_IP4_CONFIG_VARIABLE, Ip4ConfigInstance->ChildHandle);\r
+      Size = (StrLen (ConfigRequestHdr) + 32 + 1) * sizeof (CHAR16);\r
+      ConfigRequest = AllocateZeroPool (Size);\r
+      ASSERT (ConfigRequest != NULL);\r
+      AllocatedRequest = TRUE;\r
+      BufferSize = sizeof (IP4_CONFIG_IFR_NVDATA);\r
+      UnicodeSPrint (ConfigRequest, Size, L"%s&OFFSET=0&WIDTH=%016LX", ConfigRequestHdr, (UINT64)BufferSize);\r
+      FreePool (ConfigRequestHdr);\r
+    }\r
+\r
     //\r
     // Convert buffer data to <ConfigResp> by helper function BlockToConfig()\r
     //\r
     Status = gHiiConfigRouting->BlockToConfig (\r
                                   gHiiConfigRouting,\r
-                                  Request,\r
+                                  ConfigRequest,\r
                                   (UINT8 *) IfrFormNvData,\r
                                   sizeof (IP4_CONFIG_IFR_NVDATA),\r
-                                  Results,\r
+                                  &FormResult,\r
                                   Progress\r
                                   );\r
-  \r
+\r
     FreePool (IfrFormNvData);\r
+    //\r
+    // Free the allocated config request string.\r
+    //\r
+    if (AllocatedRequest) {\r
+      FreePool (ConfigRequest);\r
+      ConfigRequest = NULL;\r
+    }\r
 \r
+    if (EFI_ERROR (Status)) {\r
+      goto Failure;\r
+    }\r
+  }\r
+\r
+  if (Request == NULL) {\r
+    Size = StrLen (DeviceResult);\r
+    Size = Size + 1;\r
+    Size = Size + StrLen (FormResult) + 1;\r
+    *Results = AllocateZeroPool (Size * sizeof (CHAR16));\r
+    ASSERT (*Results != NULL);\r
+    StrPointer  = *Results;\r
+    StrCpy (StrPointer, DeviceResult);\r
+    StrPointer  = StrPointer + StrLen (StrPointer);\r
+    *StrPointer = L'&';\r
+    StrCpy (StrPointer + 1, FormResult);\r
+    FreePool (DeviceResult);\r
+    FreePool (FormResult);\r
+  } else if (HiiIsConfigHdrMatch (Request, &gEfiNicIp4ConfigVariableGuid, EFI_NIC_IP4_CONFIG_VARIABLE)) {\r
+    *Results = DeviceResult;\r
+  } else if (HiiIsConfigHdrMatch (Request, &gNicIp4ConfigNvDataGuid, EFI_NIC_IP4_CONFIG_VARIABLE)) {\r
+    *Results = FormResult;\r
   } else {\r
     return EFI_NOT_FOUND;\r
   }\r
 \r
+Failure:\r
+  //\r
+  // Set Progress string to the original request string.\r
+  //\r
+  if (Request == NULL) {\r
+    *Progress = NULL;\r
+  } else if (StrStr (Request, L"OFFSET") == NULL) {\r
+    *Progress = Request + StrLen (Request);\r
+  }\r
 \r
   return Status;\r
 }\r
@@ -428,7 +558,7 @@ Ip4DeviceRouteConfig (
   EFI_STATUS                       Status;\r
   UINTN                            BufferSize;\r
   NIC_IP4_CONFIG_INFO              *IfrDeviceNvData;\r
-  IP4_CONFIG_IFR_NVDATA            *IfrFormNvData; \r
+  IP4_CONFIG_IFR_NVDATA            *IfrFormNvData;\r
   NIC_IP4_CONFIG_INFO              *NicInfo;\r
   IP4_CONFIG_INSTANCE              *Ip4ConfigInstance;\r
   EFI_MAC_ADDRESS                  ZeroMac;\r
@@ -437,6 +567,11 @@ Ip4DeviceRouteConfig (
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
+  //\r
+  // Reclaim Ip4Config variable\r
+  //\r
+  Ip4ConfigReclaimVariable ();\r
+\r
   *Progress = Configuration;\r
 \r
   Ip4ConfigInstance = IP4_CONFIG_INSTANCE_FROM_CONFIG_ACCESS (This);\r
@@ -444,7 +579,7 @@ Ip4DeviceRouteConfig (
   //\r
   // Check Routing data in <ConfigHdr>.\r
   //\r
-  if (HiiIsConfigHdrMatch (Configuration, &mNicIp4ConfigNvDataGuid, EFI_NIC_IP4_CONFIG_VARIABLE)) {\r
+  if (HiiIsConfigHdrMatch (Configuration, &gNicIp4ConfigNvDataGuid, EFI_NIC_IP4_CONFIG_VARIABLE)) {\r
     //\r
     // Convert buffer data to <ConfigResp> by helper function BlockToConfig()\r
     //\r
@@ -464,7 +599,7 @@ Ip4DeviceRouteConfig (
     if (!EFI_ERROR (Status)) {\r
       Status = Ip4ConfigConvertIfrNvDataToDeviceConfigData (IfrFormNvData, Ip4ConfigInstance);\r
     }\r
-    \r
+\r
     FreePool (IfrFormNvData);\r
 \r
   } else if (HiiIsConfigHdrMatch (Configuration, &gEfiNicIp4ConfigVariableGuid, EFI_NIC_IP4_CONFIG_VARIABLE)) {\r
@@ -485,10 +620,15 @@ Ip4DeviceRouteConfig (
     if (!EFI_ERROR (Status)) {\r
       ZeroMem (&ZeroMac, sizeof (EFI_MAC_ADDRESS));\r
       if (CompareMem (&IfrDeviceNvData->NicAddr.MacAddr, &ZeroMac, IfrDeviceNvData->NicAddr.Len) != 0) {\r
-        BufferSize = sizeof (NIC_IP4_CONFIG_INFO) + sizeof (EFI_IP4_ROUTE_TABLE) * IfrDeviceNvData->Ip4Info.RouteTableSize;\r
-        NicInfo = AllocateCopyPool (BufferSize, IfrDeviceNvData); \r
+        BufferSize = SIZEOF_NIC_IP4_CONFIG_INFO (IfrDeviceNvData);\r
+        NicInfo = AllocateCopyPool (BufferSize, IfrDeviceNvData);\r
+        if (NicInfo == NULL) {\r
+          return EFI_OUT_OF_RESOURCES;\r
+        }\r
         Status = EfiNicIp4ConfigSetInfo (Ip4ConfigInstance, NicInfo, TRUE);\r
+        FreePool (NicInfo);\r
       } else {\r
+        ZeroMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo, sizeof (IP4_SETTING_INFO));\r
         Status = EfiNicIp4ConfigSetInfo (Ip4ConfigInstance, NULL, TRUE);\r
       }\r
     }\r
@@ -499,7 +639,7 @@ Ip4DeviceRouteConfig (
 \r
     return EFI_NOT_FOUND;\r
   }\r
-  \r
+\r
   return Status;\r
 \r
 }\r
@@ -542,7 +682,6 @@ Ip4FormCallback (
   )\r
 {\r
   IP4_CONFIG_INSTANCE       *Ip4ConfigInstance;\r
-  CHAR8                     Ip4String[IP4_STR_MAX_SIZE];\r
   IP4_CONFIG_IFR_NVDATA     *IfrFormNvData;\r
   EFI_IP_ADDRESS            HostIp;\r
   EFI_IP_ADDRESS            SubnetMask;\r
@@ -550,92 +689,76 @@ Ip4FormCallback (
   EFI_STATUS                Status;\r
   EFI_INPUT_KEY             Key;\r
 \r
-  Ip4ConfigInstance = IP4_CONFIG_INSTANCE_FROM_CONFIG_ACCESS (This);\r
-\r
-  IfrFormNvData = AllocateZeroPool (sizeof (IP4_CONFIG_IFR_NVDATA));\r
-  if (IfrFormNvData == NULL) {\r
-    return EFI_OUT_OF_RESOURCES;\r
-  }\r
-\r
-  //\r
-  // Retrive uncommitted data from Browser\r
-  //\r
-  if (!HiiGetBrowserData (&mNicIp4ConfigNvDataGuid, EFI_NIC_IP4_CONFIG_VARIABLE, sizeof (IP4_CONFIG_IFR_NVDATA), (UINT8 *) IfrFormNvData)) {\r
-    FreePool (IfrFormNvData);\r
-    return EFI_NOT_FOUND;\r
-  }\r
-\r
-  Status = EFI_SUCCESS;\r
+  if (Action == EFI_BROWSER_ACTION_CHANGING) {\r
+    Ip4ConfigInstance = IP4_CONFIG_INSTANCE_FROM_CONFIG_ACCESS (This);\r
 \r
-  switch (QuestionId) {\r
+    IfrFormNvData = AllocateZeroPool (sizeof (IP4_CONFIG_IFR_NVDATA));\r
+    if (IfrFormNvData == NULL) {\r
+      return EFI_OUT_OF_RESOURCES;\r
+    }\r
 \r
-  case KEY_DHCP_ENABLE:\r
-    if (IfrFormNvData->DhcpEnable == 0) {\r
-      Ip4ConfigInstance->Ip4ConfigCallbackInfo.Enabled = FALSE;\r
-    } else {\r
-      Ip4ConfigInstance->Ip4ConfigCallbackInfo.Enabled = TRUE;\r
+    //\r
+    // Retrive uncommitted data from Browser\r
+    //\r
+    if (!HiiGetBrowserData (&gNicIp4ConfigNvDataGuid, EFI_NIC_IP4_CONFIG_VARIABLE, sizeof (IP4_CONFIG_IFR_NVDATA), (UINT8 *) IfrFormNvData)) {\r
+      FreePool (IfrFormNvData);\r
+      return EFI_NOT_FOUND;\r
     }\r
 \r
-    break;\r
+    Status = EFI_SUCCESS;\r
 \r
-  case KEY_LOCAL_IP:\r
-    UnicodeStrToAsciiStr (IfrFormNvData->StationAddress, Ip4String);\r
-    Status = Ip4AsciiStrToIp (Ip4String, &HostIp.v4);\r
-    if (EFI_ERROR (Status) || !NetIp4IsUnicast (NTOHL (HostIp.Addr[0]), 0)) {\r
-      CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Invalid IP address!", NULL);\r
-      Status = EFI_INVALID_PARAMETER;\r
-    } else {\r
-      CopyMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.LocalIp, &HostIp.v4, sizeof (HostIp.v4));\r
-    }\r
+    switch (QuestionId) {\r
+    case KEY_LOCAL_IP:\r
+      Status = Ip4StrToIp (IfrFormNvData->StationAddress, &HostIp.v4);\r
+      if (EFI_ERROR (Status) || !NetIp4IsUnicast (NTOHL (HostIp.Addr[0]), 0)) {\r
+        CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Invalid IP address!", NULL);\r
+        Status = EFI_INVALID_PARAMETER;\r
+      } else {\r
+        CopyMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.LocalIp, &HostIp.v4, sizeof (HostIp.v4));\r
+      }\r
 \r
-    break;\r
+      break;\r
 \r
-  case KEY_SUBNET_MASK:\r
-    UnicodeStrToAsciiStr (IfrFormNvData->SubnetMask, Ip4String);\r
-    Status = Ip4AsciiStrToIp (Ip4String, &SubnetMask.v4);\r
-    if (EFI_ERROR (Status) || ((SubnetMask.Addr[0] != 0) && (GetSubnetMaskPrefixLength (&SubnetMask.v4) == 0))) {\r
-      CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Invalid SubnetMask!", NULL);\r
-      Status = EFI_INVALID_PARAMETER;\r
-    } else {\r
-      CopyMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.SubnetMask, &SubnetMask.v4, sizeof (SubnetMask.v4));\r
-    }\r
+    case KEY_SUBNET_MASK:\r
+      Status = Ip4StrToIp (IfrFormNvData->SubnetMask, &SubnetMask.v4);\r
+      if (EFI_ERROR (Status) || ((SubnetMask.Addr[0] != 0) && (GetSubnetMaskPrefixLength (&SubnetMask.v4) == 0))) {\r
+        CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Invalid Subnet Mask!", NULL);\r
+        Status = EFI_INVALID_PARAMETER;\r
+      } else {\r
+        CopyMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.SubnetMask, &SubnetMask.v4, sizeof (SubnetMask.v4));\r
+      }\r
 \r
-    break;\r
+      break;\r
 \r
-  case KEY_GATE_WAY:\r
-    UnicodeStrToAsciiStr (IfrFormNvData->GatewayAddress, Ip4String);\r
-    Status = Ip4AsciiStrToIp (Ip4String, &Gateway.v4);\r
-    if (EFI_ERROR (Status) || ((Gateway.Addr[0] != 0) && !NetIp4IsUnicast (NTOHL (Gateway.Addr[0]), 0))) {\r
-      CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Invalid Gateway!", NULL);\r
-      Status = EFI_INVALID_PARAMETER;\r
-    } else {\r
-      CopyMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.Gateway, &Gateway.v4, sizeof (Gateway.v4));\r
-    }\r
+    case KEY_GATE_WAY:\r
+      Status = Ip4StrToIp (IfrFormNvData->GatewayAddress, &Gateway.v4);\r
+      if (EFI_ERROR (Status) || ((Gateway.Addr[0] != 0) && !NetIp4IsUnicast (NTOHL (Gateway.Addr[0]), 0))) {\r
+        CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Invalid Gateway!", NULL);\r
+        Status = EFI_INVALID_PARAMETER;\r
+      } else {\r
+        CopyMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.Gateway, &Gateway.v4, sizeof (Gateway.v4));\r
+      }\r
 \r
-    break;\r
+      break;\r
 \r
-  case KEY_SAVE_CHANGES:\r
-    Status = Ip4ConfigConvertIfrNvDataToDeviceConfigData (IfrFormNvData, Ip4ConfigInstance);\r
\r
-    *ActionRequest = EFI_BROWSER_ACTION_REQUEST_SUBMIT;\r
+    case KEY_SAVE_CHANGES:\r
+      Status = Ip4ConfigConvertIfrNvDataToDeviceConfigData (IfrFormNvData, Ip4ConfigInstance);\r
+      *ActionRequest = EFI_BROWSER_ACTION_REQUEST_SUBMIT;\r
+      break;\r
 \r
-    break;\r
+    default:\r
+      break;\r
+    }\r
 \r
-  default:\r
-   \r
-    break;\r
-  }\r
+    FreePool (IfrFormNvData);\r
 \r
-  if (!EFI_ERROR (Status)) {\r
-    //\r
-    // Pass changed uncommitted data back to Form Browser\r
-    //\r
-    HiiSetBrowserData (&gEfiNicIp4ConfigVariableGuid, EFI_NIC_IP4_CONFIG_VARIABLE, sizeof (IP4_CONFIG_IFR_NVDATA), (UINT8 *) IfrFormNvData, NULL);\r
+    return Status;\r
   }\r
 \r
-  FreePool (IfrFormNvData);\r
-\r
-  return Status;\r
+  //\r
+  // All other action return unsupported.\r
+  //\r
+  return EFI_UNSUPPORTED;\r
 }\r
 \r
 /**\r
@@ -690,7 +813,7 @@ Ip4ConfigDeviceInit (
   Status = gBS->InstallMultipleProtocolInterfaces (\r
                   &Instance->ChildHandle,\r
                   &gEfiDevicePathProtocolGuid,\r
-                  Instance->HiiVendorDevicePath,      \r
+                  Instance->HiiVendorDevicePath,\r
                   &gEfiHiiConfigAccessProtocolGuid,\r
                   ConfigAccess,\r
                   NULL\r
@@ -704,18 +827,18 @@ Ip4ConfigDeviceInit (
                     &gEfiManagedNetworkServiceBindingProtocolGuid,\r
                     (VOID **) &MnpSb,\r
                     Instance->Image,\r
-                    Instance->ChildHandle, \r
+                    Instance->ChildHandle,\r
                     EFI_OPEN_PROTOCOL_BY_CHILD_CONTROLLER\r
                     );\r
   }\r
 \r
   ASSERT_EFI_ERROR (Status);\r
\r
+\r
   //\r
   // Publish our HII data\r
   //\r
   Instance->RegisteredHandle = HiiAddPackages (\r
-                                 &mNicIp4ConfigNvDataGuid,\r
+                                 &gNicIp4ConfigNvDataGuid,\r
                                  Instance->ChildHandle,\r
                                  Ip4ConfigDxeStrings,\r
                                  Ip4ConfigDxeBin,\r
@@ -726,16 +849,16 @@ Ip4ConfigDeviceInit (
   }\r
 \r
   //\r
-  // Append MAC string in the menu string and tile string\r
+  // Append MAC string in the menu help string and tile help string\r
   //\r
   Status = NetLibGetMacString (Instance->Controller, Instance->Image, &MacString);\r
   if (!EFI_ERROR (Status)) {\r
-    OldMenuString = HiiGetString (Instance->RegisteredHandle, STRING_TOKEN (STR_IP4_CONFIG_FORM_TITLE), NULL);\r
+    OldMenuString = HiiGetString (Instance->RegisteredHandle, STRING_TOKEN (STR_IP4_CONFIG_FORM_HELP), NULL);\r
     UnicodeSPrint (MenuString, 128, L"%s (MAC:%s)", OldMenuString, MacString);\r
-    HiiSetString (Instance->RegisteredHandle, STRING_TOKEN (STR_IP4_CONFIG_FORM_TITLE), MenuString, NULL);\r
-  \r
+    HiiSetString (Instance->RegisteredHandle, STRING_TOKEN (STR_IP4_CONFIG_FORM_HELP), MenuString, NULL);\r
+\r
     UnicodeSPrint (PortString, 128, L"MAC:%s", MacString);\r
-    HiiSetString (Instance->RegisteredHandle, STRING_TOKEN (STR_IP4_DEVICE_FORM_TITLE), PortString, NULL);\r
+    HiiSetString (Instance->RegisteredHandle, STRING_TOKEN (STR_IP4_DEVICE_FORM_HELP), PortString, NULL);\r
     FreePool (MacString);\r
   }\r
 \r
@@ -776,7 +899,7 @@ Ip4ConfigDeviceUnload (
   gBS->UninstallMultipleProtocolInterfaces (\r
          Instance->ChildHandle,\r
          &gEfiDevicePathProtocolGuid,\r
-         Instance->HiiVendorDevicePath, \r
+         Instance->HiiVendorDevicePath,\r
          &gEfiHiiConfigAccessProtocolGuid,\r
          &Instance->HiiConfigAccessProtocol,\r
          NULL\r