]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Universal/Network/Ip4ConfigDxe/Ip4ConfigNv.c
Avoid change the return status from above code which will also be used later.
[mirror_edk2.git] / MdeModulePkg / Universal / Network / Ip4ConfigDxe / Ip4ConfigNv.c
index 9417a44c25864ed2329528e7b90170b0f7632a63..89e1ddaf39063cb49df33f03993f80291bab6667 100644 (file)
@@ -1,8 +1,8 @@
 /** @file\r
   Helper functions for configuring or getting the parameters relating to Ip4.\r
 \r
-Copyright (c) 2009 - 2010, Intel Corporation.<BR>\r
-All rights reserved. This program and the accompanying materials\r
+Copyright (c) 2009 - 2012, 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,35 +146,39 @@ Ip4ConfigConvertDeviceConfigDataToIfrNvData (
   OUT IP4_CONFIG_IFR_NVDATA     *IfrFormNvData\r
   )\r
 {\r
-  EFI_STATUS                    Status;\r
-  NIC_IP4_CONFIG_INFO           *NicConfig;\r
-  UINTN                         ConfigLen;\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
+  NIC_IP4_CONFIG_INFO  *NicConfig;\r
+\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
-\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, out]  Ip4ConfigInstance The IP4Config instance.\r
 \r
   @retval EFI_SUCCESS            The configure parameter for this NIC was\r
@@ -187,6 +189,7 @@ Ip4ConfigConvertDeviceConfigDataToIfrNvData (
 **/\r
 EFI_STATUS\r
 Ip4ConfigConvertIfrNvDataToDeviceConfigData (\r
+  IN     IP4_CONFIG_IFR_NVDATA     *IfrFormNvData,\r
   IN OUT IP4_CONFIG_INSTANCE       *Ip4ConfigInstance\r
   )\r
 {\r
@@ -198,6 +201,14 @@ 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
@@ -260,7 +271,7 @@ Ip4ConfigConvertIfrNvDataToDeviceConfigData (
     ZeroMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.Gateway, sizeof (EFI_IPv4_ADDRESS));\r
   }\r
 \r
-  NicInfo->Perment = TRUE;\r
+  NicInfo->Permanent = TRUE;\r
   CopyMem (&NicInfo->NicAddr, &Ip4ConfigInstance->NicAddr, sizeof (NIC_ADDR));\r
 \r
   return EfiNicIp4ConfigSetInfo (Ip4ConfigInstance, NicInfo, TRUE);\r
@@ -307,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
@@ -338,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
+    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
     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
     Ip4ConfigConvertDeviceConfigDataToIfrNvData (Ip4ConfigInstance, IfrFormNvData);\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
     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
@@ -465,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
@@ -472,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
@@ -490,7 +597,7 @@ Ip4DeviceRouteConfig (
                                   Progress\r
                                   );\r
     if (!EFI_ERROR (Status)) {\r
-      Status = Ip4ConfigConvertIfrNvDataToDeviceConfigData (Ip4ConfigInstance);\r
+      Status = Ip4ConfigConvertIfrNvDataToDeviceConfigData (IfrFormNvData, Ip4ConfigInstance);\r
     }\r
 \r
     FreePool (IfrFormNvData);\r
@@ -513,9 +620,13 @@ 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
+        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
@@ -571,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
@@ -579,101 +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
-\r
-  switch (QuestionId) {\r
-\r
-  case KEY_ENABLE:\r
-    if (IfrFormNvData->Configure == 0) {\r
-      Ip4ConfigInstance->Ip4ConfigCallbackInfo.Configured = FALSE;\r
-    } else {\r
-      Ip4ConfigInstance->Ip4ConfigCallbackInfo.Configured = TRUE;\r
-    }\r
-    break;\r
+  if (Action == EFI_BROWSER_ACTION_CHANGED) {\r
+    Ip4ConfigInstance = IP4_CONFIG_INSTANCE_FROM_CONFIG_ACCESS (This);\r
 \r
-  case KEY_DHCP_ENABLE:\r
-    if (IfrFormNvData->DhcpEnable == 0) {\r
-      Ip4ConfigInstance->Ip4ConfigCallbackInfo.DhcpEnabled = FALSE;\r
-    } else {\r
-      Ip4ConfigInstance->Ip4ConfigCallbackInfo.DhcpEnabled = TRUE;\r
+    IfrFormNvData = AllocateZeroPool (sizeof (IP4_CONFIG_IFR_NVDATA));\r
+    if (IfrFormNvData == NULL) {\r
+      return EFI_OUT_OF_RESOURCES;\r
     }\r
 \r
-    break;\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
+    // 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
-\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
+    Status = EFI_SUCCESS;\r
 \r
-    break;\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
-  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
+      break;\r
 \r
-    break;\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
-  case KEY_SAVE_CHANGES:\r
+      break;\r
 \r
-    Status = Ip4ConfigConvertIfrNvDataToDeviceConfigData (Ip4ConfigInstance);\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
-    *ActionRequest = EFI_BROWSER_ACTION_REQUEST_SUBMIT;\r
+      break;\r
 \r
-    break;\r
+    case KEY_SAVE_CHANGES:\r
+      Status = Ip4ConfigConvertIfrNvDataToDeviceConfigData (IfrFormNvData, Ip4ConfigInstance);\r
+      *ActionRequest = EFI_BROWSER_ACTION_REQUEST_SUBMIT;\r
+      break;\r
 \r
-  default:\r
+    default:\r
+      break;\r
+    }\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
@@ -753,7 +838,7 @@ Ip4ConfigDeviceInit (
   // Publish our HII data\r
   //\r
   Instance->RegisteredHandle = HiiAddPackages (\r
-                                 &mNicIp4ConfigNvDataGuid,\r
+                                 &gNicIp4ConfigNvDataGuid,\r
                                  Instance->ChildHandle,\r
                                  Ip4ConfigDxeStrings,\r
                                  Ip4ConfigDxeBin,\r
@@ -764,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
+    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