]> git.proxmox.com Git - mirror_edk2.git/blobdiff - NetworkPkg/UefiPxeBcDxe/PxeBcDhcp4.c
NetworkPkg: Apply uncrustify changes
[mirror_edk2.git] / NetworkPkg / UefiPxeBcDxe / PxeBcDhcp4.c
index 0edf7cc7999079c4bcdfe6d4922158454cfebeda..91146b78cb1e4931b137b8d060ac5f6f043512cb 100644 (file)
@@ -12,7 +12,7 @@
 //\r
 // This is a map from the interested DHCP4 option tags' index to the tag value.\r
 //\r
-UINT8 mInterestedDhcp4Tags[PXEBC_DHCP4_TAG_INDEX_MAX] = {\r
+UINT8  mInterestedDhcp4Tags[PXEBC_DHCP4_TAG_INDEX_MAX] = {\r
   DHCP4_TAG_BOOTFILE_LEN,\r
   DHCP4_TAG_VENDOR,\r
   DHCP4_TAG_OVERLOAD,\r
@@ -25,8 +25,7 @@ UINT8 mInterestedDhcp4Tags[PXEBC_DHCP4_TAG_INDEX_MAX] = {
 //\r
 // There are 4 times retries with the value of 4, 8, 16 and 32, refers to PXE2.1 spec.\r
 //\r
-UINT32 mPxeDhcpTimeout[4] = {4, 8, 16, 32};\r
-\r
+UINT32  mPxeDhcpTimeout[4] = { 4, 8, 16, 32 };\r
 \r
 /**\r
   Parse a certain dhcp4 option by OptTag in Buffer, and return with start pointer.\r
@@ -41,19 +40,18 @@ UINT32 mPxeDhcpTimeout[4] = {4, 8, 16, 32};
 **/\r
 EFI_DHCP4_PACKET_OPTION *\r
 PxeBcParseDhcp4Options (\r
-  IN UINT8                      *Buffer,\r
-  IN UINT32                     Length,\r
-  IN UINT8                      OptTag\r
+  IN UINT8   *Buffer,\r
+  IN UINT32  Length,\r
+  IN UINT8   OptTag\r
   )\r
 {\r
-  EFI_DHCP4_PACKET_OPTION       *Option;\r
-  UINT32                        Offset;\r
+  EFI_DHCP4_PACKET_OPTION  *Option;\r
+  UINT32                   Offset;\r
 \r
-  Option  = (EFI_DHCP4_PACKET_OPTION *) Buffer;\r
-  Offset  = 0;\r
+  Option = (EFI_DHCP4_PACKET_OPTION *)Buffer;\r
+  Offset = 0;\r
 \r
   while (Offset < Length && Option->OpCode != DHCP4_TAG_EOP) {\r
-\r
     if (Option->OpCode == OptTag) {\r
       //\r
       // Found the required option.\r
@@ -70,13 +68,12 @@ PxeBcParseDhcp4Options (
       Offset += Option->Length + 2;\r
     }\r
 \r
-    Option = (EFI_DHCP4_PACKET_OPTION *) (Buffer + Offset);\r
+    Option = (EFI_DHCP4_PACKET_OPTION *)(Buffer + Offset);\r
   }\r
 \r
   return NULL;\r
 }\r
 \r
-\r
 /**\r
   Parse the PXE vendor options and extract the information from them.\r
 \r
@@ -86,18 +83,18 @@ PxeBcParseDhcp4Options (
 **/\r
 VOID\r
 PxeBcParseVendorOptions (\r
-  IN EFI_DHCP4_PACKET_OPTION    *Dhcp4Option,\r
-  IN PXEBC_VENDOR_OPTION        *VendorOption\r
+  IN EFI_DHCP4_PACKET_OPTION  *Dhcp4Option,\r
+  IN PXEBC_VENDOR_OPTION      *VendorOption\r
   )\r
 {\r
-  UINT32                        *BitMap;\r
-  UINT8                         VendorOptionLen;\r
-  EFI_DHCP4_PACKET_OPTION       *PxeOption;\r
-  UINT8                         Offset;\r
+  UINT32                   *BitMap;\r
+  UINT8                    VendorOptionLen;\r
+  EFI_DHCP4_PACKET_OPTION  *PxeOption;\r
+  UINT8                    Offset;\r
 \r
   BitMap          = VendorOption->BitMap;\r
   VendorOptionLen = Dhcp4Option->Length;\r
-  PxeOption       = (EFI_DHCP4_PACKET_OPTION *) &Dhcp4Option->Data[0];\r
+  PxeOption       = (EFI_DHCP4_PACKET_OPTION *)&Dhcp4Option->Data[0];\r
   Offset          = 0;\r
 \r
   ASSERT (PxeOption != NULL);\r
@@ -107,84 +104,83 @@ PxeBcParseVendorOptions (
     // Parse all the interesting PXE vendor options one by one.\r
     //\r
     switch (PxeOption->OpCode) {\r
+      case PXEBC_VENDOR_TAG_MTFTP_IP:\r
 \r
-    case PXEBC_VENDOR_TAG_MTFTP_IP:\r
-\r
-      CopyMem (&VendorOption->MtftpIp, PxeOption->Data, sizeof (EFI_IPv4_ADDRESS));\r
-      break;\r
+        CopyMem (&VendorOption->MtftpIp, PxeOption->Data, sizeof (EFI_IPv4_ADDRESS));\r
+        break;\r
 \r
-    case PXEBC_VENDOR_TAG_MTFTP_CPORT:\r
+      case PXEBC_VENDOR_TAG_MTFTP_CPORT:\r
 \r
-      CopyMem (&VendorOption->MtftpCPort, PxeOption->Data, sizeof (VendorOption->MtftpCPort));\r
-      break;\r
+        CopyMem (&VendorOption->MtftpCPort, PxeOption->Data, sizeof (VendorOption->MtftpCPort));\r
+        break;\r
 \r
-    case PXEBC_VENDOR_TAG_MTFTP_SPORT:\r
+      case PXEBC_VENDOR_TAG_MTFTP_SPORT:\r
 \r
-      CopyMem (&VendorOption->MtftpSPort, PxeOption->Data, sizeof (VendorOption->MtftpSPort));\r
-      break;\r
+        CopyMem (&VendorOption->MtftpSPort, PxeOption->Data, sizeof (VendorOption->MtftpSPort));\r
+        break;\r
 \r
-    case PXEBC_VENDOR_TAG_MTFTP_TIMEOUT:\r
+      case PXEBC_VENDOR_TAG_MTFTP_TIMEOUT:\r
 \r
-      VendorOption->MtftpTimeout = *PxeOption->Data;\r
-      break;\r
+        VendorOption->MtftpTimeout = *PxeOption->Data;\r
+        break;\r
 \r
-    case PXEBC_VENDOR_TAG_MTFTP_DELAY:\r
+      case PXEBC_VENDOR_TAG_MTFTP_DELAY:\r
 \r
-      VendorOption->MtftpDelay = *PxeOption->Data;\r
-      break;\r
+        VendorOption->MtftpDelay = *PxeOption->Data;\r
+        break;\r
 \r
-    case PXEBC_VENDOR_TAG_DISCOVER_CTRL:\r
+      case PXEBC_VENDOR_TAG_DISCOVER_CTRL:\r
 \r
-      VendorOption->DiscoverCtrl = *PxeOption->Data;\r
-      break;\r
+        VendorOption->DiscoverCtrl = *PxeOption->Data;\r
+        break;\r
 \r
-    case PXEBC_VENDOR_TAG_DISCOVER_MCAST:\r
+      case PXEBC_VENDOR_TAG_DISCOVER_MCAST:\r
 \r
-      CopyMem (&VendorOption->DiscoverMcastIp, PxeOption->Data, sizeof (EFI_IPv4_ADDRESS));\r
-      break;\r
+        CopyMem (&VendorOption->DiscoverMcastIp, PxeOption->Data, sizeof (EFI_IPv4_ADDRESS));\r
+        break;\r
 \r
-    case PXEBC_VENDOR_TAG_BOOT_SERVERS:\r
+      case PXEBC_VENDOR_TAG_BOOT_SERVERS:\r
 \r
-      VendorOption->BootSvrLen  = PxeOption->Length;\r
-      VendorOption->BootSvr     = (PXEBC_BOOT_SVR_ENTRY *) PxeOption->Data;\r
-      break;\r
+        VendorOption->BootSvrLen = PxeOption->Length;\r
+        VendorOption->BootSvr    = (PXEBC_BOOT_SVR_ENTRY *)PxeOption->Data;\r
+        break;\r
 \r
-    case PXEBC_VENDOR_TAG_BOOT_MENU:\r
+      case PXEBC_VENDOR_TAG_BOOT_MENU:\r
 \r
-      VendorOption->BootMenuLen = PxeOption->Length;\r
-      VendorOption->BootMenu    = (PXEBC_BOOT_MENU_ENTRY *) PxeOption->Data;\r
-      break;\r
+        VendorOption->BootMenuLen = PxeOption->Length;\r
+        VendorOption->BootMenu    = (PXEBC_BOOT_MENU_ENTRY *)PxeOption->Data;\r
+        break;\r
 \r
-    case PXEBC_VENDOR_TAG_MENU_PROMPT:\r
+      case PXEBC_VENDOR_TAG_MENU_PROMPT:\r
 \r
-      VendorOption->MenuPromptLen = PxeOption->Length;\r
-      VendorOption->MenuPrompt    = (PXEBC_MENU_PROMPT *) PxeOption->Data;\r
-      break;\r
+        VendorOption->MenuPromptLen = PxeOption->Length;\r
+        VendorOption->MenuPrompt    = (PXEBC_MENU_PROMPT *)PxeOption->Data;\r
+        break;\r
 \r
-    case PXEBC_VENDOR_TAG_MCAST_ALLOC:\r
+      case PXEBC_VENDOR_TAG_MCAST_ALLOC:\r
 \r
-      CopyMem (&VendorOption->McastIpBase, PxeOption->Data, sizeof (EFI_IPv4_ADDRESS));\r
-      CopyMem (&VendorOption->McastIpBlock, PxeOption->Data + 4, sizeof (VendorOption->McastIpBlock));\r
-      CopyMem (&VendorOption->McastIpRange, PxeOption->Data + 6, sizeof (VendorOption->McastIpRange));\r
-      break;\r
+        CopyMem (&VendorOption->McastIpBase, PxeOption->Data, sizeof (EFI_IPv4_ADDRESS));\r
+        CopyMem (&VendorOption->McastIpBlock, PxeOption->Data + 4, sizeof (VendorOption->McastIpBlock));\r
+        CopyMem (&VendorOption->McastIpRange, PxeOption->Data + 6, sizeof (VendorOption->McastIpRange));\r
+        break;\r
 \r
-    case PXEBC_VENDOR_TAG_CREDENTIAL_TYPES:\r
+      case PXEBC_VENDOR_TAG_CREDENTIAL_TYPES:\r
 \r
-      VendorOption->CredTypeLen = PxeOption->Length;\r
-      VendorOption->CredType    = (UINT32 *) PxeOption->Data;\r
-      break;\r
+        VendorOption->CredTypeLen = PxeOption->Length;\r
+        VendorOption->CredType    = (UINT32 *)PxeOption->Data;\r
+        break;\r
 \r
-    case PXEBC_VENDOR_TAG_BOOT_ITEM:\r
+      case PXEBC_VENDOR_TAG_BOOT_ITEM:\r
 \r
-      CopyMem (&VendorOption->BootSrvType, PxeOption->Data, sizeof (VendorOption->BootSrvType));\r
-      CopyMem (&VendorOption->BootSrvLayer, PxeOption->Data + 2, sizeof (VendorOption->BootSrvLayer));\r
-      break;\r
+        CopyMem (&VendorOption->BootSrvType, PxeOption->Data, sizeof (VendorOption->BootSrvType));\r
+        CopyMem (&VendorOption->BootSrvLayer, PxeOption->Data + 2, sizeof (VendorOption->BootSrvLayer));\r
+        break;\r
 \r
-    default:\r
-      //\r
-      // Not interesting PXE vendor options.\r
-      //\r
-      break;\r
+      default:\r
+        //\r
+        // Not interesting PXE vendor options.\r
+        //\r
+        break;\r
     }\r
 \r
     //\r
@@ -198,14 +194,13 @@ PxeBcParseVendorOptions (
     if (PxeOption->OpCode == DHCP4_TAG_PAD) {\r
       Offset++;\r
     } else {\r
-      Offset = (UINT8) (Offset + PxeOption->Length + 2);\r
+      Offset = (UINT8)(Offset + PxeOption->Length + 2);\r
     }\r
 \r
-    PxeOption = (EFI_DHCP4_PACKET_OPTION *) (Dhcp4Option->Data + Offset);\r
+    PxeOption = (EFI_DHCP4_PACKET_OPTION *)(Dhcp4Option->Data + Offset);\r
   }\r
 }\r
 \r
-\r
 /**\r
   Build the options buffer for the DHCPv4 request packet.\r
 \r
@@ -226,34 +221,34 @@ PxeBcBuildDhcp4Options (
   IN  BOOLEAN                  NeedMsgType\r
   )\r
 {\r
-  UINT32                       Index;\r
-  PXEBC_DHCP4_OPTION_ENTRY     OptEnt;\r
-  UINT16                       Value;\r
+  UINT32                    Index;\r
+  PXEBC_DHCP4_OPTION_ENTRY  OptEnt;\r
+  UINT16                    Value;\r
 \r
   Index      = 0;\r
-  OptList[0] = (EFI_DHCP4_PACKET_OPTION *) Buffer;\r
+  OptList[0] = (EFI_DHCP4_PACKET_OPTION *)Buffer;\r
 \r
   if (NeedMsgType) {\r
     //\r
     // Append message type.\r
     //\r
-    OptList[Index]->OpCode  = DHCP4_TAG_MSG_TYPE;\r
-    OptList[Index]->Length  = 1;\r
-    OptEnt.Mesg             = (PXEBC_DHCP4_OPTION_MESG *) OptList[Index]->Data;\r
-    OptEnt.Mesg->Type       = PXEBC_DHCP4_MSG_TYPE_REQUEST;\r
+    OptList[Index]->OpCode = DHCP4_TAG_MSG_TYPE;\r
+    OptList[Index]->Length = 1;\r
+    OptEnt.Mesg            = (PXEBC_DHCP4_OPTION_MESG *)OptList[Index]->Data;\r
+    OptEnt.Mesg->Type      = PXEBC_DHCP4_MSG_TYPE_REQUEST;\r
     Index++;\r
-    OptList[Index]          = GET_NEXT_DHCP_OPTION (OptList[Index - 1]);\r
+    OptList[Index] = GET_NEXT_DHCP_OPTION (OptList[Index - 1]);\r
 \r
     //\r
     // Append max message size.\r
     //\r
-    OptList[Index]->OpCode  = DHCP4_TAG_MAXMSG;\r
-    OptList[Index]->Length  = (UINT8) sizeof (PXEBC_DHCP4_OPTION_MAX_MESG_SIZE);\r
-    OptEnt.MaxMesgSize      = (PXEBC_DHCP4_OPTION_MAX_MESG_SIZE *) OptList[Index]->Data;\r
-    Value                   = NTOHS (PXEBC_DHCP4_PACKET_MAX_SIZE);\r
+    OptList[Index]->OpCode = DHCP4_TAG_MAXMSG;\r
+    OptList[Index]->Length = (UINT8)sizeof (PXEBC_DHCP4_OPTION_MAX_MESG_SIZE);\r
+    OptEnt.MaxMesgSize     = (PXEBC_DHCP4_OPTION_MAX_MESG_SIZE *)OptList[Index]->Data;\r
+    Value                  = NTOHS (PXEBC_DHCP4_PACKET_MAX_SIZE);\r
     CopyMem (&OptEnt.MaxMesgSize->Size, &Value, sizeof (UINT16));\r
     Index++;\r
-    OptList[Index]          = GET_NEXT_DHCP_OPTION (OptList[Index - 1]);\r
+    OptList[Index] = GET_NEXT_DHCP_OPTION (OptList[Index - 1]);\r
   }\r
 \r
   //\r
@@ -261,7 +256,7 @@ PxeBcBuildDhcp4Options (
   //\r
   OptList[Index]->OpCode    = DHCP4_TAG_PARA_LIST;\r
   OptList[Index]->Length    = 35;\r
-  OptEnt.Para               = (PXEBC_DHCP4_OPTION_PARA *) OptList[Index]->Data;\r
+  OptEnt.Para               = (PXEBC_DHCP4_OPTION_PARA *)OptList[Index]->Data;\r
   OptEnt.Para->ParaList[0]  = DHCP4_TAG_NETMASK;\r
   OptEnt.Para->ParaList[1]  = DHCP4_TAG_TIME_OFFSET;\r
   OptEnt.Para->ParaList[2]  = DHCP4_TAG_ROUTER;\r
@@ -298,19 +293,19 @@ PxeBcBuildDhcp4Options (
   OptEnt.Para->ParaList[33] = 0x86;\r
   OptEnt.Para->ParaList[34] = 0x87;\r
   Index++;\r
-  OptList[Index]            = GET_NEXT_DHCP_OPTION (OptList[Index - 1]);\r
+  OptList[Index] = GET_NEXT_DHCP_OPTION (OptList[Index - 1]);\r
 \r
   //\r
   // Append UUID/Guid-based client identifier option\r
   //\r
-  OptList[Index]->OpCode  = DHCP4_TAG_UUID;\r
-  OptList[Index]->Length  = (UINT8) sizeof (PXEBC_DHCP4_OPTION_UUID);\r
-  OptEnt.Uuid             = (PXEBC_DHCP4_OPTION_UUID *) OptList[Index]->Data;\r
-  OptEnt.Uuid->Type       = 0;\r
+  OptList[Index]->OpCode = DHCP4_TAG_UUID;\r
+  OptList[Index]->Length = (UINT8)sizeof (PXEBC_DHCP4_OPTION_UUID);\r
+  OptEnt.Uuid            = (PXEBC_DHCP4_OPTION_UUID *)OptList[Index]->Data;\r
+  OptEnt.Uuid->Type      = 0;\r
   Index++;\r
-  OptList[Index]          = GET_NEXT_DHCP_OPTION (OptList[Index - 1]);\r
+  OptList[Index] = GET_NEXT_DHCP_OPTION (OptList[Index - 1]);\r
 \r
-  if (EFI_ERROR (NetLibGetSystemGuid ((EFI_GUID *) OptEnt.Uuid->Guid))) {\r
+  if (EFI_ERROR (NetLibGetSystemGuid ((EFI_GUID *)OptEnt.Uuid->Guid))) {\r
     //\r
     // Zero the Guid to indicate NOT programmable if failed to get system Guid.\r
     //\r
@@ -321,9 +316,9 @@ PxeBcBuildDhcp4Options (
   //\r
   // Append client network device interface option\r
   //\r
-  OptList[Index]->OpCode  = DHCP4_TAG_UNDI;\r
-  OptList[Index]->Length  = (UINT8) sizeof (PXEBC_DHCP4_OPTION_UNDI);\r
-  OptEnt.Undi             = (PXEBC_DHCP4_OPTION_UNDI *) OptList[Index]->Data;\r
+  OptList[Index]->OpCode = DHCP4_TAG_UNDI;\r
+  OptList[Index]->Length = (UINT8)sizeof (PXEBC_DHCP4_OPTION_UNDI);\r
+  OptEnt.Undi            = (PXEBC_DHCP4_OPTION_UNDI *)OptList[Index]->Data;\r
 \r
   if (Private->Nii != NULL) {\r
     OptEnt.Undi->Type     = Private->Nii->Type;\r
@@ -341,20 +336,20 @@ PxeBcBuildDhcp4Options (
   //\r
   // Append client system architecture option\r
   //\r
-  OptList[Index]->OpCode  = DHCP4_TAG_ARCH;\r
-  OptList[Index]->Length  = (UINT8) sizeof (PXEBC_DHCP4_OPTION_ARCH);\r
-  OptEnt.Arch             = (PXEBC_DHCP4_OPTION_ARCH *) OptList[Index]->Data;\r
-  Value                   = HTONS (EFI_PXE_CLIENT_SYSTEM_ARCHITECTURE);\r
+  OptList[Index]->OpCode = DHCP4_TAG_ARCH;\r
+  OptList[Index]->Length = (UINT8)sizeof (PXEBC_DHCP4_OPTION_ARCH);\r
+  OptEnt.Arch            = (PXEBC_DHCP4_OPTION_ARCH *)OptList[Index]->Data;\r
+  Value                  = HTONS (EFI_PXE_CLIENT_SYSTEM_ARCHITECTURE);\r
   CopyMem (&OptEnt.Arch->Type, &Value, sizeof (UINT16));\r
   Index++;\r
-  OptList[Index]          = GET_NEXT_DHCP_OPTION (OptList[Index - 1]);\r
+  OptList[Index] = GET_NEXT_DHCP_OPTION (OptList[Index - 1]);\r
 \r
   //\r
   // Append vendor class identify option\r
   //\r
-  OptList[Index]->OpCode  = DHCP4_TAG_VENDOR_CLASS_ID;\r
-  OptList[Index]->Length  = (UINT8) sizeof (PXEBC_DHCP4_OPTION_CLID);\r
-  OptEnt.Clid             = (PXEBC_DHCP4_OPTION_CLID *) OptList[Index]->Data;\r
+  OptList[Index]->OpCode = DHCP4_TAG_VENDOR_CLASS_ID;\r
+  OptList[Index]->Length = (UINT8)sizeof (PXEBC_DHCP4_OPTION_CLID);\r
+  OptEnt.Clid            = (PXEBC_DHCP4_OPTION_CLID *)OptList[Index]->Data;\r
   CopyMem (\r
     OptEnt.Clid,\r
     DEFAULT_CLASS_ID_DATA,\r
@@ -377,7 +372,6 @@ PxeBcBuildDhcp4Options (
   return Index;\r
 }\r
 \r
-\r
 /**\r
   Create a template DHCPv4 packet as a seed.\r
 \r
@@ -387,32 +381,31 @@ PxeBcBuildDhcp4Options (
 **/\r
 VOID\r
 PxeBcSeedDhcp4Packet (\r
-  OUT EFI_DHCP4_PACKET       *Seed,\r
-  IN  EFI_UDP4_PROTOCOL      *Udp4\r
+  OUT EFI_DHCP4_PACKET   *Seed,\r
+  IN  EFI_UDP4_PROTOCOL  *Udp4\r
   )\r
 {\r
-  EFI_SIMPLE_NETWORK_MODE    Mode;\r
-  EFI_DHCP4_HEADER           *Header;\r
+  EFI_SIMPLE_NETWORK_MODE  Mode;\r
+  EFI_DHCP4_HEADER         *Header;\r
 \r
   //\r
   // Get IfType and HwAddressSize from SNP mode data.\r
   //\r
   Udp4->GetModeData (Udp4, NULL, NULL, NULL, &Mode);\r
 \r
-  Seed->Size            = sizeof (EFI_DHCP4_PACKET);\r
-  Seed->Length          = sizeof (Seed->Dhcp4);\r
-  Header                = &Seed->Dhcp4.Header;\r
+  Seed->Size   = sizeof (EFI_DHCP4_PACKET);\r
+  Seed->Length = sizeof (Seed->Dhcp4);\r
+  Header       = &Seed->Dhcp4.Header;\r
   ZeroMem (Header, sizeof (EFI_DHCP4_HEADER));\r
-  Header->OpCode        = PXEBC_DHCP4_OPCODE_REQUEST;\r
-  Header->HwType        = Mode.IfType;\r
-  Header->HwAddrLen     = (UINT8) Mode.HwAddressSize;\r
+  Header->OpCode    = PXEBC_DHCP4_OPCODE_REQUEST;\r
+  Header->HwType    = Mode.IfType;\r
+  Header->HwAddrLen = (UINT8)Mode.HwAddressSize;\r
   CopyMem (Header->ClientHwAddr, &Mode.CurrentAddress, Header->HwAddrLen);\r
 \r
   Seed->Dhcp4.Magik     = PXEBC_DHCP4_MAGIC;\r
   Seed->Dhcp4.Option[0] = DHCP4_TAG_EOP;\r
 }\r
 \r
-\r
 /**\r
   Cache the DHCPv4 packet.\r
 \r
@@ -425,8 +418,8 @@ PxeBcSeedDhcp4Packet (
 **/\r
 EFI_STATUS\r
 PxeBcCacheDhcp4Packet (\r
-  IN EFI_DHCP4_PACKET     *Dst,\r
-  IN EFI_DHCP4_PACKET     *Src\r
+  IN EFI_DHCP4_PACKET  *Dst,\r
+  IN EFI_DHCP4_PACKET  *Src\r
   )\r
 {\r
   if (Dst->Size < Src->Length) {\r
@@ -439,7 +432,6 @@ PxeBcCacheDhcp4Packet (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Parse the cached DHCPv4 packet, including all the options.\r
 \r
@@ -451,21 +443,21 @@ PxeBcCacheDhcp4Packet (
 **/\r
 EFI_STATUS\r
 PxeBcParseDhcp4Packet (\r
-  IN PXEBC_DHCP4_PACKET_CACHE    *Cache4\r
+  IN PXEBC_DHCP4_PACKET_CACHE  *Cache4\r
   )\r
 {\r
-  EFI_DHCP4_PACKET               *Offer;\r
-  EFI_DHCP4_PACKET_OPTION        **Options;\r
-  EFI_DHCP4_PACKET_OPTION        *Option;\r
-  PXEBC_OFFER_TYPE               OfferType;\r
-  UINTN                          Index;\r
-  BOOLEAN                        IsProxyOffer;\r
-  BOOLEAN                        IsPxeOffer;\r
-  UINT8                          *Ptr8;\r
-  BOOLEAN                        FileFieldOverloaded;\r
-\r
-  IsProxyOffer = FALSE;\r
-  IsPxeOffer   = FALSE;\r
+  EFI_DHCP4_PACKET         *Offer;\r
+  EFI_DHCP4_PACKET_OPTION  **Options;\r
+  EFI_DHCP4_PACKET_OPTION  *Option;\r
+  PXEBC_OFFER_TYPE         OfferType;\r
+  UINTN                    Index;\r
+  BOOLEAN                  IsProxyOffer;\r
+  BOOLEAN                  IsPxeOffer;\r
+  UINT8                    *Ptr8;\r
+  BOOLEAN                  FileFieldOverloaded;\r
+\r
+  IsProxyOffer        = FALSE;\r
+  IsPxeOffer          = FALSE;\r
   FileFieldOverloaded = FALSE;\r
 \r
   ZeroMem (Cache4->OptList, sizeof (Cache4->OptList));\r
@@ -485,6 +477,7 @@ PxeBcParseDhcp4Packet (
                        mInterestedDhcp4Tags[Index]\r
                        );\r
   }\r
+\r
   //\r
   // Second, Check if bootfilename and serverhostname is overloaded to carry DHCP options refers to rfc-2132.\r
   // If yes, try to parse options from the BootFileName field, then ServerName field.\r
@@ -496,18 +489,19 @@ PxeBcParseDhcp4Packet (
       for (Index = 0; Index < PXEBC_DHCP4_TAG_INDEX_MAX; Index++) {\r
         if (Options[Index] == NULL) {\r
           Options[Index] = PxeBcParseDhcp4Options (\r
-                             (UINT8 *) Offer->Dhcp4.Header.BootFileName,\r
+                             (UINT8 *)Offer->Dhcp4.Header.BootFileName,\r
                              sizeof (Offer->Dhcp4.Header.BootFileName),\r
                              mInterestedDhcp4Tags[Index]\r
                              );\r
         }\r
       }\r
     }\r
+\r
     if ((Option->Data[0] & PXEBC_DHCP4_OVERLOAD_SERVER_NAME) != 0) {\r
       for (Index = 0; Index < PXEBC_DHCP4_TAG_INDEX_MAX; Index++) {\r
         if (Options[Index] == NULL) {\r
           Options[Index] = PxeBcParseDhcp4Options (\r
-                             (UINT8 *) Offer->Dhcp4.Header.ServerName,\r
+                             (UINT8 *)Offer->Dhcp4.Header.ServerName,\r
                              sizeof (Offer->Dhcp4.Header.ServerName),\r
                              mInterestedDhcp4Tags[Index]\r
                              );\r
@@ -528,7 +522,8 @@ PxeBcParseDhcp4Packet (
   //\r
   Option = Options[PXEBC_DHCP4_TAG_INDEX_CLASS_ID];\r
   if ((Option != NULL) && (Option->Length >= 9) &&\r
-      (CompareMem (Option->Data, DEFAULT_CLASS_ID_DATA, 9) == 0)) {\r
+      (CompareMem (Option->Data, DEFAULT_CLASS_ID_DATA, 9) == 0))\r
+  {\r
     IsPxeOffer = TRUE;\r
   }\r
 \r
@@ -536,7 +531,7 @@ PxeBcParseDhcp4Packet (
   // Parse PXE vendor options in this offer, and store the contents/pointers.\r
   //\r
   Option = Options[PXEBC_DHCP4_TAG_INDEX_VENDOR];\r
-  if (IsPxeOffer && Option != NULL) {\r
+  if (IsPxeOffer && (Option != NULL)) {\r
     PxeBcParseVendorOptions (Option, &Cache4->VendorOpt);\r
   }\r
 \r
@@ -550,33 +545,32 @@ PxeBcParseDhcp4Packet (
     // RFC 2132, Section 9.5 does not strictly state Bootfile name (option 67) is null\r
     // terminated string. So force to append null terminated character at the end of string.\r
     //\r
-    Ptr8 =  (UINT8*)&Options[PXEBC_DHCP4_TAG_INDEX_BOOTFILE]->Data[0];\r
+    Ptr8  =  (UINT8 *)&Options[PXEBC_DHCP4_TAG_INDEX_BOOTFILE]->Data[0];\r
     Ptr8 += Options[PXEBC_DHCP4_TAG_INDEX_BOOTFILE]->Length;\r
     if (*(Ptr8 - 1) != '\0') {\r
       *Ptr8 = '\0';\r
     }\r
-  } else if (!FileFieldOverloaded && Offer->Dhcp4.Header.BootFileName[0] != 0) {\r
+  } else if (!FileFieldOverloaded && (Offer->Dhcp4.Header.BootFileName[0] != 0)) {\r
     //\r
     // If the bootfile is not present and bootfilename is present in DHCPv4 packet, just parse it.\r
     // Do not count dhcp option header here, or else will destroy the serverhostname.\r
     //\r
     Options[PXEBC_DHCP4_TAG_INDEX_BOOTFILE] = (EFI_DHCP4_PACKET_OPTION *)\r
-                                                (&Offer->Dhcp4.Header.BootFileName[0] -\r
-                                                OFFSET_OF (EFI_DHCP4_PACKET_OPTION, Data[0]));\r
-\r
+                                              (&Offer->Dhcp4.Header.BootFileName[0] -\r
+                                               OFFSET_OF (EFI_DHCP4_PACKET_OPTION, Data[0]));\r
   }\r
 \r
   //\r
   // Determine offer type of the DHCPv4 packet.\r
   //\r
   Option = Options[PXEBC_DHCP4_TAG_INDEX_MSG_TYPE];\r
-  if (Option == NULL || Option->Data[0] == 0) {\r
+  if ((Option == NULL) || (Option->Data[0] == 0)) {\r
     //\r
     // It's a Bootp offer.\r
     //\r
     OfferType = PxeOfferTypeBootp;\r
 \r
-    Option    = Cache4->OptList[PXEBC_DHCP4_TAG_INDEX_BOOTFILE];\r
+    Option = Cache4->OptList[PXEBC_DHCP4_TAG_INDEX_BOOTFILE];\r
     if (Option == NULL) {\r
       //\r
       // If the Bootp offer without bootfilename, discard it.\r
@@ -584,7 +578,6 @@ PxeBcParseDhcp4Packet (
       return EFI_DEVICE_ERROR;\r
     }\r
   } else {\r
-\r
     if (IS_VALID_DISCOVER_VENDOR_OPTION (Cache4->VendorOpt.BitMap)) {\r
       //\r
       // It's a PXE10 offer with PXEClient and discover vendor option.\r
@@ -614,7 +607,6 @@ PxeBcParseDhcp4Packet (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Cache the DHCPv4 ack packet, and parse it on demand.\r
 \r
@@ -628,9 +620,9 @@ PxeBcParseDhcp4Packet (
 **/\r
 EFI_STATUS\r
 PxeBcCopyDhcp4Ack (\r
-  IN PXEBC_PRIVATE_DATA   *Private,\r
-  IN EFI_DHCP4_PACKET     *Ack,\r
-  IN BOOLEAN              Verified\r
+  IN PXEBC_PRIVATE_DATA  *Private,\r
+  IN EFI_DHCP4_PACKET    *Ack,\r
+  IN BOOLEAN             Verified\r
   )\r
 {\r
   EFI_PXE_BASE_CODE_MODE  *Mode;\r
@@ -655,7 +647,6 @@ PxeBcCopyDhcp4Ack (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Cache the DHCPv4 proxy offer packet according to the received order.\r
 \r
@@ -668,8 +659,8 @@ PxeBcCopyDhcp4Ack (
 **/\r
 EFI_STATUS\r
 PxeBcCopyProxyOffer (\r
-  IN PXEBC_PRIVATE_DATA   *Private,\r
-  IN UINT32               OfferIndex\r
+  IN PXEBC_PRIVATE_DATA  *Private,\r
+  IN UINT32              OfferIndex\r
   )\r
 {\r
   EFI_PXE_BASE_CODE_MODE  *Mode;\r
@@ -686,7 +677,7 @@ PxeBcCopyProxyOffer (
   // Cache the proxy offer packet and parse it.\r
   //\r
   Status = PxeBcCacheDhcp4Packet (&Private->ProxyOffer.Dhcp4.Packet.Offer, Offer);\r
-  if (EFI_ERROR(Status)) {\r
+  if (EFI_ERROR (Status)) {\r
     return Status;\r
   }\r
 \r
@@ -701,7 +692,6 @@ PxeBcCopyProxyOffer (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Retry to request bootfile name by the BINL offer.\r
 \r
@@ -714,8 +704,8 @@ PxeBcCopyProxyOffer (
 **/\r
 EFI_STATUS\r
 PxeBcRetryBinlOffer (\r
-  IN PXEBC_PRIVATE_DATA     *Private,\r
-  IN UINT32                 Index\r
+  IN PXEBC_PRIVATE_DATA  *Private,\r
+  IN UINT32              Index\r
   )\r
 {\r
   EFI_DHCP4_PACKET          *Offer;\r
@@ -725,8 +715,10 @@ PxeBcRetryBinlOffer (
   EFI_DHCP4_PACKET          *Reply;\r
 \r
   ASSERT (Index < PXEBC_OFFER_MAX_NUM);\r
-  ASSERT (Private->OfferBuffer[Index].Dhcp4.OfferType == PxeOfferTypeDhcpBinl ||\r
-          Private->OfferBuffer[Index].Dhcp4.OfferType == PxeOfferTypeProxyBinl);\r
+  ASSERT (\r
+    Private->OfferBuffer[Index].Dhcp4.OfferType == PxeOfferTypeDhcpBinl ||\r
+    Private->OfferBuffer[Index].Dhcp4.OfferType == PxeOfferTypeProxyBinl\r
+    );\r
 \r
   Offer = &Private->OfferBuffer[Index].Dhcp4.Packet.Offer;\r
 \r
@@ -775,9 +767,10 @@ PxeBcRetryBinlOffer (
     return Status;\r
   }\r
 \r
-  if (Cache4->OfferType != PxeOfferTypeProxyPxe10 &&\r
-      Cache4->OfferType != PxeOfferTypeProxyWfm11a &&\r
-      Cache4->OptList[PXEBC_DHCP4_TAG_INDEX_BOOTFILE] == NULL) {\r
+  if ((Cache4->OfferType != PxeOfferTypeProxyPxe10) &&\r
+      (Cache4->OfferType != PxeOfferTypeProxyWfm11a) &&\r
+      (Cache4->OptList[PXEBC_DHCP4_TAG_INDEX_BOOTFILE] == NULL))\r
+  {\r
     //\r
     // This BINL ack doesn't have discovery option set or multicast option set\r
     // or bootfile name specified.\r
@@ -794,7 +787,6 @@ PxeBcRetryBinlOffer (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Cache all the received DHCPv4 offers, and set OfferIndex and OfferCount.\r
 \r
@@ -807,8 +799,8 @@ PxeBcRetryBinlOffer (
 **/\r
 EFI_STATUS\r
 PxeBcCacheDhcp4Offer (\r
-  IN PXEBC_PRIVATE_DATA     *Private,\r
-  IN EFI_DHCP4_PACKET       *RcvdOffer\r
+  IN PXEBC_PRIVATE_DATA  *Private,\r
+  IN EFI_DHCP4_PACKET    *RcvdOffer\r
   )\r
 {\r
   PXEBC_DHCP4_PACKET_CACHE  *Cache4;\r
@@ -824,7 +816,7 @@ PxeBcCacheDhcp4Offer (
   // Cache the content of DHCPv4 packet firstly.\r
   //\r
   Status = PxeBcCacheDhcp4Packet (Offer, RcvdOffer);\r
-  if (EFI_ERROR(Status)) {\r
+  if (EFI_ERROR (Status)) {\r
     return Status;\r
   }\r
 \r
@@ -865,8 +857,9 @@ PxeBcCacheDhcp4Offer (
         //\r
         Private->OfferIndex[OfferType][Private->OfferCount[OfferType]] = Private->OfferNum;\r
         Private->OfferCount[OfferType]++;\r
-      } else if ((OfferType == PxeOfferTypeProxyPxe10 || OfferType == PxeOfferTypeProxyWfm11a) &&\r
-                 Private->OfferCount[OfferType] < 1) {\r
+      } else if (((OfferType == PxeOfferTypeProxyPxe10) || (OfferType == PxeOfferTypeProxyWfm11a)) &&\r
+                 (Private->OfferCount[OfferType] < 1))\r
+      {\r
         //\r
         // Only cache the first PXE10/WFM11a offer, and discard the others.\r
         //\r
@@ -889,7 +882,6 @@ PxeBcCacheDhcp4Offer (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Select an DHCPv4 offer, and record SelectIndex and SelectProxyType.\r
 \r
@@ -898,12 +890,12 @@ PxeBcCacheDhcp4Offer (
 **/\r
 VOID\r
 PxeBcSelectDhcp4Offer (\r
-  IN PXEBC_PRIVATE_DATA       *Private\r
+  IN PXEBC_PRIVATE_DATA  *Private\r
   )\r
 {\r
-  UINT32                      Index;\r
-  UINT32                      OfferIndex;\r
-  EFI_DHCP4_PACKET            *Offer;\r
+  UINT32            Index;\r
+  UINT32            OfferIndex;\r
+  EFI_DHCP4_PACKET  *Offer;\r
 \r
   Private->SelectIndex = 0;\r
 \r
@@ -916,43 +908,40 @@ PxeBcSelectDhcp4Offer (
       // 1. DhcpPxe10 offer\r
       //\r
       Private->SelectIndex = Private->OfferIndex[PxeOfferTypeDhcpPxe10][0] + 1;\r
-\r
     } else if (Private->OfferCount[PxeOfferTypeDhcpWfm11a] > 0) {\r
       //\r
       // 2. DhcpWfm11a offer\r
       //\r
       Private->SelectIndex = Private->OfferIndex[PxeOfferTypeDhcpWfm11a][0] + 1;\r
-\r
-    } else if (Private->OfferCount[PxeOfferTypeDhcpOnly] > 0 &&\r
-               Private->OfferCount[PxeOfferTypeProxyPxe10] > 0) {\r
+    } else if ((Private->OfferCount[PxeOfferTypeDhcpOnly] > 0) &&\r
+               (Private->OfferCount[PxeOfferTypeProxyPxe10] > 0))\r
+    {\r
       //\r
       // 3. DhcpOnly offer and ProxyPxe10 offer.\r
       //\r
       Private->SelectIndex     = Private->OfferIndex[PxeOfferTypeDhcpOnly][0] + 1;\r
       Private->SelectProxyType = PxeOfferTypeProxyPxe10;\r
-\r
-    } else if (Private->OfferCount[PxeOfferTypeDhcpOnly] > 0 &&\r
-               Private->OfferCount[PxeOfferTypeProxyWfm11a] > 0) {\r
+    } else if ((Private->OfferCount[PxeOfferTypeDhcpOnly] > 0) &&\r
+               (Private->OfferCount[PxeOfferTypeProxyWfm11a] > 0))\r
+    {\r
       //\r
       // 4. DhcpOnly offer and ProxyWfm11a offer.\r
       //\r
       Private->SelectIndex     = Private->OfferIndex[PxeOfferTypeDhcpOnly][0] + 1;\r
       Private->SelectProxyType = PxeOfferTypeProxyWfm11a;\r
-\r
     } else if (Private->OfferCount[PxeOfferTypeDhcpBinl] > 0) {\r
       //\r
       // 5. DhcpBinl offer.\r
       //\r
       Private->SelectIndex = Private->OfferIndex[PxeOfferTypeDhcpBinl][0] + 1;\r
-\r
-    } else if (Private->OfferCount[PxeOfferTypeDhcpOnly] > 0 &&\r
-               Private->OfferCount[PxeOfferTypeProxyBinl] > 0) {\r
+    } else if ((Private->OfferCount[PxeOfferTypeDhcpOnly] > 0) &&\r
+               (Private->OfferCount[PxeOfferTypeProxyBinl] > 0))\r
+    {\r
       //\r
       // 6. DhcpOnly offer and ProxyBinl offer.\r
       //\r
       Private->SelectIndex     = Private->OfferIndex[PxeOfferTypeDhcpOnly][0] + 1;\r
       Private->SelectProxyType = PxeOfferTypeProxyBinl;\r
-\r
     } else {\r
       //\r
       // 7. DhcpOnly offer with bootfilename.\r
@@ -964,13 +953,15 @@ PxeBcSelectDhcp4Offer (
           break;\r
         }\r
       }\r
+\r
       //\r
       // 8. Bootp offer with bootfilename.\r
       //\r
       OfferIndex = Private->OfferIndex[PxeOfferTypeBootp][0];\r
-      if (Private->SelectIndex == 0 &&\r
-          Private->OfferCount[PxeOfferTypeBootp] > 0 &&\r
-          Private->OfferBuffer[OfferIndex].Dhcp4.OptList[PXEBC_DHCP4_TAG_INDEX_BOOTFILE] != NULL) {\r
+      if ((Private->SelectIndex == 0) &&\r
+          (Private->OfferCount[PxeOfferTypeBootp] > 0) &&\r
+          (Private->OfferBuffer[OfferIndex].Dhcp4.OptList[PXEBC_DHCP4_TAG_INDEX_BOOTFILE] != NULL))\r
+      {\r
         Private->SelectIndex = OfferIndex + 1;\r
       }\r
     }\r
@@ -979,7 +970,6 @@ PxeBcSelectDhcp4Offer (
     // Select offer by received order.\r
     //\r
     for (Index = 0; Index < Private->OfferNum; Index++) {\r
-\r
       Offer = &Private->OfferBuffer[Index].Dhcp4.Packet.Offer;\r
 \r
       if (IS_PROXY_DHCP_OFFER (Offer)) {\r
@@ -990,8 +980,9 @@ PxeBcSelectDhcp4Offer (
       }\r
 \r
       if (!Private->IsProxyRecved &&\r
-          Private->OfferBuffer[Index].Dhcp4.OfferType == PxeOfferTypeDhcpOnly &&\r
-          Private->OfferBuffer[Index].Dhcp4.OptList[PXEBC_DHCP4_TAG_INDEX_BOOTFILE] == NULL) {\r
+          (Private->OfferBuffer[Index].Dhcp4.OfferType == PxeOfferTypeDhcpOnly) &&\r
+          (Private->OfferBuffer[Index].Dhcp4.OptList[PXEBC_DHCP4_TAG_INDEX_BOOTFILE] == NULL))\r
+      {\r
         //\r
         // Skip if DhcpOnly offer without any other proxy offers or bootfilename.\r
         //\r
@@ -1007,7 +998,6 @@ PxeBcSelectDhcp4Offer (
   }\r
 }\r
 \r
-\r
 /**\r
   Handle the DHCPv4 offer packet.\r
 \r
@@ -1021,7 +1011,7 @@ PxeBcSelectDhcp4Offer (
 **/\r
 EFI_STATUS\r
 PxeBcHandleDhcp4Offer (\r
-  IN PXEBC_PRIVATE_DATA     *Private\r
+  IN PXEBC_PRIVATE_DATA  *Private\r
   )\r
 {\r
   PXEBC_DHCP4_PACKET_CACHE  *Cache4;\r
@@ -1036,11 +1026,11 @@ PxeBcHandleDhcp4Offer (
   EFI_DHCP4_PACKET          *Ack;\r
 \r
   ASSERT (Private->SelectIndex > 0);\r
-  SelectIndex = (UINT32) (Private->SelectIndex - 1);\r
+  SelectIndex = (UINT32)(Private->SelectIndex - 1);\r
   ASSERT (SelectIndex < PXEBC_OFFER_MAX_NUM);\r
-  Cache4      = &Private->OfferBuffer[SelectIndex].Dhcp4;\r
-  Options     = Cache4->OptList;\r
-  Status      = EFI_SUCCESS;\r
+  Cache4  = &Private->OfferBuffer[SelectIndex].Dhcp4;\r
+  Options = Cache4->OptList;\r
+  Status  = EFI_SUCCESS;\r
 \r
   if (Cache4->OfferType == PxeOfferTypeDhcpBinl) {\r
     //\r
@@ -1050,7 +1040,6 @@ PxeBcHandleDhcp4Offer (
       Status = EFI_NO_RESPONSE;\r
     }\r
   } else if (Cache4->OfferType == PxeOfferTypeDhcpOnly) {\r
-\r
     if (Private->IsProxyRecved) {\r
       //\r
       // DhcpOnly offer is selected, so need try to request bootfile name.\r
@@ -1075,6 +1064,7 @@ PxeBcHandleDhcp4Offer (
               break;\r
             }\r
           }\r
+\r
           if (Index == Private->OfferCount[Private->SelectProxyType]) {\r
             Status = EFI_NO_RESPONSE;\r
           }\r
@@ -1117,7 +1107,7 @@ PxeBcHandleDhcp4Offer (
         }\r
       }\r
 \r
-      if (!EFI_ERROR (Status) && Private->SelectProxyType != PxeOfferTypeProxyBinl) {\r
+      if (!EFI_ERROR (Status) && (Private->SelectProxyType != PxeOfferTypeProxyBinl)) {\r
         //\r
         // Success to try to request by a ProxyPxe10 or ProxyWfm11a offer, copy and parse it.\r
         //\r
@@ -1152,13 +1142,13 @@ PxeBcHandleDhcp4Offer (
     if (EFI_ERROR (Status)) {\r
       return Status;\r
     }\r
+\r
     Mode->DhcpDiscoverValid = TRUE;\r
   }\r
 \r
   return Status;\r
 }\r
 \r
-\r
 /**\r
   EFI_DHCP4_CALLBACK is provided by the consumer of the EFI DHCPv4 Protocol driver\r
   to intercept events that occurred in the configuration process.\r
@@ -1182,12 +1172,12 @@ PxeBcHandleDhcp4Offer (
 EFI_STATUS\r
 EFIAPI\r
 PxeBcDhcp4CallBack (\r
-  IN  EFI_DHCP4_PROTOCOL               *This,\r
-  IN  VOID                             *Context,\r
-  IN  EFI_DHCP4_STATE                  CurrentState,\r
-  IN  EFI_DHCP4_EVENT                  Dhcp4Event,\r
-  IN  EFI_DHCP4_PACKET                 *Packet            OPTIONAL,\r
-  OUT EFI_DHCP4_PACKET                 **NewPacket        OPTIONAL\r
+  IN  EFI_DHCP4_PROTOCOL  *This,\r
+  IN  VOID                *Context,\r
+  IN  EFI_DHCP4_STATE     CurrentState,\r
+  IN  EFI_DHCP4_EVENT     Dhcp4Event,\r
+  IN  EFI_DHCP4_PACKET    *Packet            OPTIONAL,\r
+  OUT EFI_DHCP4_PACKET    **NewPacket        OPTIONAL\r
   )\r
 {\r
   PXEBC_PRIVATE_DATA                   *Private;\r
@@ -1201,15 +1191,16 @@ PxeBcDhcp4CallBack (
   if ((Dhcp4Event != Dhcp4RcvdOffer) &&\r
       (Dhcp4Event != Dhcp4SelectOffer) &&\r
       (Dhcp4Event != Dhcp4SendDiscover) &&\r
-      (Dhcp4Event != Dhcp4RcvdAck)) {\r
+      (Dhcp4Event != Dhcp4RcvdAck))\r
+  {\r
     return EFI_SUCCESS;\r
   }\r
 \r
   ASSERT (Packet != NULL);\r
 \r
-  Private   = (PXEBC_PRIVATE_DATA *) Context;\r
-  Mode      = Private->PxeBc.Mode;\r
-  Callback  = Private->PxeBcCallback;\r
+  Private  = (PXEBC_PRIVATE_DATA *)Context;\r
+  Mode     = Private->PxeBc.Mode;\r
+  Callback = Private->PxeBcCallback;\r
 \r
   //\r
   // Override the Maximum DHCP Message Size.\r
@@ -1227,15 +1218,15 @@ PxeBcDhcp4CallBack (
   //\r
   // Callback to user if any packets sent or received.\r
   //\r
-  if (Dhcp4Event != Dhcp4SelectOffer && Callback != NULL) {\r
-    Received = (BOOLEAN) (Dhcp4Event == Dhcp4RcvdOffer || Dhcp4Event == Dhcp4RcvdAck);\r
-    Status = Callback->Callback (\r
-                         Callback,\r
-                         Private->Function,\r
-                         Received,\r
-                         Packet->Length,\r
-                         (EFI_PXE_BASE_CODE_PACKET *) &Packet->Dhcp4\r
-                         );\r
+  if ((Dhcp4Event != Dhcp4SelectOffer) && (Callback != NULL)) {\r
+    Received = (BOOLEAN)(Dhcp4Event == Dhcp4RcvdOffer || Dhcp4Event == Dhcp4RcvdAck);\r
+    Status   = Callback->Callback (\r
+                           Callback,\r
+                           Private->Function,\r
+                           Received,\r
+                           Packet->Length,\r
+                           (EFI_PXE_BASE_CODE_PACKET *)&Packet->Dhcp4\r
+                           );\r
     if (Status != EFI_PXE_BASE_CODE_CALLBACK_STATUS_CONTINUE) {\r
       return EFI_ABORTED;\r
     }\r
@@ -1244,101 +1235,105 @@ PxeBcDhcp4CallBack (
   Status = EFI_SUCCESS;\r
 \r
   switch (Dhcp4Event) {\r
+    case Dhcp4SendDiscover:\r
+      if (Packet->Length > PXEBC_DHCP4_PACKET_MAX_SIZE) {\r
+        //\r
+        // If the to be sent packet exceeds the maximum length, abort the DHCP process.\r
+        //\r
+        Status = EFI_ABORTED;\r
+        break;\r
+      }\r
 \r
-  case Dhcp4SendDiscover:\r
-    if (Packet->Length > PXEBC_DHCP4_PACKET_MAX_SIZE) {\r
       //\r
-      // If the to be sent packet exceeds the maximum length, abort the DHCP process.\r
+      // Cache the DHCPv4 discover packet to mode data directly.\r
+      // It need to check SendGuid as well as Dhcp4SendRequest.\r
       //\r
-      Status = EFI_ABORTED;\r
-      break;\r
-    }\r
+      CopyMem (&Mode->DhcpDiscover.Dhcpv4, &Packet->Dhcp4, Packet->Length);\r
 \r
-    //\r
-    // Cache the DHCPv4 discover packet to mode data directly.\r
-    // It need to check SendGuid as well as Dhcp4SendRequest.\r
-    //\r
-    CopyMem (&Mode->DhcpDiscover.Dhcpv4, &Packet->Dhcp4, Packet->Length);\r
+    case Dhcp4SendRequest:\r
+      if (Packet->Length > PXEBC_DHCP4_PACKET_MAX_SIZE) {\r
+        //\r
+        // If the to be sent packet exceeds the maximum length, abort the DHCP process.\r
+        //\r
+        Status = EFI_ABORTED;\r
+        break;\r
+      }\r
+\r
+      if (Mode->SendGUID) {\r
+        //\r
+        // Send the system Guid instead of the MAC address as the hardware address if required.\r
+        //\r
+        if (EFI_ERROR (NetLibGetSystemGuid ((EFI_GUID *)Packet->Dhcp4.Header.ClientHwAddr))) {\r
+          //\r
+          // Zero the Guid to indicate NOT programmable if failed to get system Guid.\r
+          //\r
+          DEBUG ((DEBUG_WARN, "PXE: Failed to read system GUID from the smbios table!\n"));\r
+          ZeroMem (Packet->Dhcp4.Header.ClientHwAddr, sizeof (EFI_GUID));\r
+        }\r
+\r
+        Packet->Dhcp4.Header.HwAddrLen = (UINT8)sizeof (EFI_GUID);\r
+      }\r
 \r
-  case Dhcp4SendRequest:\r
-    if (Packet->Length > PXEBC_DHCP4_PACKET_MAX_SIZE) {\r
-      //\r
-      // If the to be sent packet exceeds the maximum length, abort the DHCP process.\r
-      //\r
-      Status = EFI_ABORTED;\r
       break;\r
-    }\r
 \r
-    if (Mode->SendGUID) {\r
-      //\r
-      // Send the system Guid instead of the MAC address as the hardware address if required.\r
-      //\r
-      if (EFI_ERROR (NetLibGetSystemGuid ((EFI_GUID *) Packet->Dhcp4.Header.ClientHwAddr))) {\r
+    case Dhcp4RcvdOffer:\r
+      Status = EFI_NOT_READY;\r
+      if (Packet->Length > PXEBC_DHCP4_PACKET_MAX_SIZE) {\r
         //\r
-        // Zero the Guid to indicate NOT programmable if failed to get system Guid.\r
+        // Ignore the incoming packets which exceed the maximum length.\r
         //\r
-        DEBUG ((DEBUG_WARN, "PXE: Failed to read system GUID from the smbios table!\n"));\r
-        ZeroMem (Packet->Dhcp4.Header.ClientHwAddr, sizeof (EFI_GUID));\r
+        break;\r
+      }\r
+\r
+      if (Private->OfferNum < PXEBC_OFFER_MAX_NUM) {\r
+        //\r
+        // Cache the DHCPv4 offers to OfferBuffer[] for select later, and record\r
+        // the OfferIndex and OfferCount.\r
+        // If error happens, just ignore this packet and continue to wait more offer.\r
+        //\r
+        PxeBcCacheDhcp4Offer (Private, Packet);\r
       }\r
-      Packet->Dhcp4.Header.HwAddrLen = (UINT8) sizeof (EFI_GUID);\r
-    }\r
-    break;\r
 \r
-  case Dhcp4RcvdOffer:\r
-    Status = EFI_NOT_READY;\r
-    if (Packet->Length > PXEBC_DHCP4_PACKET_MAX_SIZE) {\r
-      //\r
-      // Ignore the incoming packets which exceed the maximum length.\r
-      //\r
       break;\r
-    }\r
-    if (Private->OfferNum < PXEBC_OFFER_MAX_NUM) {\r
+\r
+    case Dhcp4SelectOffer:\r
+      ASSERT (NewPacket != NULL);\r
+\r
       //\r
-      // Cache the DHCPv4 offers to OfferBuffer[] for select later, and record\r
-      // the OfferIndex and OfferCount.\r
-      // If error happens, just ignore this packet and continue to wait more offer.\r
+      // Select offer by the default policy or by order, and record the SelectIndex\r
+      // and SelectProxyType.\r
       //\r
-      PxeBcCacheDhcp4Offer (Private, Packet);\r
-    }\r
-    break;\r
+      PxeBcSelectDhcp4Offer (Private);\r
 \r
-  case Dhcp4SelectOffer:\r
-    ASSERT (NewPacket != NULL);\r
+      if (Private->SelectIndex == 0) {\r
+        Status = EFI_ABORTED;\r
+      } else {\r
+        *NewPacket = &Private->OfferBuffer[Private->SelectIndex - 1].Dhcp4.Packet.Offer;\r
+      }\r
 \r
-    //\r
-    // Select offer by the default policy or by order, and record the SelectIndex\r
-    // and SelectProxyType.\r
-    //\r
-    PxeBcSelectDhcp4Offer (Private);\r
+      break;\r
 \r
-    if (Private->SelectIndex == 0) {\r
-      Status = EFI_ABORTED;\r
-    } else {\r
-      *NewPacket = &Private->OfferBuffer[Private->SelectIndex - 1].Dhcp4.Packet.Offer;\r
-    }\r
-    break;\r
+    case Dhcp4RcvdAck:\r
+      //\r
+      // Cache the DHCPv4 ack to Private->Dhcp4Ack, but it's not the final ack in mode data\r
+      // without verification.\r
+      //\r
+      ASSERT (Private->SelectIndex != 0);\r
 \r
-  case Dhcp4RcvdAck:\r
-    //\r
-    // Cache the DHCPv4 ack to Private->Dhcp4Ack, but it's not the final ack in mode data\r
-    // without verification.\r
-    //\r
-    ASSERT (Private->SelectIndex != 0);\r
+      Status = PxeBcCopyDhcp4Ack (Private, Packet, FALSE);\r
+      if (EFI_ERROR (Status)) {\r
+        Status = EFI_ABORTED;\r
+      }\r
 \r
-    Status = PxeBcCopyDhcp4Ack (Private, Packet, FALSE);\r
-    if (EFI_ERROR (Status)) {\r
-      Status = EFI_ABORTED;\r
-    }\r
-    break;\r
+      break;\r
 \r
-  default:\r
-    break;\r
+    default:\r
+      break;\r
   }\r
 \r
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Build and send out the request packet for the bootfile, and parse the reply.\r
 \r
@@ -1358,37 +1353,37 @@ PxeBcDhcp4CallBack (
 **/\r
 EFI_STATUS\r
 PxeBcDhcp4Discover (\r
-  IN  PXEBC_PRIVATE_DATA              *Private,\r
-  IN  UINT16                          Type,\r
-  IN  UINT16                          *Layer,\r
-  IN  BOOLEAN                         UseBis,\r
-  IN  EFI_IP_ADDRESS                  *DestIp,\r
-  IN  UINT16                          IpCount,\r
-  IN  EFI_PXE_BASE_CODE_SRVLIST       *SrvList\r
+  IN  PXEBC_PRIVATE_DATA         *Private,\r
+  IN  UINT16                     Type,\r
+  IN  UINT16                     *Layer,\r
+  IN  BOOLEAN                    UseBis,\r
+  IN  EFI_IP_ADDRESS             *DestIp,\r
+  IN  UINT16                     IpCount,\r
+  IN  EFI_PXE_BASE_CODE_SRVLIST  *SrvList\r
   )\r
 {\r
-  EFI_PXE_BASE_CODE_UDP_PORT          Sport;\r
-  EFI_PXE_BASE_CODE_MODE              *Mode;\r
-  EFI_DHCP4_PROTOCOL                  *Dhcp4;\r
-  EFI_DHCP4_TRANSMIT_RECEIVE_TOKEN    Token;\r
-  BOOLEAN                             IsBCast;\r
-  EFI_STATUS                          Status;\r
-  UINT16                              RepIndex;\r
-  UINT16                              SrvIndex;\r
-  UINT16                              TryIndex;\r
-  EFI_DHCP4_LISTEN_POINT              ListenPoint;\r
-  EFI_DHCP4_PACKET                    *Response;\r
-  UINT8                               Buffer[PXEBC_DHCP4_OPTION_MAX_SIZE];\r
-  EFI_DHCP4_PACKET_OPTION             *OptList[PXEBC_DHCP4_OPTION_MAX_NUM];\r
-  UINT32                              OptCount;\r
-  EFI_DHCP4_PACKET_OPTION             *PxeOpt;\r
-  PXEBC_OPTION_BOOT_ITEM              *PxeBootItem;\r
-  UINT8                               VendorOptLen;\r
-  UINT32                              Xid;\r
-\r
-  Mode      = Private->PxeBc.Mode;\r
-  Dhcp4     = Private->Dhcp4;\r
-  Status    = EFI_SUCCESS;\r
+  EFI_PXE_BASE_CODE_UDP_PORT        Sport;\r
+  EFI_PXE_BASE_CODE_MODE            *Mode;\r
+  EFI_DHCP4_PROTOCOL                *Dhcp4;\r
+  EFI_DHCP4_TRANSMIT_RECEIVE_TOKEN  Token;\r
+  BOOLEAN                           IsBCast;\r
+  EFI_STATUS                        Status;\r
+  UINT16                            RepIndex;\r
+  UINT16                            SrvIndex;\r
+  UINT16                            TryIndex;\r
+  EFI_DHCP4_LISTEN_POINT            ListenPoint;\r
+  EFI_DHCP4_PACKET                  *Response;\r
+  UINT8                             Buffer[PXEBC_DHCP4_OPTION_MAX_SIZE];\r
+  EFI_DHCP4_PACKET_OPTION           *OptList[PXEBC_DHCP4_OPTION_MAX_NUM];\r
+  UINT32                            OptCount;\r
+  EFI_DHCP4_PACKET_OPTION           *PxeOpt;\r
+  PXEBC_OPTION_BOOT_ITEM            *PxeBootItem;\r
+  UINT8                             VendorOptLen;\r
+  UINT32                            Xid;\r
+\r
+  Mode   = Private->PxeBc.Mode;\r
+  Dhcp4  = Private->Dhcp4;\r
+  Status = EFI_SUCCESS;\r
 \r
   ZeroMem (&Token, sizeof (EFI_DHCP4_TRANSMIT_RECEIVE_TOKEN));\r
 \r
@@ -1403,7 +1398,7 @@ PxeBcDhcp4Discover (
     IsBCast = FALSE;\r
   }\r
 \r
-  if (!UseBis && Layer != NULL) {\r
+  if (!UseBis && (Layer != NULL)) {\r
     *Layer &= EFI_PXE_BASE_CODE_BOOT_LAYER_MASK;\r
   }\r
 \r
@@ -1416,23 +1411,23 @@ PxeBcDhcp4Discover (
     //\r
     // Add vendor option of PXE_BOOT_ITEM\r
     //\r
-    VendorOptLen      = (UINT8) ((sizeof (EFI_DHCP4_PACKET_OPTION) - 1) * 2 + sizeof (PXEBC_OPTION_BOOT_ITEM) + 1);\r
+    VendorOptLen      = (UINT8)((sizeof (EFI_DHCP4_PACKET_OPTION) - 1) * 2 + sizeof (PXEBC_OPTION_BOOT_ITEM) + 1);\r
     OptList[OptCount] = AllocateZeroPool (VendorOptLen);\r
     if (OptList[OptCount] == NULL) {\r
       return EFI_OUT_OF_RESOURCES;\r
     }\r
 \r
-    OptList[OptCount]->OpCode     = DHCP4_TAG_VENDOR;\r
-    OptList[OptCount]->Length     = (UINT8) (VendorOptLen - 2);\r
-    PxeOpt                        = (EFI_DHCP4_PACKET_OPTION *) OptList[OptCount]->Data;\r
-    PxeOpt->OpCode                = PXEBC_VENDOR_TAG_BOOT_ITEM;\r
-    PxeOpt->Length                = (UINT8) sizeof (PXEBC_OPTION_BOOT_ITEM);\r
-    PxeBootItem                   = (PXEBC_OPTION_BOOT_ITEM *) PxeOpt->Data;\r
-    PxeBootItem->Type             = HTONS (Type);\r
-    PxeOpt->Data[PxeOpt->Length]  = DHCP4_TAG_EOP;\r
+    OptList[OptCount]->OpCode    = DHCP4_TAG_VENDOR;\r
+    OptList[OptCount]->Length    = (UINT8)(VendorOptLen - 2);\r
+    PxeOpt                       = (EFI_DHCP4_PACKET_OPTION *)OptList[OptCount]->Data;\r
+    PxeOpt->OpCode               = PXEBC_VENDOR_TAG_BOOT_ITEM;\r
+    PxeOpt->Length               = (UINT8)sizeof (PXEBC_OPTION_BOOT_ITEM);\r
+    PxeBootItem                  = (PXEBC_OPTION_BOOT_ITEM *)PxeOpt->Data;\r
+    PxeBootItem->Type            = HTONS (Type);\r
+    PxeOpt->Data[PxeOpt->Length] = DHCP4_TAG_EOP;\r
 \r
     if (Layer != NULL) {\r
-      PxeBootItem->Layer          = HTONS (*Layer);\r
+      PxeBootItem->Layer = HTONS (*Layer);\r
     }\r
 \r
     OptCount++;\r
@@ -1462,14 +1457,15 @@ PxeBcDhcp4Discover (
   }\r
 \r
   if (Mode->SendGUID) {\r
-    if (EFI_ERROR (NetLibGetSystemGuid ((EFI_GUID *) Token.Packet->Dhcp4.Header.ClientHwAddr))) {\r
+    if (EFI_ERROR (NetLibGetSystemGuid ((EFI_GUID *)Token.Packet->Dhcp4.Header.ClientHwAddr))) {\r
       //\r
       // Zero the Guid to indicate NOT programmable if failed to get system Guid.\r
       //\r
       DEBUG ((DEBUG_WARN, "PXE: Failed to read system GUID from the smbios table!\n"));\r
       ZeroMem (Token.Packet->Dhcp4.Header.ClientHwAddr, sizeof (EFI_GUID));\r
     }\r
-    Token.Packet->Dhcp4.Header.HwAddrLen = (UINT8)  sizeof (EFI_GUID);\r
+\r
+    Token.Packet->Dhcp4.Header.HwAddrLen = (UINT8)sizeof (EFI_GUID);\r
   }\r
 \r
   //\r
@@ -1477,7 +1473,7 @@ PxeBcDhcp4Discover (
   //\r
   Xid                                 = NET_RANDOM (NetRandomInitSeed ());\r
   Token.Packet->Dhcp4.Header.Xid      = HTONL (Xid);\r
-  Token.Packet->Dhcp4.Header.Reserved = HTONS ((UINT16) ((IsBCast) ? 0x8000 : 0x0));\r
+  Token.Packet->Dhcp4.Header.Reserved = HTONS ((UINT16)((IsBCast) ? 0x8000 : 0x0));\r
   CopyMem (&Token.Packet->Dhcp4.Header.ClientAddr, &Private->StationIp, sizeof (EFI_IPv4_ADDRESS));\r
 \r
   Token.RemotePort = Sport;\r
@@ -1491,20 +1487,19 @@ PxeBcDhcp4Discover (
   CopyMem (&Token.GatewayAddress, &Private->GatewayIp, sizeof (EFI_IPv4_ADDRESS));\r
 \r
   if (!IsBCast) {\r
-    Token.ListenPointCount            = 1;\r
-    Token.ListenPoints                = &ListenPoint;\r
-    Token.ListenPoints[0].ListenPort  = PXEBC_BS_DISCOVER_PORT;\r
-    CopyMem (&Token.ListenPoints[0].ListenAddress, &Private->StationIp, sizeof(EFI_IPv4_ADDRESS));\r
-    CopyMem (&Token.ListenPoints[0].SubnetMask, &Private->SubnetMask, sizeof(EFI_IPv4_ADDRESS));\r
+    Token.ListenPointCount           = 1;\r
+    Token.ListenPoints               = &ListenPoint;\r
+    Token.ListenPoints[0].ListenPort = PXEBC_BS_DISCOVER_PORT;\r
+    CopyMem (&Token.ListenPoints[0].ListenAddress, &Private->StationIp, sizeof (EFI_IPv4_ADDRESS));\r
+    CopyMem (&Token.ListenPoints[0].SubnetMask, &Private->SubnetMask, sizeof (EFI_IPv4_ADDRESS));\r
   }\r
 \r
   //\r
   // Send out the request packet to discover the bootfile.\r
   //\r
   for (TryIndex = 1; TryIndex <= PXEBC_BOOT_REQUEST_RETRIES; TryIndex++) {\r
-\r
-    Token.TimeoutValue                  = (UINT16) (PXEBC_BOOT_REQUEST_TIMEOUT * TryIndex);\r
-    Token.Packet->Dhcp4.Header.Seconds  = (UINT16) (PXEBC_BOOT_REQUEST_TIMEOUT * (TryIndex - 1));\r
+    Token.TimeoutValue                 = (UINT16)(PXEBC_BOOT_REQUEST_TIMEOUT * TryIndex);\r
+    Token.Packet->Dhcp4.Header.Seconds = (UINT16)(PXEBC_BOOT_REQUEST_TIMEOUT * (TryIndex - 1));\r
 \r
     Status = Dhcp4->TransmitReceive (Dhcp4, &Token);\r
     if (Token.Status != EFI_TIMEOUT) {\r
@@ -1520,10 +1515,9 @@ PxeBcDhcp4Discover (
   }\r
 \r
   if (!EFI_ERROR (Status)) {\r
-\r
-    RepIndex  = 0;\r
-    SrvIndex  = 0;\r
-    Response  = Token.ResponseList;\r
+    RepIndex = 0;\r
+    SrvIndex = 0;\r
+    Response = Token.ResponseList;\r
     //\r
     // Find the right PXE Reply according to server address.\r
     //\r
@@ -1531,7 +1525,7 @@ PxeBcDhcp4Discover (
       if (Response->Length > PXEBC_DHCP4_PACKET_MAX_SIZE) {\r
         SrvIndex = 0;\r
         RepIndex++;\r
-        Response = (EFI_DHCP4_PACKET *) ((UINT8 *) Response + Response->Size);\r
+        Response = (EFI_DHCP4_PACKET *)((UINT8 *)Response + Response->Size);\r
         continue;\r
       }\r
 \r
@@ -1539,10 +1533,13 @@ PxeBcDhcp4Discover (
         if (SrvList[SrvIndex].AcceptAnyResponse) {\r
           break;\r
         }\r
+\r
         if ((SrvList[SrvIndex].Type == Type) &&\r
-            EFI_IP4_EQUAL (&Response->Dhcp4.Header.ServerAddr, &SrvList[SrvIndex].IpAddr)) {\r
+            EFI_IP4_EQUAL (&Response->Dhcp4.Header.ServerAddr, &SrvList[SrvIndex].IpAddr))\r
+        {\r
           break;\r
         }\r
+\r
         SrvIndex++;\r
       }\r
 \r
@@ -1552,7 +1549,7 @@ PxeBcDhcp4Discover (
 \r
       SrvIndex = 0;\r
       RepIndex++;\r
-      Response = (EFI_DHCP4_PACKET *) ((UINT8 *) Response + Response->Size);\r
+      Response = (EFI_DHCP4_PACKET *)((UINT8 *)Response + Response->Size);\r
     }\r
 \r
     if (RepIndex < Token.ResponseCount) {\r
@@ -1562,13 +1559,14 @@ PxeBcDhcp4Discover (
       //\r
       if (Private->IsDoDiscover) {\r
         Status = PxeBcCacheDhcp4Packet (&Private->PxeReply.Dhcp4.Packet.Ack, Response);\r
-        if (EFI_ERROR(Status)) {\r
+        if (EFI_ERROR (Status)) {\r
           goto ON_EXIT;\r
         }\r
+\r
         CopyMem (&Mode->PxeDiscover, &Token.Packet->Dhcp4, Token.Packet->Length);\r
       } else {\r
         Status = PxeBcCacheDhcp4Packet (&Private->ProxyOffer.Dhcp4.Packet.Offer, Response);\r
-        if (EFI_ERROR(Status)) {\r
+        if (EFI_ERROR (Status)) {\r
           goto ON_EXIT;\r
         }\r
       }\r
@@ -1579,14 +1577,17 @@ PxeBcDhcp4Discover (
       Status = EFI_NOT_FOUND;\r
     }\r
   }\r
+\r
 ON_EXIT:\r
 \r
   if (Token.ResponseList != NULL) {\r
     FreePool (Token.ResponseList);\r
   }\r
+\r
   if (Token.Packet != NULL) {\r
     FreePool (Token.Packet);\r
   }\r
+\r
   return Status;\r
 }\r
 \r
@@ -1601,40 +1602,40 @@ ON_EXIT:
 **/\r
 EFI_STATUS\r
 PxeBcSetIp4Policy (\r
-  IN PXEBC_PRIVATE_DATA            *Private\r
+  IN PXEBC_PRIVATE_DATA  *Private\r
   )\r
 {\r
-  EFI_STATUS                   Status;\r
-  EFI_IP4_CONFIG2_PROTOCOL     *Ip4Config2;\r
-  EFI_IP4_CONFIG2_POLICY       Policy;\r
-  UINTN                        DataSize;\r
+  EFI_STATUS                Status;\r
+  EFI_IP4_CONFIG2_PROTOCOL  *Ip4Config2;\r
+  EFI_IP4_CONFIG2_POLICY    Policy;\r
+  UINTN                     DataSize;\r
 \r
   Ip4Config2 = Private->Ip4Config2;\r
-  DataSize = sizeof (EFI_IP4_CONFIG2_POLICY);\r
-  Status = Ip4Config2->GetData (\r
-                       Ip4Config2,\r
-                       Ip4Config2DataTypePolicy,\r
-                       &DataSize,\r
-                       &Policy\r
-                       );\r
+  DataSize   = sizeof (EFI_IP4_CONFIG2_POLICY);\r
+  Status     = Ip4Config2->GetData (\r
+                             Ip4Config2,\r
+                             Ip4Config2DataTypePolicy,\r
+                             &DataSize,\r
+                             &Policy\r
+                             );\r
   if (EFI_ERROR (Status)) {\r
     return Status;\r
   }\r
 \r
   if (Policy != Ip4Config2PolicyStatic) {\r
     Policy = Ip4Config2PolicyStatic;\r
-    Status= Ip4Config2->SetData (\r
-                          Ip4Config2,\r
-                          Ip4Config2DataTypePolicy,\r
-                          sizeof (EFI_IP4_CONFIG2_POLICY),\r
-                          &Policy\r
-                          );\r
+    Status = Ip4Config2->SetData (\r
+                           Ip4Config2,\r
+                           Ip4Config2DataTypePolicy,\r
+                           sizeof (EFI_IP4_CONFIG2_POLICY),\r
+                           &Policy\r
+                           );\r
     if (EFI_ERROR (Status)) {\r
       return Status;\r
     }\r
   }\r
 \r
-  return  EFI_SUCCESS;\r
+  return EFI_SUCCESS;\r
 }\r
 \r
 /**\r
@@ -1649,28 +1650,28 @@ PxeBcSetIp4Policy (
 **/\r
 EFI_STATUS\r
 PxeBcDhcp4Dora (\r
-  IN PXEBC_PRIVATE_DATA        *Private,\r
-  IN EFI_DHCP4_PROTOCOL        *Dhcp4\r
+  IN PXEBC_PRIVATE_DATA  *Private,\r
+  IN EFI_DHCP4_PROTOCOL  *Dhcp4\r
   )\r
 {\r
-  EFI_PXE_BASE_CODE_MODE       *PxeMode;\r
-  EFI_DHCP4_CONFIG_DATA        Config;\r
-  EFI_DHCP4_MODE_DATA          Mode;\r
-  EFI_DHCP4_PACKET_OPTION      *OptList[PXEBC_DHCP4_OPTION_MAX_NUM];\r
-  UINT8                        Buffer[PXEBC_DHCP4_OPTION_MAX_SIZE];\r
-  UINT32                       OptCount;\r
-  EFI_STATUS                   Status;\r
+  EFI_PXE_BASE_CODE_MODE   *PxeMode;\r
+  EFI_DHCP4_CONFIG_DATA    Config;\r
+  EFI_DHCP4_MODE_DATA      Mode;\r
+  EFI_DHCP4_PACKET_OPTION  *OptList[PXEBC_DHCP4_OPTION_MAX_NUM];\r
+  UINT8                    Buffer[PXEBC_DHCP4_OPTION_MAX_SIZE];\r
+  UINT32                   OptCount;\r
+  EFI_STATUS               Status;\r
 \r
   ASSERT (Dhcp4 != NULL);\r
 \r
-  Status   = EFI_SUCCESS;\r
-  PxeMode  = Private->PxeBc.Mode;\r
+  Status  = EFI_SUCCESS;\r
+  PxeMode = Private->PxeBc.Mode;\r
 \r
   //\r
   // Build option list for the request packet.\r
   //\r
   OptCount = PxeBcBuildDhcp4Options (Private, OptList, Buffer, FALSE);\r
-  ASSERT (OptCount> 0);\r
+  ASSERT (OptCount > 0);\r
 \r
   ZeroMem (&Mode, sizeof (EFI_DHCP4_MODE_DATA));\r
   ZeroMem (&Config, sizeof (EFI_DHCP4_CONFIG_DATA));\r
@@ -1704,7 +1705,7 @@ PxeBcDhcp4Dora (
       PxeMode->IcmpErrorReceived = TRUE;\r
     }\r
 \r
-    if (Status == EFI_TIMEOUT && Private->OfferNum > 0) {\r
+    if ((Status == EFI_TIMEOUT) && (Private->OfferNum > 0)) {\r
       Status = EFI_NO_RESPONSE;\r
     }\r
 \r