]> git.proxmox.com Git - mirror_edk2.git/blobdiff - NetworkPkg/UefiPxeBcDxe/PxeBcImpl.c
NetworkPkg: Apply uncrustify changes
[mirror_edk2.git] / NetworkPkg / UefiPxeBcDxe / PxeBcImpl.c
index 2e9d31547dd6f4a6933a7d124f372d17342ff980..0a4baf6f90e8583c884cd2b4484401de35cc5f21 100644 (file)
@@ -9,7 +9,6 @@
 \r
 #include "PxeBcImpl.h"\r
 \r
-\r
 /**\r
   Enables the use of the PXE Base Code Protocol functions.\r
 \r
@@ -42,8 +41,8 @@
 EFI_STATUS\r
 EFIAPI\r
 EfiPxeBcStart (\r
-  IN EFI_PXE_BASE_CODE_PROTOCOL       *This,\r
-  IN BOOLEAN                          UseIpv6\r
+  IN EFI_PXE_BASE_CODE_PROTOCOL  *This,\r
+  IN BOOLEAN                     UseIpv6\r
   )\r
 {\r
   PXEBC_PRIVATE_DATA      *Private;\r
@@ -65,9 +64,9 @@ EfiPxeBcStart (
   //\r
   // Detect whether using IPv6 or not, and set it into mode data.\r
   //\r
-  if (UseIpv6 && Mode->Ipv6Available && Mode->Ipv6Supported && Private->Ip6Nic != NULL) {\r
+  if (UseIpv6 && Mode->Ipv6Available && Mode->Ipv6Supported && (Private->Ip6Nic != NULL)) {\r
     Mode->UsingIpv6 = TRUE;\r
-  } else if (!UseIpv6 && Private->Ip4Nic != NULL) {\r
+  } else if (!UseIpv6 && (Private->Ip4Nic != NULL)) {\r
     Mode->UsingIpv6 = FALSE;\r
   } else {\r
     return EFI_UNSUPPORTED;\r
@@ -90,7 +89,7 @@ EfiPxeBcStart (
     // Configure block size for TFTP as a default value to handle all link layers.\r
     //\r
     Private->BlockSize = Private->Ip6MaxPacketSize -\r
-                           PXEBC_DEFAULT_UDP_OVERHEAD_SIZE - PXEBC_DEFAULT_TFTP_OVERHEAD_SIZE;\r
+                         PXEBC_DEFAULT_UDP_OVERHEAD_SIZE - PXEBC_DEFAULT_TFTP_OVERHEAD_SIZE;\r
 \r
     //\r
     // PXE over IPv6 starts here, initialize the fields and list header.\r
@@ -108,13 +107,13 @@ EfiPxeBcStart (
     // Create event and set status for token to capture ICMP6 error message.\r
     //\r
     Private->Icmp6Token.Status = EFI_NOT_READY;\r
-    Status = gBS->CreateEvent (\r
-                    EVT_NOTIFY_SIGNAL,\r
-                    TPL_NOTIFY,\r
-                    PxeBcIcmp6ErrorUpdate,\r
-                    Private,\r
-                    &Private->Icmp6Token.Event\r
-                    );\r
+    Status                     = gBS->CreateEvent (\r
+                                        EVT_NOTIFY_SIGNAL,\r
+                                        TPL_NOTIFY,\r
+                                        PxeBcIcmp6ErrorUpdate,\r
+                                        Private,\r
+                                        &Private->Icmp6Token.Event\r
+                                        );\r
     if (EFI_ERROR (Status)) {\r
       goto ON_ERROR;\r
     }\r
@@ -143,7 +142,7 @@ EfiPxeBcStart (
     // Configure block size for TFTP as a default value to handle all link layers.\r
     //\r
     Private->BlockSize = Private->Ip4MaxPacketSize -\r
-                           PXEBC_DEFAULT_UDP_OVERHEAD_SIZE - PXEBC_DEFAULT_TFTP_OVERHEAD_SIZE;\r
+                         PXEBC_DEFAULT_UDP_OVERHEAD_SIZE - PXEBC_DEFAULT_TFTP_OVERHEAD_SIZE;\r
 \r
     //\r
     // PXE over IPv4 starts here, initialize the fields.\r
@@ -188,23 +187,23 @@ EfiPxeBcStart (
     // Create event and set status for token to capture ICMP error message.\r
     //\r
     Private->Icmp6Token.Status = EFI_NOT_READY;\r
-    Status = gBS->CreateEvent (\r
-                    EVT_NOTIFY_SIGNAL,\r
-                    TPL_NOTIFY,\r
-                    PxeBcIcmpErrorUpdate,\r
-                    Private,\r
-                    &Private->IcmpToken.Event\r
-                    );\r
+    Status                     = gBS->CreateEvent (\r
+                                        EVT_NOTIFY_SIGNAL,\r
+                                        TPL_NOTIFY,\r
+                                        PxeBcIcmpErrorUpdate,\r
+                                        Private,\r
+                                        &Private->IcmpToken.Event\r
+                                        );\r
     if (EFI_ERROR (Status)) {\r
       goto ON_ERROR;\r
     }\r
 \r
     //\r
-    //DHCP4 service allows only one of its children to be configured in\r
-    //the active state, If the DHCP4 D.O.R.A started by IP4 auto\r
-    //configuration and has not been completed, the Dhcp4 state machine\r
-    //will not be in the right state for the PXE to start a new round D.O.R.A.\r
-    //so we need to switch its policy to static.\r
+    // DHCP4 service allows only one of its children to be configured in\r
+    // the active state, If the DHCP4 D.O.R.A started by IP4 auto\r
+    // configuration and has not been completed, the Dhcp4 state machine\r
+    // will not be in the right state for the PXE to start a new round D.O.R.A.\r
+    // so we need to switch its policy to static.\r
     //\r
     Status = PxeBcSetIp4Policy (Private);\r
     if (EFI_ERROR (Status)) {\r
@@ -216,7 +215,7 @@ EfiPxeBcStart (
   // If PcdTftpBlockSize is set to non-zero, override the default value.\r
   //\r
   if (PcdGet64 (PcdTftpBlockSize) != 0) {\r
-    Private->BlockSize   = (UINTN) PcdGet64 (PcdTftpBlockSize);\r
+    Private->BlockSize = (UINTN)PcdGet64 (PcdTftpBlockSize);\r
   }\r
 \r
   //\r
@@ -244,6 +243,7 @@ ON_ERROR:
       gBS->CloseEvent (Private->Icmp6Token.Event);\r
       Private->Icmp6Token.Event = NULL;\r
     }\r
+\r
     Private->Udp6Read->Configure (Private->Udp6Read, NULL);\r
     Private->Ip6->Configure (Private->Ip6, NULL);\r
   } else {\r
@@ -251,17 +251,19 @@ ON_ERROR:
       gBS->CloseEvent (Private->ArpUpdateEvent);\r
       Private->ArpUpdateEvent = NULL;\r
     }\r
+\r
     if (Private->IcmpToken.Event != NULL) {\r
       gBS->CloseEvent (Private->IcmpToken.Event);\r
       Private->IcmpToken.Event = NULL;\r
     }\r
+\r
     Private->Udp4Read->Configure (Private->Udp4Read, NULL);\r
     Private->Ip4->Configure (Private->Ip4, NULL);\r
   }\r
+\r
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Disable the use of the PXE Base Code Protocol functions.\r
 \r
@@ -282,7 +284,7 @@ ON_ERROR:
 EFI_STATUS\r
 EFIAPI\r
 EfiPxeBcStop (\r
-  IN EFI_PXE_BASE_CODE_PROTOCOL       *This\r
+  IN EFI_PXE_BASE_CODE_PROTOCOL  *This\r
   )\r
 {\r
   PXEBC_PRIVATE_DATA      *Private;\r
@@ -321,10 +323,12 @@ EfiPxeBcStop (
       gBS->CloseEvent (Private->Icmp6Token.Event);\r
       Private->Icmp6Token.Event = NULL;\r
     }\r
+\r
     if (Private->Dhcp6Request != NULL) {\r
       FreePool (Private->Dhcp6Request);\r
       Private->Dhcp6Request = NULL;\r
     }\r
+\r
     if (Private->BootFileName != NULL) {\r
       FreePool (Private->BootFileName);\r
       Private->BootFileName = NULL;\r
@@ -348,10 +352,12 @@ EfiPxeBcStop (
       gBS->CloseEvent (Private->ArpUpdateEvent);\r
       Private->ArpUpdateEvent = NULL;\r
     }\r
+\r
     if (Private->IcmpToken.Event != NULL) {\r
       gBS->CloseEvent (Private->IcmpToken.Event);\r
       Private->IcmpToken.Event = NULL;\r
     }\r
+\r
     Private->BootFileName = NULL;\r
   }\r
 \r
@@ -378,7 +384,6 @@ EfiPxeBcStop (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Attempts to complete a DHCPv4 D.O.R.A. (discover / offer / request / acknowledge) or DHCPv6\r
   S.A.R.R (solicit / advertise / request / reply) sequence.\r
@@ -410,8 +415,8 @@ EfiPxeBcStop (
 EFI_STATUS\r
 EFIAPI\r
 EfiPxeBcDhcp (\r
-  IN EFI_PXE_BASE_CODE_PROTOCOL       *This,\r
-  IN BOOLEAN                          SortOffers\r
+  IN EFI_PXE_BASE_CODE_PROTOCOL  *This,\r
+  IN BOOLEAN                     SortOffers\r
   )\r
 {\r
   PXEBC_PRIVATE_DATA           *Private;\r
@@ -437,7 +442,6 @@ EfiPxeBcDhcp (
   }\r
 \r
   if (Mode->UsingIpv6) {\r
-\r
     //\r
     // Stop Udp6Read instance\r
     //\r
@@ -448,7 +452,6 @@ EfiPxeBcDhcp (
     //\r
     Status = PxeBcDhcp6Sarr (Private, Private->Dhcp6);\r
   } else {\r
-\r
     //\r
     // Stop Udp4Read instance\r
     //\r
@@ -468,18 +471,18 @@ EfiPxeBcDhcp (
   } else {\r
     Private->Udp4Read->Configure (Private->Udp4Read, &Private->Udp4CfgData);\r
   }\r
+\r
   //\r
   // Dhcp(), Discover(), and Mtftp() set the IP filter, and return with the IP\r
   // receive filter list emptied and the filter set to EFI_PXE_BASE_CODE_IP_FILTER_STATION_IP.\r
   //\r
-  ZeroMem(&IpFilter, sizeof (EFI_PXE_BASE_CODE_IP_FILTER));\r
+  ZeroMem (&IpFilter, sizeof (EFI_PXE_BASE_CODE_IP_FILTER));\r
   IpFilter.Filters = EFI_PXE_BASE_CODE_IP_FILTER_STATION_IP;\r
   This->SetIpFilter (This, &IpFilter);\r
 \r
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Attempts to complete the PXE Boot Server and/or boot image discovery sequence.\r
 \r
@@ -534,15 +537,15 @@ EfiPxeBcDiscover (
   IN EFI_PXE_BASE_CODE_DISCOVER_INFO  *Info   OPTIONAL\r
   )\r
 {\r
-  PXEBC_PRIVATE_DATA              *Private;\r
-  EFI_PXE_BASE_CODE_MODE          *Mode;\r
-  EFI_PXE_BASE_CODE_DISCOVER_INFO DefaultInfo;\r
-  EFI_PXE_BASE_CODE_SRVLIST       *SrvList;\r
-  PXEBC_BOOT_SVR_ENTRY            *BootSvrEntry;\r
-  UINT16                          Index;\r
-  EFI_STATUS                      Status;\r
-  EFI_PXE_BASE_CODE_IP_FILTER     IpFilter;\r
-  EFI_PXE_BASE_CODE_DISCOVER_INFO *NewCreatedInfo;\r
+  PXEBC_PRIVATE_DATA               *Private;\r
+  EFI_PXE_BASE_CODE_MODE           *Mode;\r
+  EFI_PXE_BASE_CODE_DISCOVER_INFO  DefaultInfo;\r
+  EFI_PXE_BASE_CODE_SRVLIST        *SrvList;\r
+  PXEBC_BOOT_SVR_ENTRY             *BootSvrEntry;\r
+  UINT16                           Index;\r
+  EFI_STATUS                       Status;\r
+  EFI_PXE_BASE_CODE_IP_FILTER      IpFilter;\r
+  EFI_PXE_BASE_CODE_DISCOVER_INFO  *NewCreatedInfo;\r
 \r
   if (This == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
@@ -569,13 +572,11 @@ EfiPxeBcDiscover (
   }\r
 \r
   if (Mode->UsingIpv6) {\r
-\r
     //\r
     // Stop Udp6Read instance\r
     //\r
     Private->Udp6Read->Configure (Private->Udp6Read, NULL);\r
   } else {\r
-\r
     //\r
     // Stop Udp4Read instance\r
     //\r
@@ -592,7 +593,8 @@ EfiPxeBcDiscover (
     //\r
     if (!Mode->PxeDiscoverValid ||\r
         !Mode->PxeReplyReceived ||\r
-        (!Mode->PxeBisReplyReceived && UseBis)) {\r
+        (!Mode->PxeBisReplyReceived && UseBis))\r
+    {\r
       Status = EFI_INVALID_PARAMETER;\r
       goto ON_EXIT;\r
     }\r
@@ -605,16 +607,16 @@ EfiPxeBcDiscover (
     SrvList[0].AcceptAnyResponse = FALSE;\r
 \r
     CopyMem (&SrvList->IpAddr, &Private->ServerIp, sizeof (EFI_IP_ADDRESS));\r
-\r
   } else if (Info == NULL) {\r
     //\r
     // 2. Extract the discover information from the cached packets if unspecified.\r
     //\r
     NewCreatedInfo = &DefaultInfo;\r
-    Status = PxeBcExtractDiscoverInfo (Private, Type, &NewCreatedInfo, &BootSvrEntry, &SrvList);\r
+    Status         = PxeBcExtractDiscoverInfo (Private, Type, &NewCreatedInfo, &BootSvrEntry, &SrvList);\r
     if (EFI_ERROR (Status)) {\r
       goto ON_EXIT;\r
     }\r
+\r
     ASSERT (NewCreatedInfo != NULL);\r
     Info = NewCreatedInfo;\r
   } else {\r
@@ -629,6 +631,7 @@ EfiPxeBcDiscover (
           break;\r
         }\r
       }\r
+\r
       if (Index != Info->IpCnt) {\r
         //\r
         // It's invalid if the first server doesn't accept any response\r
@@ -644,7 +647,8 @@ EfiPxeBcDiscover (
   // Info and BootSvrEntry/SrvList are all ready by now, so execute discover by UniCast/BroadCast/MultiCast.\r
   //\r
   if ((!Info->UseUCast && !Info->UseBCast && !Info->UseMCast) ||\r
-      (Info->MustUseList && Info->IpCnt == 0)) {\r
+      (Info->MustUseList && (Info->IpCnt == 0)))\r
+  {\r
     Status = EFI_INVALID_PARAMETER;\r
     goto ON_EXIT;\r
   }\r
@@ -664,7 +668,6 @@ EfiPxeBcDiscover (
                Info->IpCnt,\r
                SrvList\r
                );\r
-\r
   } else if (Info->UseBCast) {\r
     //\r
     // Do discover by broadcast, but only valid for IPv4.\r
@@ -679,7 +682,6 @@ EfiPxeBcDiscover (
                Info->IpCnt,\r
                SrvList\r
                );\r
-\r
   } else if (Info->UseUCast) {\r
     //\r
     // Do discover by unicast.\r
@@ -702,7 +704,7 @@ EfiPxeBcDiscover (
                  Info->IpCnt,\r
                  SrvList\r
                  );\r
-      }\r
+    }\r
   }\r
 \r
   if (!EFI_ERROR (Status)) {\r
@@ -736,7 +738,7 @@ EfiPxeBcDiscover (
 \r
 ON_EXIT:\r
 \r
-  if (NewCreatedInfo != NULL && NewCreatedInfo != &DefaultInfo) {\r
+  if ((NewCreatedInfo != NULL) && (NewCreatedInfo != &DefaultInfo)) {\r
     FreePool (NewCreatedInfo);\r
   }\r
 \r
@@ -750,14 +752,13 @@ ON_EXIT:
   // Dhcp(), Discover(), and Mtftp() set the IP filter, and return with the IP\r
   // receive filter list emptied and the filter set to EFI_PXE_BASE_CODE_IP_FILTER_STATION_IP.\r
   //\r
-  ZeroMem(&IpFilter, sizeof (EFI_PXE_BASE_CODE_IP_FILTER));\r
+  ZeroMem (&IpFilter, sizeof (EFI_PXE_BASE_CODE_IP_FILTER));\r
   IpFilter.Filters = EFI_PXE_BASE_CODE_IP_FILTER_STATION_IP;\r
   This->SetIpFilter (This, &IpFilter);\r
 \r
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Used to perform TFTP and MTFTP services.\r
 \r
@@ -824,60 +825,62 @@ ON_EXIT:
 EFI_STATUS\r
 EFIAPI\r
 EfiPxeBcMtftp (\r
-  IN     EFI_PXE_BASE_CODE_PROTOCOL       *This,\r
-  IN     EFI_PXE_BASE_CODE_TFTP_OPCODE    Operation,\r
-  IN OUT VOID                             *BufferPtr    OPTIONAL,\r
-  IN     BOOLEAN                          Overwrite,\r
-  IN OUT UINT64                           *BufferSize,\r
-  IN     UINTN                            *BlockSize    OPTIONAL,\r
-  IN     EFI_IP_ADDRESS                   *ServerIp,\r
-  IN     UINT8                            *Filename,\r
-  IN     EFI_PXE_BASE_CODE_MTFTP_INFO     *Info         OPTIONAL,\r
-  IN     BOOLEAN                          DontUseBuffer\r
+  IN     EFI_PXE_BASE_CODE_PROTOCOL     *This,\r
+  IN     EFI_PXE_BASE_CODE_TFTP_OPCODE  Operation,\r
+  IN OUT VOID                           *BufferPtr    OPTIONAL,\r
+  IN     BOOLEAN                        Overwrite,\r
+  IN OUT UINT64                         *BufferSize,\r
+  IN     UINTN                          *BlockSize    OPTIONAL,\r
+  IN     EFI_IP_ADDRESS                 *ServerIp,\r
+  IN     UINT8                          *Filename,\r
+  IN     EFI_PXE_BASE_CODE_MTFTP_INFO   *Info         OPTIONAL,\r
+  IN     BOOLEAN                        DontUseBuffer\r
   )\r
 {\r
-  PXEBC_PRIVATE_DATA              *Private;\r
-  EFI_PXE_BASE_CODE_MODE          *Mode;\r
-  EFI_MTFTP4_CONFIG_DATA          Mtftp4Config;\r
-  EFI_MTFTP6_CONFIG_DATA          Mtftp6Config;\r
-  VOID                            *Config;\r
-  EFI_STATUS                      Status;\r
-  EFI_PXE_BASE_CODE_IP_FILTER     IpFilter;\r
-  UINTN                           WindowSize;\r
+  PXEBC_PRIVATE_DATA           *Private;\r
+  EFI_PXE_BASE_CODE_MODE       *Mode;\r
+  EFI_MTFTP4_CONFIG_DATA       Mtftp4Config;\r
+  EFI_MTFTP6_CONFIG_DATA       Mtftp6Config;\r
+  VOID                         *Config;\r
+  EFI_STATUS                   Status;\r
+  EFI_PXE_BASE_CODE_IP_FILTER  IpFilter;\r
+  UINTN                        WindowSize;\r
 \r
   if ((This == NULL) ||\r
       (Filename == NULL) ||\r
       (BufferSize == NULL) ||\r
       (ServerIp == NULL) ||\r
-      ((BlockSize != NULL) && (*BlockSize < PXE_MTFTP_DEFAULT_BLOCK_SIZE))) {\r
+      ((BlockSize != NULL) && (*BlockSize < PXE_MTFTP_DEFAULT_BLOCK_SIZE)))\r
+  {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
-  if (Operation == EFI_PXE_BASE_CODE_TFTP_READ_FILE ||\r
-      Operation == EFI_PXE_BASE_CODE_TFTP_READ_DIRECTORY ||\r
-      Operation == EFI_PXE_BASE_CODE_MTFTP_READ_FILE ||\r
-      Operation == EFI_PXE_BASE_CODE_MTFTP_READ_DIRECTORY) {\r
-    if (BufferPtr == NULL && !DontUseBuffer) {\r
+  if ((Operation == EFI_PXE_BASE_CODE_TFTP_READ_FILE) ||\r
+      (Operation == EFI_PXE_BASE_CODE_TFTP_READ_DIRECTORY) ||\r
+      (Operation == EFI_PXE_BASE_CODE_MTFTP_READ_FILE) ||\r
+      (Operation == EFI_PXE_BASE_CODE_MTFTP_READ_DIRECTORY))\r
+  {\r
+    if ((BufferPtr == NULL) && !DontUseBuffer) {\r
       return EFI_INVALID_PARAMETER;\r
     }\r
   }\r
 \r
-  Config    = NULL;\r
-  Status    = EFI_DEVICE_ERROR;\r
-  Private   = PXEBC_PRIVATE_DATA_FROM_PXEBC (This);\r
-  Mode      = Private->PxeBc.Mode;\r
+  Config  = NULL;\r
+  Status  = EFI_DEVICE_ERROR;\r
+  Private = PXEBC_PRIVATE_DATA_FROM_PXEBC (This);\r
+  Mode    = Private->PxeBc.Mode;\r
 \r
   //\r
   // Get PcdPxeTftpWindowSize.\r
   //\r
-  WindowSize = (UINTN) PcdGet64 (PcdPxeTftpWindowSize);\r
+  WindowSize = (UINTN)PcdGet64 (PcdPxeTftpWindowSize);\r
 \r
   if (Mode->UsingIpv6) {\r
     if (!NetIp6IsValidUnicast (&ServerIp->v6)) {\r
       return EFI_INVALID_PARAMETER;\r
     }\r
   } else {\r
-    if (IP4_IS_UNSPECIFIED (NTOHL (ServerIp->Addr[0])) || IP4_IS_LOCAL_BROADCAST (NTOHL (ServerIp->Addr[0])))   {\r
+    if (IP4_IS_UNSPECIFIED (NTOHL (ServerIp->Addr[0])) || IP4_IS_LOCAL_BROADCAST (NTOHL (ServerIp->Addr[0]))) {\r
       return EFI_INVALID_PARAMETER;\r
     }\r
   }\r
@@ -887,9 +890,9 @@ EfiPxeBcMtftp (
     // Set configuration data for Mtftp6 instance.\r
     //\r
     ZeroMem (&Mtftp6Config, sizeof (EFI_MTFTP6_CONFIG_DATA));\r
-    Config                         = &Mtftp6Config;\r
-    Mtftp6Config.TimeoutValue      = PXEBC_MTFTP_TIMEOUT;\r
-    Mtftp6Config.TryCount          = PXEBC_MTFTP_RETRIES;\r
+    Config                    = &Mtftp6Config;\r
+    Mtftp6Config.TimeoutValue = PXEBC_MTFTP_TIMEOUT;\r
+    Mtftp6Config.TryCount     = PXEBC_MTFTP_RETRIES;\r
     CopyMem (&Mtftp6Config.StationIp, &Private->StationIp.v6, sizeof (EFI_IPv6_ADDRESS));\r
     CopyMem (&Mtftp6Config.ServerIp, &ServerIp->v6, sizeof (EFI_IPv6_ADDRESS));\r
     //\r
@@ -919,83 +922,82 @@ EfiPxeBcMtftp (
   Mode->IcmpErrorReceived = FALSE;\r
 \r
   switch (Operation) {\r
+    case EFI_PXE_BASE_CODE_TFTP_GET_FILE_SIZE:\r
+      //\r
+      // Send TFTP request to get file size.\r
+      //\r
+      Status = PxeBcTftpGetFileSize (\r
+                 Private,\r
+                 Config,\r
+                 Filename,\r
+                 BlockSize,\r
+                 (WindowSize > 1) ? &WindowSize : NULL,\r
+                 BufferSize\r
+                 );\r
 \r
-  case EFI_PXE_BASE_CODE_TFTP_GET_FILE_SIZE:\r
-    //\r
-    // Send TFTP request to get file size.\r
-    //\r
-    Status = PxeBcTftpGetFileSize (\r
-               Private,\r
-               Config,\r
-               Filename,\r
-               BlockSize,\r
-               (WindowSize > 1) ? &WindowSize : NULL,\r
-               BufferSize\r
-               );\r
-\r
-    break;\r
+      break;\r
 \r
-  case EFI_PXE_BASE_CODE_TFTP_READ_FILE:\r
-    //\r
-    // Send TFTP request to read file.\r
-    //\r
-    Status = PxeBcTftpReadFile (\r
-               Private,\r
-               Config,\r
-               Filename,\r
-               BlockSize,\r
-               (WindowSize > 1) ? &WindowSize : NULL,\r
-               BufferPtr,\r
-               BufferSize,\r
-               DontUseBuffer\r
-               );\r
+    case EFI_PXE_BASE_CODE_TFTP_READ_FILE:\r
+      //\r
+      // Send TFTP request to read file.\r
+      //\r
+      Status = PxeBcTftpReadFile (\r
+                 Private,\r
+                 Config,\r
+                 Filename,\r
+                 BlockSize,\r
+                 (WindowSize > 1) ? &WindowSize : NULL,\r
+                 BufferPtr,\r
+                 BufferSize,\r
+                 DontUseBuffer\r
+                 );\r
 \r
-    break;\r
+      break;\r
 \r
-  case EFI_PXE_BASE_CODE_TFTP_WRITE_FILE:\r
-    //\r
-    // Send TFTP request to write file.\r
-    //\r
-    Status = PxeBcTftpWriteFile (\r
-               Private,\r
-               Config,\r
-               Filename,\r
-               Overwrite,\r
-               BlockSize,\r
-               BufferPtr,\r
-               BufferSize\r
-               );\r
+    case EFI_PXE_BASE_CODE_TFTP_WRITE_FILE:\r
+      //\r
+      // Send TFTP request to write file.\r
+      //\r
+      Status = PxeBcTftpWriteFile (\r
+                 Private,\r
+                 Config,\r
+                 Filename,\r
+                 Overwrite,\r
+                 BlockSize,\r
+                 BufferPtr,\r
+                 BufferSize\r
+                 );\r
 \r
-    break;\r
+      break;\r
 \r
-  case EFI_PXE_BASE_CODE_TFTP_READ_DIRECTORY:\r
-    //\r
-    // Send TFTP request to read directory.\r
-    //\r
-    Status = PxeBcTftpReadDirectory (\r
-               Private,\r
-               Config,\r
-               Filename,\r
-               BlockSize,\r
-               (WindowSize > 1) ? &WindowSize : NULL,\r
-               BufferPtr,\r
-               BufferSize,\r
-               DontUseBuffer\r
-               );\r
+    case EFI_PXE_BASE_CODE_TFTP_READ_DIRECTORY:\r
+      //\r
+      // Send TFTP request to read directory.\r
+      //\r
+      Status = PxeBcTftpReadDirectory (\r
+                 Private,\r
+                 Config,\r
+                 Filename,\r
+                 BlockSize,\r
+                 (WindowSize > 1) ? &WindowSize : NULL,\r
+                 BufferPtr,\r
+                 BufferSize,\r
+                 DontUseBuffer\r
+                 );\r
 \r
-    break;\r
+      break;\r
 \r
-  case EFI_PXE_BASE_CODE_MTFTP_GET_FILE_SIZE:\r
-  case EFI_PXE_BASE_CODE_MTFTP_READ_FILE:\r
-  case EFI_PXE_BASE_CODE_MTFTP_READ_DIRECTORY:\r
-    Status = EFI_UNSUPPORTED;\r
+    case EFI_PXE_BASE_CODE_MTFTP_GET_FILE_SIZE:\r
+    case EFI_PXE_BASE_CODE_MTFTP_READ_FILE:\r
+    case EFI_PXE_BASE_CODE_MTFTP_READ_DIRECTORY:\r
+      Status = EFI_UNSUPPORTED;\r
 \r
-    break;\r
+      break;\r
 \r
-  default:\r
-    Status = EFI_INVALID_PARAMETER;\r
+    default:\r
+      Status = EFI_INVALID_PARAMETER;\r
 \r
-    break;\r
+      break;\r
   }\r
 \r
   if (Status == EFI_ICMP_ERROR) {\r
@@ -1010,18 +1012,18 @@ EfiPxeBcMtftp (
   } else {\r
     Private->Udp4Read->Configure (Private->Udp4Read, &Private->Udp4CfgData);\r
   }\r
+\r
   //\r
   // Dhcp(), Discover(), and Mtftp() set the IP filter, and return with the IP\r
   // receive filter list emptied and the filter set to EFI_PXE_BASE_CODE_IP_FILTER_STATION_IP.\r
   //\r
-  ZeroMem(&IpFilter, sizeof (EFI_PXE_BASE_CODE_IP_FILTER));\r
+  ZeroMem (&IpFilter, sizeof (EFI_PXE_BASE_CODE_IP_FILTER));\r
   IpFilter.Filters = EFI_PXE_BASE_CODE_IP_FILTER_STATION_IP;\r
   This->SetIpFilter (This, &IpFilter);\r
 \r
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Writes a UDP packet to the network interface.\r
 \r
@@ -1062,27 +1064,27 @@ EfiPxeBcMtftp (
 EFI_STATUS\r
 EFIAPI\r
 EfiPxeBcUdpWrite (\r
-  IN     EFI_PXE_BASE_CODE_PROTOCOL       *This,\r
-  IN     UINT16                           OpFlags,\r
-  IN     EFI_IP_ADDRESS                   *DestIp,\r
-  IN     EFI_PXE_BASE_CODE_UDP_PORT       *DestPort,\r
-  IN     EFI_IP_ADDRESS                   *GatewayIp  OPTIONAL,\r
-  IN     EFI_IP_ADDRESS                   *SrcIp      OPTIONAL,\r
-  IN OUT EFI_PXE_BASE_CODE_UDP_PORT       *SrcPort    OPTIONAL,\r
-  IN     UINTN                            *HeaderSize OPTIONAL,\r
-  IN     VOID                             *HeaderPtr  OPTIONAL,\r
-  IN     UINTN                            *BufferSize,\r
-  IN     VOID                             *BufferPtr\r
+  IN     EFI_PXE_BASE_CODE_PROTOCOL  *This,\r
+  IN     UINT16                      OpFlags,\r
+  IN     EFI_IP_ADDRESS              *DestIp,\r
+  IN     EFI_PXE_BASE_CODE_UDP_PORT  *DestPort,\r
+  IN     EFI_IP_ADDRESS              *GatewayIp  OPTIONAL,\r
+  IN     EFI_IP_ADDRESS              *SrcIp      OPTIONAL,\r
+  IN OUT EFI_PXE_BASE_CODE_UDP_PORT  *SrcPort    OPTIONAL,\r
+  IN     UINTN                       *HeaderSize OPTIONAL,\r
+  IN     VOID                        *HeaderPtr  OPTIONAL,\r
+  IN     UINTN                       *BufferSize,\r
+  IN     VOID                        *BufferPtr\r
   )\r
 {\r
-  PXEBC_PRIVATE_DATA        *Private;\r
-  EFI_PXE_BASE_CODE_MODE    *Mode;\r
-  EFI_UDP4_SESSION_DATA     Udp4Session;\r
-  EFI_UDP6_SESSION_DATA     Udp6Session;\r
-  EFI_STATUS                Status;\r
-  BOOLEAN                   DoNotFragment;\r
-\r
-  if (This == NULL || DestIp == NULL || DestPort == NULL) {\r
+  PXEBC_PRIVATE_DATA      *Private;\r
+  EFI_PXE_BASE_CODE_MODE  *Mode;\r
+  EFI_UDP4_SESSION_DATA   Udp4Session;\r
+  EFI_UDP6_SESSION_DATA   Udp6Session;\r
+  EFI_STATUS              Status;\r
+  BOOLEAN                 DoNotFragment;\r
+\r
+  if ((This == NULL) || (DestIp == NULL) || (DestPort == NULL)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -1095,19 +1097,20 @@ EfiPxeBcUdpWrite (
     DoNotFragment = TRUE;\r
   }\r
 \r
-  if (!Mode->UsingIpv6 && GatewayIp != NULL && Mode->SubnetMask.Addr[0] != 0 &&\r
-      !NetIp4IsUnicast (NTOHL (GatewayIp->Addr[0]), EFI_NTOHL(Mode->SubnetMask))) {\r
+  if (!Mode->UsingIpv6 && (GatewayIp != NULL) && (Mode->SubnetMask.Addr[0] != 0) &&\r
+      !NetIp4IsUnicast (NTOHL (GatewayIp->Addr[0]), EFI_NTOHL (Mode->SubnetMask)))\r
+  {\r
     //\r
     // Gateway is provided but it's not a unicast IPv4 address, while it will be ignored for IPv6.\r
     //\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
-  if (HeaderSize != NULL && (*HeaderSize == 0 || HeaderPtr == NULL)) {\r
+  if ((HeaderSize != NULL) && ((*HeaderSize == 0) || (HeaderPtr == NULL))) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
-  if (BufferSize == NULL || (*BufferSize != 0 && BufferPtr == NULL)) {\r
+  if ((BufferSize == NULL) || ((*BufferSize != 0) && (BufferPtr == NULL))) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -1115,12 +1118,13 @@ EfiPxeBcUdpWrite (
     return EFI_NOT_STARTED;\r
   }\r
 \r
-  if (!Private->IsAddressOk && SrcIp == NULL) {\r
+  if (!Private->IsAddressOk && (SrcIp == NULL)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
-  if (Private->CurSrcPort == 0 ||\r
-      (SrcPort != NULL && *SrcPort != Private->CurSrcPort)) {\r
+  if ((Private->CurSrcPort == 0) ||\r
+      ((SrcPort != NULL) && (*SrcPort != Private->CurSrcPort)))\r
+  {\r
     //\r
     // Reconfigure UDPv4/UDPv6 for UdpWrite if the source port changed.\r
     //\r
@@ -1173,6 +1177,7 @@ EfiPxeBcUdpWrite (
     if (SrcIp != NULL) {\r
       CopyMem (&Udp6Session.SourceAddress, SrcIp, sizeof (EFI_IPv6_ADDRESS));\r
     }\r
+\r
     if (SrcPort != NULL) {\r
       Udp6Session.SourcePort = *SrcPort;\r
     }\r
@@ -1196,9 +1201,11 @@ EfiPxeBcUdpWrite (
     if (SrcIp != NULL) {\r
       CopyMem (&Udp4Session.SourceAddress, SrcIp, sizeof (EFI_IPv4_ADDRESS));\r
     }\r
+\r
     if (SrcPort != NULL) {\r
       Udp4Session.SourcePort = *SrcPort;\r
     }\r
+\r
     //\r
     // Override the gateway information if user specified.\r
     //\r
@@ -1206,7 +1213,7 @@ EfiPxeBcUdpWrite (
                Private->Udp4Write,\r
                &Udp4Session,\r
                Private->UdpTimeOutEvent,\r
-               (EFI_IPv4_ADDRESS *) GatewayIp,\r
+               (EFI_IPv4_ADDRESS *)GatewayIp,\r
                HeaderSize,\r
                HeaderPtr,\r
                BufferSize,\r
@@ -1216,7 +1223,6 @@ EfiPxeBcUdpWrite (
 \r
   gBS->SetTimer (Private->UdpTimeOutEvent, TimerCancel, 0);\r
 \r
-\r
   //\r
   // Reset the UdpWrite instance.\r
   //\r
@@ -1229,7 +1235,6 @@ EfiPxeBcUdpWrite (
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Reads a UDP packet from the network interface.\r
 +\r
@@ -1270,34 +1275,34 @@ EfiPxeBcUdpWrite (
 EFI_STATUS\r
 EFIAPI\r
 EfiPxeBcUdpRead (\r
-  IN     EFI_PXE_BASE_CODE_PROTOCOL   *This,\r
-  IN     UINT16                       OpFlags,\r
-  IN OUT EFI_IP_ADDRESS               *DestIp      OPTIONAL,\r
-  IN OUT EFI_PXE_BASE_CODE_UDP_PORT   *DestPort    OPTIONAL,\r
-  IN OUT EFI_IP_ADDRESS               *SrcIp       OPTIONAL,\r
-  IN OUT EFI_PXE_BASE_CODE_UDP_PORT   *SrcPort     OPTIONAL,\r
-  IN     UINTN                        *HeaderSize  OPTIONAL,\r
-  IN     VOID                         *HeaderPtr   OPTIONAL,\r
-  IN OUT UINTN                        *BufferSize,\r
-  IN     VOID                         *BufferPtr\r
+  IN     EFI_PXE_BASE_CODE_PROTOCOL  *This,\r
+  IN     UINT16                      OpFlags,\r
+  IN OUT EFI_IP_ADDRESS              *DestIp      OPTIONAL,\r
+  IN OUT EFI_PXE_BASE_CODE_UDP_PORT  *DestPort    OPTIONAL,\r
+  IN OUT EFI_IP_ADDRESS              *SrcIp       OPTIONAL,\r
+  IN OUT EFI_PXE_BASE_CODE_UDP_PORT  *SrcPort     OPTIONAL,\r
+  IN     UINTN                       *HeaderSize  OPTIONAL,\r
+  IN     VOID                        *HeaderPtr   OPTIONAL,\r
+  IN OUT UINTN                       *BufferSize,\r
+  IN     VOID                        *BufferPtr\r
   )\r
 {\r
-  PXEBC_PRIVATE_DATA          *Private;\r
-  EFI_PXE_BASE_CODE_MODE      *Mode;\r
-  EFI_UDP4_COMPLETION_TOKEN   Udp4Token;\r
-  EFI_UDP6_COMPLETION_TOKEN   Udp6Token;\r
-  EFI_UDP4_RECEIVE_DATA       *Udp4Rx;\r
-  EFI_UDP6_RECEIVE_DATA       *Udp6Rx;\r
-  EFI_STATUS                  Status;\r
-  BOOLEAN                     IsDone;\r
-  BOOLEAN                     IsMatched;\r
-  UINTN                       CopiedLen;\r
-  UINTN                       HeaderLen;\r
-  UINTN                       HeaderCopiedLen;\r
-  UINTN                       BufferCopiedLen;\r
-  UINT32                      FragmentLength;\r
-  UINTN                       FragmentIndex;\r
-  UINT8                       *FragmentBuffer;\r
+  PXEBC_PRIVATE_DATA         *Private;\r
+  EFI_PXE_BASE_CODE_MODE     *Mode;\r
+  EFI_UDP4_COMPLETION_TOKEN  Udp4Token;\r
+  EFI_UDP6_COMPLETION_TOKEN  Udp6Token;\r
+  EFI_UDP4_RECEIVE_DATA      *Udp4Rx;\r
+  EFI_UDP6_RECEIVE_DATA      *Udp6Rx;\r
+  EFI_STATUS                 Status;\r
+  BOOLEAN                    IsDone;\r
+  BOOLEAN                    IsMatched;\r
+  UINTN                      CopiedLen;\r
+  UINTN                      HeaderLen;\r
+  UINTN                      HeaderCopiedLen;\r
+  UINTN                      BufferCopiedLen;\r
+  UINT32                     FragmentLength;\r
+  UINTN                      FragmentIndex;\r
+  UINT8                      *FragmentBuffer;\r
 \r
   if (This == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
@@ -1310,13 +1315,14 @@ EfiPxeBcUdpRead (
   Udp4Rx    = NULL;\r
   Udp6Rx    = NULL;\r
 \r
-  if (((OpFlags & EFI_PXE_BASE_CODE_UDP_OPFLAGS_ANY_DEST_PORT) == 0 && DestPort == NULL) ||\r
-      ((OpFlags & EFI_PXE_BASE_CODE_UDP_OPFLAGS_ANY_SRC_IP) == 0 && SrcIp == NULL) ||\r
-      ((OpFlags & EFI_PXE_BASE_CODE_UDP_OPFLAGS_ANY_SRC_PORT) == 0 && SrcPort == NULL)) {\r
+  if ((((OpFlags & EFI_PXE_BASE_CODE_UDP_OPFLAGS_ANY_DEST_PORT) == 0) && (DestPort == NULL)) ||\r
+      (((OpFlags & EFI_PXE_BASE_CODE_UDP_OPFLAGS_ANY_SRC_IP) == 0) && (SrcIp == NULL)) ||\r
+      (((OpFlags & EFI_PXE_BASE_CODE_UDP_OPFLAGS_ANY_SRC_PORT) == 0) && (SrcPort == NULL)))\r
+  {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
-  if ((HeaderSize != NULL && *HeaderSize == 0) || (HeaderSize != NULL && HeaderPtr == NULL)) {\r
+  if (((HeaderSize != NULL) && (*HeaderSize == 0)) || ((HeaderSize != NULL) && (HeaderPtr == NULL))) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -1396,16 +1402,18 @@ EfiPxeBcUdpRead (
     }\r
   }\r
 \r
-  if (Status == EFI_ICMP_ERROR ||\r
-      Status == EFI_NETWORK_UNREACHABLE ||\r
-      Status == EFI_HOST_UNREACHABLE ||\r
-      Status == EFI_PROTOCOL_UNREACHABLE ||\r
-      Status == EFI_PORT_UNREACHABLE) {\r
+  if ((Status == EFI_ICMP_ERROR) ||\r
+      (Status == EFI_NETWORK_UNREACHABLE) ||\r
+      (Status == EFI_HOST_UNREACHABLE) ||\r
+      (Status == EFI_PROTOCOL_UNREACHABLE) ||\r
+      (Status == EFI_PORT_UNREACHABLE))\r
+  {\r
     //\r
     // Get different return status for icmp error from Udp, refers to UEFI spec.\r
     //\r
     Mode->IcmpErrorReceived = TRUE;\r
   }\r
+\r
   gBS->SetTimer (Private->UdpTimeOutEvent, TimerCancel, 0);\r
 \r
   if (IsMatched) {\r
@@ -1427,6 +1435,7 @@ EfiPxeBcUdpRead (
         if (HeaderSize != NULL) {\r
           *HeaderSize = HeaderLen;\r
         }\r
+\r
         *BufferSize = Udp6Rx->DataLength - HeaderLen;\r
 \r
         HeaderCopiedLen = 0;\r
@@ -1438,30 +1447,31 @@ EfiPxeBcUdpRead (
             //\r
             // Copy the header part of received data.\r
             //\r
-            CopyMem ((UINT8 *) HeaderPtr + HeaderCopiedLen, FragmentBuffer, FragmentLength);\r
+            CopyMem ((UINT8 *)HeaderPtr + HeaderCopiedLen, FragmentBuffer, FragmentLength);\r
             HeaderCopiedLen += FragmentLength;\r
           } else if (HeaderCopiedLen < HeaderLen) {\r
             //\r
             // Copy the header part of received data.\r
             //\r
             CopiedLen = HeaderLen - HeaderCopiedLen;\r
-            CopyMem ((UINT8 *) HeaderPtr + HeaderCopiedLen, FragmentBuffer, CopiedLen);\r
+            CopyMem ((UINT8 *)HeaderPtr + HeaderCopiedLen, FragmentBuffer, CopiedLen);\r
             HeaderCopiedLen += CopiedLen;\r
 \r
             //\r
             // Copy the other part of received data.\r
             //\r
-            CopyMem ((UINT8 *) BufferPtr + BufferCopiedLen, FragmentBuffer + CopiedLen, FragmentLength - CopiedLen);\r
+            CopyMem ((UINT8 *)BufferPtr + BufferCopiedLen, FragmentBuffer + CopiedLen, FragmentLength - CopiedLen);\r
             BufferCopiedLen += (FragmentLength - CopiedLen);\r
           } else {\r
             //\r
             // Copy the other part of received data.\r
             //\r
-            CopyMem ((UINT8 *) BufferPtr + BufferCopiedLen, FragmentBuffer, FragmentLength);\r
+            CopyMem ((UINT8 *)BufferPtr + BufferCopiedLen, FragmentBuffer, FragmentLength);\r
             BufferCopiedLen += FragmentLength;\r
           }\r
         }\r
       }\r
+\r
       //\r
       // Recycle the receiving buffer after copy to user.\r
       //\r
@@ -1481,6 +1491,7 @@ EfiPxeBcUdpRead (
         if (HeaderSize != NULL) {\r
           *HeaderSize = HeaderLen;\r
         }\r
+\r
         *BufferSize = Udp4Rx->DataLength - HeaderLen;\r
 \r
         HeaderCopiedLen = 0;\r
@@ -1492,30 +1503,31 @@ EfiPxeBcUdpRead (
             //\r
             // Copy the header part of received data.\r
             //\r
-            CopyMem ((UINT8 *) HeaderPtr + HeaderCopiedLen, FragmentBuffer, FragmentLength);\r
+            CopyMem ((UINT8 *)HeaderPtr + HeaderCopiedLen, FragmentBuffer, FragmentLength);\r
             HeaderCopiedLen += FragmentLength;\r
           } else if (HeaderCopiedLen < HeaderLen) {\r
             //\r
             // Copy the header part of received data.\r
             //\r
             CopiedLen = HeaderLen - HeaderCopiedLen;\r
-            CopyMem ((UINT8 *) HeaderPtr + HeaderCopiedLen, FragmentBuffer, CopiedLen);\r
+            CopyMem ((UINT8 *)HeaderPtr + HeaderCopiedLen, FragmentBuffer, CopiedLen);\r
             HeaderCopiedLen += CopiedLen;\r
 \r
             //\r
             // Copy the other part of received data.\r
             //\r
-            CopyMem ((UINT8 *) BufferPtr + BufferCopiedLen, FragmentBuffer + CopiedLen, FragmentLength - CopiedLen);\r
+            CopyMem ((UINT8 *)BufferPtr + BufferCopiedLen, FragmentBuffer + CopiedLen, FragmentLength - CopiedLen);\r
             BufferCopiedLen += (FragmentLength - CopiedLen);\r
           } else {\r
             //\r
             // Copy the other part of received data.\r
             //\r
-            CopyMem ((UINT8 *) BufferPtr + BufferCopiedLen, FragmentBuffer, FragmentLength);\r
+            CopyMem ((UINT8 *)BufferPtr + BufferCopiedLen, FragmentBuffer, FragmentLength);\r
             BufferCopiedLen += FragmentLength;\r
           }\r
         }\r
       }\r
+\r
       //\r
       // Recycle the receiving buffer after copy to user.\r
       //\r
@@ -1534,7 +1546,6 @@ EfiPxeBcUdpRead (
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Updates the IP receive filters of a network device and enables software filtering.\r
 \r
@@ -1570,22 +1581,22 @@ EfiPxeBcUdpRead (
 EFI_STATUS\r
 EFIAPI\r
 EfiPxeBcSetIpFilter (\r
-  IN EFI_PXE_BASE_CODE_PROTOCOL       *This,\r
-  IN EFI_PXE_BASE_CODE_IP_FILTER      *NewFilter\r
+  IN EFI_PXE_BASE_CODE_PROTOCOL   *This,\r
+  IN EFI_PXE_BASE_CODE_IP_FILTER  *NewFilter\r
   )\r
 {\r
-  EFI_STATUS                Status;\r
-  PXEBC_PRIVATE_DATA        *Private;\r
-  EFI_PXE_BASE_CODE_MODE    *Mode;\r
-  EFI_UDP4_CONFIG_DATA      *Udp4Cfg;\r
-  EFI_UDP6_CONFIG_DATA      *Udp6Cfg;\r
-  UINTN                     Index;\r
-  BOOLEAN                   NeedPromiscuous;\r
-  BOOLEAN                   AcceptPromiscuous;\r
-  BOOLEAN                   AcceptBroadcast;\r
-  BOOLEAN                   MultiCastUpdate;\r
-\r
-  if (This == NULL || NewFilter == NULL) {\r
+  EFI_STATUS              Status;\r
+  PXEBC_PRIVATE_DATA      *Private;\r
+  EFI_PXE_BASE_CODE_MODE  *Mode;\r
+  EFI_UDP4_CONFIG_DATA    *Udp4Cfg;\r
+  EFI_UDP6_CONFIG_DATA    *Udp6Cfg;\r
+  UINTN                   Index;\r
+  BOOLEAN                 NeedPromiscuous;\r
+  BOOLEAN                 AcceptPromiscuous;\r
+  BOOLEAN                 AcceptBroadcast;\r
+  BOOLEAN                 MultiCastUpdate;\r
+\r
+  if ((This == NULL) || (NewFilter == NULL)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -1601,22 +1612,26 @@ EfiPxeBcSetIpFilter (
   for (Index = 0; Index < NewFilter->IpCnt; Index++) {\r
     ASSERT (Index < EFI_PXE_BASE_CODE_MAX_IPCNT);\r
     if (!Mode->UsingIpv6 &&\r
-        IP4_IS_LOCAL_BROADCAST (EFI_IP4 (NewFilter->IpList[Index].v4))) {\r
+        IP4_IS_LOCAL_BROADCAST (EFI_IP4 (NewFilter->IpList[Index].v4)))\r
+    {\r
       //\r
       // IPv4 broadcast address should not be in IP filter.\r
       //\r
       return EFI_INVALID_PARAMETER;\r
     }\r
+\r
     if (Mode->UsingIpv6) {\r
-      if ((NewFilter->Filters & EFI_PXE_BASE_CODE_IP_FILTER_STATION_IP) != 0 &&\r
-          NetIp6IsValidUnicast (&NewFilter->IpList[Index].v6)) {\r
+      if (((NewFilter->Filters & EFI_PXE_BASE_CODE_IP_FILTER_STATION_IP) != 0) &&\r
+          NetIp6IsValidUnicast (&NewFilter->IpList[Index].v6))\r
+      {\r
         NeedPromiscuous = TRUE;\r
       }\r
-    } else if ((EFI_NTOHL(Mode->StationIp) != 0) &&\r
-               (EFI_NTOHL(Mode->SubnetMask) != 0) &&\r
-               IP4_NET_EQUAL(EFI_NTOHL(Mode->StationIp), EFI_NTOHL(NewFilter->IpList[Index].v4), EFI_NTOHL(Mode->SubnetMask.v4)) &&\r
-               NetIp4IsUnicast (EFI_IP4 (NewFilter->IpList[Index].v4), EFI_NTOHL(Mode->SubnetMask)) &&\r
-               ((NewFilter->Filters & EFI_PXE_BASE_CODE_IP_FILTER_STATION_IP) != 0)) {\r
+    } else if ((EFI_NTOHL (Mode->StationIp) != 0) &&\r
+               (EFI_NTOHL (Mode->SubnetMask) != 0) &&\r
+               IP4_NET_EQUAL (EFI_NTOHL (Mode->StationIp), EFI_NTOHL (NewFilter->IpList[Index].v4), EFI_NTOHL (Mode->SubnetMask.v4)) &&\r
+               NetIp4IsUnicast (EFI_IP4 (NewFilter->IpList[Index].v4), EFI_NTOHL (Mode->SubnetMask)) &&\r
+               ((NewFilter->Filters & EFI_PXE_BASE_CODE_IP_FILTER_STATION_IP) != 0))\r
+    {\r
       NeedPromiscuous = TRUE;\r
     }\r
   }\r
@@ -1626,8 +1641,9 @@ EfiPxeBcSetIpFilter (
   MultiCastUpdate   = FALSE;\r
 \r
   if (NeedPromiscuous ||\r
-      (NewFilter->Filters & EFI_PXE_BASE_CODE_IP_FILTER_PROMISCUOUS) != 0 ||\r
-      (NewFilter->Filters & EFI_PXE_BASE_CODE_IP_FILTER_PROMISCUOUS_MULTICAST) != 0) {\r
+      ((NewFilter->Filters & EFI_PXE_BASE_CODE_IP_FILTER_PROMISCUOUS) != 0) ||\r
+      ((NewFilter->Filters & EFI_PXE_BASE_CODE_IP_FILTER_PROMISCUOUS_MULTICAST) != 0))\r
+  {\r
     //\r
     // Configure UDPv4/UDPv6 as promiscuous mode to receive all packets.\r
     //\r
@@ -1636,7 +1652,7 @@ EfiPxeBcSetIpFilter (
     //\r
     // Configure UDPv4 to receive all broadcast packets.\r
     //\r
-    AcceptBroadcast  = TRUE;\r
+    AcceptBroadcast = TRUE;\r
   }\r
 \r
   //\r
@@ -1661,7 +1677,8 @@ EfiPxeBcSetIpFilter (
     //\r
     Udp4Cfg = &Private->Udp4CfgData;\r
     if ((AcceptPromiscuous != Udp4Cfg->AcceptPromiscuous)   ||\r
-        (AcceptBroadcast != Udp4Cfg->AcceptBroadcast)     || MultiCastUpdate) {\r
+        (AcceptBroadcast != Udp4Cfg->AcceptBroadcast)     || MultiCastUpdate)\r
+    {\r
       //\r
       // Clear the UDP4 instance configuration, all joined groups will be left\r
       // during the operation.\r
@@ -1673,7 +1690,7 @@ EfiPxeBcSetIpFilter (
       //\r
       Udp4Cfg->AcceptPromiscuous = AcceptPromiscuous;\r
       Udp4Cfg->AcceptBroadcast   = AcceptBroadcast;\r
-      Status = Private->Udp4Read->Configure (Private->Udp4Read, Udp4Cfg);\r
+      Status                     = Private->Udp4Read->Configure (Private->Udp4Read, Udp4Cfg);\r
       if (EFI_ERROR (Status)) {\r
         return Status;\r
       }\r
@@ -1711,7 +1728,7 @@ EfiPxeBcSetIpFilter (
       // Configure the UDP instance with the new configuration.\r
       //\r
       Udp6Cfg->AcceptPromiscuous = AcceptPromiscuous;\r
-      Status = Private->Udp6Read->Configure (Private->Udp6Read, Udp6Cfg);\r
+      Status                     = Private->Udp6Read->Configure (Private->Udp6Read, Udp6Cfg);\r
       if (EFI_ERROR (Status)) {\r
         return Status;\r
       }\r
@@ -1743,7 +1760,6 @@ EfiPxeBcSetIpFilter (
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Uses the ARP protocol to resolve a MAC address. It is not supported for IPv6.\r
 \r
@@ -1772,9 +1788,9 @@ EfiPxeBcSetIpFilter (
 EFI_STATUS\r
 EFIAPI\r
 EfiPxeBcArp (\r
-  IN EFI_PXE_BASE_CODE_PROTOCOL       *This,\r
-  IN EFI_IP_ADDRESS                   *IpAddr,\r
-  IN EFI_MAC_ADDRESS                  *MacAddr OPTIONAL\r
+  IN EFI_PXE_BASE_CODE_PROTOCOL  *This,\r
+  IN EFI_IP_ADDRESS              *IpAddr,\r
+  IN EFI_MAC_ADDRESS             *MacAddr OPTIONAL\r
   )\r
 {\r
   PXEBC_PRIVATE_DATA      *Private;\r
@@ -1785,7 +1801,7 @@ EfiPxeBcArp (
   EFI_MAC_ADDRESS         ZeroMac;\r
   BOOLEAN                 IsResolved;\r
 \r
-  if (This == NULL || IpAddr == NULL) {\r
+  if ((This == NULL) || (IpAddr == NULL)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -1839,7 +1855,7 @@ EfiPxeBcArp (
     // If AutoArp is TRUE, try to send Arp request on initiative.\r
     //\r
     Status = Private->Arp->Request (Private->Arp, &IpAddr->v4, ResolvedEvent, &TempMac);\r
-    if (EFI_ERROR (Status) && Status != EFI_NOT_READY) {\r
+    if (EFI_ERROR (Status) && (Status != EFI_NOT_READY)) {\r
       goto ON_EXIT;\r
     }\r
 \r
@@ -1848,6 +1864,7 @@ EfiPxeBcArp (
         break;\r
       }\r
     }\r
+\r
     if (CompareMem (&TempMac, &ZeroMac, sizeof (EFI_MAC_ADDRESS)) != 0) {\r
       Status = EFI_SUCCESS;\r
     } else {\r
@@ -1858,7 +1875,7 @@ EfiPxeBcArp (
   //\r
   // Copy the Mac address to user if needed.\r
   //\r
-  if (MacAddr != NULL && !EFI_ERROR (Status)) {\r
+  if ((MacAddr != NULL) && !EFI_ERROR (Status)) {\r
     CopyMem (MacAddr, &TempMac, sizeof (EFI_MAC_ADDRESS));\r
   }\r
 \r
@@ -1866,10 +1883,10 @@ ON_EXIT:
   if (ResolvedEvent != NULL) {\r
     gBS->CloseEvent (ResolvedEvent);\r
   }\r
+\r
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Updates the parameters that affect the operation of the PXE Base Code Protocol.\r
 \r
@@ -1905,12 +1922,12 @@ ON_EXIT:
 EFI_STATUS\r
 EFIAPI\r
 EfiPxeBcSetParameters (\r
-  IN EFI_PXE_BASE_CODE_PROTOCOL       *This,\r
-  IN BOOLEAN                          *NewAutoArp         OPTIONAL,\r
-  IN BOOLEAN                          *NewSendGUID        OPTIONAL,\r
-  IN UINT8                            *NewTTL             OPTIONAL,\r
-  IN UINT8                            *NewToS             OPTIONAL,\r
-  IN BOOLEAN                          *NewMakeCallback    OPTIONAL\r
+  IN EFI_PXE_BASE_CODE_PROTOCOL  *This,\r
+  IN BOOLEAN                     *NewAutoArp         OPTIONAL,\r
+  IN BOOLEAN                     *NewSendGUID        OPTIONAL,\r
+  IN UINT8                       *NewTTL             OPTIONAL,\r
+  IN UINT8                       *NewToS             OPTIONAL,\r
+  IN BOOLEAN                     *NewMakeCallback    OPTIONAL\r
   )\r
 {\r
   PXEBC_PRIVATE_DATA      *Private;\r
@@ -1937,7 +1954,7 @@ EfiPxeBcSetParameters (
       Status = gBS->HandleProtocol (\r
                       Mode->UsingIpv6 ? Private->Ip6Nic->Controller : Private->Ip4Nic->Controller,\r
                       &gEfiPxeBaseCodeCallbackProtocolGuid,\r
-                      (VOID **) &Private->PxeBcCallback\r
+                      (VOID **)&Private->PxeBcCallback\r
                       );\r
 \r
       if (EFI_ERROR (Status) || (Private->PxeBcCallback->Callback == NULL)) {\r
@@ -1946,6 +1963,7 @@ EfiPxeBcSetParameters (
     } else {\r
       Private->PxeBcCallback = NULL;\r
     }\r
+\r
     Mode->MakeCallbacks = *NewMakeCallback;\r
   }\r
 \r
@@ -1954,6 +1972,7 @@ EfiPxeBcSetParameters (
       DEBUG ((DEBUG_WARN, "PXE: Failed to read system GUID from the smbios table!\n"));\r
       return EFI_INVALID_PARAMETER;\r
     }\r
+\r
     Mode->SendGUID = *NewSendGUID;\r
   }\r
 \r
@@ -1972,7 +1991,6 @@ EfiPxeBcSetParameters (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Updates the station IP address and/or subnet mask values of a network device.\r
 \r
@@ -1997,9 +2015,9 @@ EfiPxeBcSetParameters (
 EFI_STATUS\r
 EFIAPI\r
 EfiPxeBcSetStationIP (\r
-  IN EFI_PXE_BASE_CODE_PROTOCOL       *This,\r
-  IN EFI_IP_ADDRESS                   *NewStationIp    OPTIONAL,\r
-  IN EFI_IP_ADDRESS                   *NewSubnetMask   OPTIONAL\r
+  IN EFI_PXE_BASE_CODE_PROTOCOL  *This,\r
+  IN EFI_IP_ADDRESS              *NewStationIp    OPTIONAL,\r
+  IN EFI_IP_ADDRESS              *NewSubnetMask   OPTIONAL\r
   )\r
 {\r
   EFI_STATUS              Status;\r
@@ -2010,7 +2028,7 @@ EfiPxeBcSetStationIP (
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
-  if (NewStationIp != NULL && !NetIp6IsValidUnicast (&NewStationIp->v6)) {\r
+  if ((NewStationIp != NULL) && !NetIp6IsValidUnicast (&NewStationIp->v6)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -2019,15 +2037,17 @@ EfiPxeBcSetStationIP (
   Status  = EFI_SUCCESS;\r
 \r
   if (!Mode->UsingIpv6 &&\r
-      NewSubnetMask != NULL &&\r
-      !IP4_IS_VALID_NETMASK (NTOHL (NewSubnetMask->Addr[0]))) {\r
+      (NewSubnetMask != NULL) &&\r
+      !IP4_IS_VALID_NETMASK (NTOHL (NewSubnetMask->Addr[0])))\r
+  {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
-  if (!Mode->UsingIpv6 && NewStationIp != NULL) {\r
-    if (IP4_IS_UNSPECIFIED(NTOHL (NewStationIp->Addr[0])) ||\r
-        IP4_IS_LOCAL_BROADCAST(NTOHL (NewStationIp->Addr[0])) ||\r
-        (NewSubnetMask != NULL && NewSubnetMask->Addr[0] != 0 && !NetIp4IsUnicast (NTOHL (NewStationIp->Addr[0]), NTOHL (NewSubnetMask->Addr[0])))) {\r
+  if (!Mode->UsingIpv6 && (NewStationIp != NULL)) {\r
+    if (IP4_IS_UNSPECIFIED (NTOHL (NewStationIp->Addr[0])) ||\r
+        IP4_IS_LOCAL_BROADCAST (NTOHL (NewStationIp->Addr[0])) ||\r
+        ((NewSubnetMask != NULL) && (NewSubnetMask->Addr[0] != 0) && !NetIp4IsUnicast (NTOHL (NewStationIp->Addr[0]), NTOHL (NewSubnetMask->Addr[0]))))\r
+    {\r
       return EFI_INVALID_PARAMETER;\r
     }\r
   }\r
@@ -2036,7 +2056,7 @@ EfiPxeBcSetStationIP (
     return EFI_NOT_STARTED;\r
   }\r
 \r
-  if (Mode->UsingIpv6 && NewStationIp != NULL) {\r
+  if (Mode->UsingIpv6 && (NewStationIp != NULL)) {\r
     //\r
     // Set the IPv6 address by Ip6Config protocol.\r
     //\r
@@ -2051,9 +2071,9 @@ EfiPxeBcSetStationIP (
     CopyMem (&Private->StationIp, NewStationIp, sizeof (EFI_IP_ADDRESS));\r
   }\r
 \r
-  if (!Mode->UsingIpv6 && NewSubnetMask != NULL) {\r
+  if (!Mode->UsingIpv6 && (NewSubnetMask != NULL)) {\r
     CopyMem (&Mode->SubnetMask, NewSubnetMask, sizeof (EFI_IP_ADDRESS));\r
-    CopyMem (&Private->SubnetMask ,NewSubnetMask, sizeof (EFI_IP_ADDRESS));\r
+    CopyMem (&Private->SubnetMaskNewSubnetMask, sizeof (EFI_IP_ADDRESS));\r
   }\r
 \r
   Status = PxeBcFlushStationIp (Private, NewStationIp, NewSubnetMask);\r
@@ -2065,7 +2085,6 @@ ON_EXIT:
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Updates the contents of the cached DHCP and Discover packets.\r
 \r
@@ -2101,19 +2120,19 @@ ON_EXIT:
 EFI_STATUS\r
 EFIAPI\r
 EfiPxeBcSetPackets (\r
-  IN EFI_PXE_BASE_CODE_PROTOCOL       *This,\r
-  IN BOOLEAN                          *NewDhcpDiscoverValid      OPTIONAL,\r
-  IN BOOLEAN                          *NewDhcpAckReceived        OPTIONAL,\r
-  IN BOOLEAN                          *NewProxyOfferReceived     OPTIONAL,\r
-  IN BOOLEAN                          *NewPxeDiscoverValid       OPTIONAL,\r
-  IN BOOLEAN                          *NewPxeReplyReceived       OPTIONAL,\r
-  IN BOOLEAN                          *NewPxeBisReplyReceived    OPTIONAL,\r
-  IN EFI_PXE_BASE_CODE_PACKET         *NewDhcpDiscover           OPTIONAL,\r
-  IN EFI_PXE_BASE_CODE_PACKET         *NewDhcpAck                OPTIONAL,\r
-  IN EFI_PXE_BASE_CODE_PACKET         *NewProxyOffer             OPTIONAL,\r
-  IN EFI_PXE_BASE_CODE_PACKET         *NewPxeDiscover            OPTIONAL,\r
-  IN EFI_PXE_BASE_CODE_PACKET         *NewPxeReply               OPTIONAL,\r
-  IN EFI_PXE_BASE_CODE_PACKET         *NewPxeBisReply            OPTIONAL\r
+  IN EFI_PXE_BASE_CODE_PROTOCOL  *This,\r
+  IN BOOLEAN                     *NewDhcpDiscoverValid      OPTIONAL,\r
+  IN BOOLEAN                     *NewDhcpAckReceived        OPTIONAL,\r
+  IN BOOLEAN                     *NewProxyOfferReceived     OPTIONAL,\r
+  IN BOOLEAN                     *NewPxeDiscoverValid       OPTIONAL,\r
+  IN BOOLEAN                     *NewPxeReplyReceived       OPTIONAL,\r
+  IN BOOLEAN                     *NewPxeBisReplyReceived    OPTIONAL,\r
+  IN EFI_PXE_BASE_CODE_PACKET    *NewDhcpDiscover           OPTIONAL,\r
+  IN EFI_PXE_BASE_CODE_PACKET    *NewDhcpAck                OPTIONAL,\r
+  IN EFI_PXE_BASE_CODE_PACKET    *NewProxyOffer             OPTIONAL,\r
+  IN EFI_PXE_BASE_CODE_PACKET    *NewPxeDiscover            OPTIONAL,\r
+  IN EFI_PXE_BASE_CODE_PACKET    *NewPxeReply               OPTIONAL,\r
+  IN EFI_PXE_BASE_CODE_PACKET    *NewPxeBisReply            OPTIONAL\r
   )\r
 {\r
   PXEBC_PRIVATE_DATA      *Private;\r
@@ -2198,7 +2217,6 @@ EFI_PXE_BASE_CODE_PROTOCOL  gPxeBcProtocolTemplate = {
   NULL\r
 };\r
 \r
-\r
 /**\r
   Callback function that is invoked when the PXE Base Code Protocol is about to transmit, has\r
   received, or is waiting to receive a packet.\r
@@ -2236,8 +2254,8 @@ EfiPxeLoadFileCallback (
   IN EFI_PXE_BASE_CODE_PACKET             *PacketPtr     OPTIONAL\r
   )\r
 {\r
-  EFI_INPUT_KEY       Key;\r
-  EFI_STATUS          Status;\r
+  EFI_INPUT_KEY  Key;\r
+  EFI_STATUS     Status;\r
 \r
   //\r
   // Catch Ctrl-C or ESC to abort.\r
@@ -2245,43 +2263,43 @@ EfiPxeLoadFileCallback (
   Status = gST->ConIn->ReadKeyStroke (gST->ConIn, &Key);\r
 \r
   if (!EFI_ERROR (Status)) {\r
-\r
-    if (Key.ScanCode == SCAN_ESC || Key.UnicodeChar == (0x1F & 'c')) {\r
-\r
+    if ((Key.ScanCode == SCAN_ESC) || (Key.UnicodeChar == (0x1F & 'c'))) {\r
       return EFI_PXE_BASE_CODE_CALLBACK_STATUS_ABORT;\r
     }\r
   }\r
+\r
   //\r
   // No print if receive packet\r
   //\r
   if (Received) {\r
     return EFI_PXE_BASE_CODE_CALLBACK_STATUS_CONTINUE;\r
   }\r
+\r
   //\r
   // Print only for three functions\r
   //\r
   switch (Function) {\r
-\r
-  case EFI_PXE_BASE_CODE_FUNCTION_MTFTP:\r
-    //\r
-    // Print only for open MTFTP packets, not every MTFTP packets\r
-    //\r
-    if (PacketLength != 0 && PacketPtr != NULL) {\r
-      if (PacketPtr->Raw[0x1C] != 0x00 || PacketPtr->Raw[0x1D] != 0x01) {\r
-        return EFI_PXE_BASE_CODE_CALLBACK_STATUS_CONTINUE;\r
+    case EFI_PXE_BASE_CODE_FUNCTION_MTFTP:\r
+      //\r
+      // Print only for open MTFTP packets, not every MTFTP packets\r
+      //\r
+      if ((PacketLength != 0) && (PacketPtr != NULL)) {\r
+        if ((PacketPtr->Raw[0x1C] != 0x00) || (PacketPtr->Raw[0x1D] != 0x01)) {\r
+          return EFI_PXE_BASE_CODE_CALLBACK_STATUS_CONTINUE;\r
+        }\r
       }\r
-    }\r
-    break;\r
 \r
-  case EFI_PXE_BASE_CODE_FUNCTION_DHCP:\r
-  case EFI_PXE_BASE_CODE_FUNCTION_DISCOVER:\r
-    break;\r
+      break;\r
 \r
-  default:\r
-    return EFI_PXE_BASE_CODE_CALLBACK_STATUS_CONTINUE;\r
+    case EFI_PXE_BASE_CODE_FUNCTION_DHCP:\r
+    case EFI_PXE_BASE_CODE_FUNCTION_DISCOVER:\r
+      break;\r
+\r
+    default:\r
+      return EFI_PXE_BASE_CODE_CALLBACK_STATUS_CONTINUE;\r
   }\r
 \r
-  if (PacketLength != 0 && PacketPtr != NULL) {\r
+  if ((PacketLength != 0) && (PacketPtr != NULL)) {\r
     //\r
     // Print '.' when transmit a packet\r
     //\r
@@ -2291,12 +2309,11 @@ EfiPxeLoadFileCallback (
   return EFI_PXE_BASE_CODE_CALLBACK_STATUS_CONTINUE;\r
 }\r
 \r
-EFI_PXE_BASE_CODE_CALLBACK_PROTOCOL gPxeBcCallBackTemplate = {\r
+EFI_PXE_BASE_CODE_CALLBACK_PROTOCOL  gPxeBcCallBackTemplate = {\r
   EFI_PXE_BASE_CODE_CALLBACK_PROTOCOL_REVISION,\r
   EfiPxeLoadFileCallback\r
 };\r
 \r
-\r
 /**\r
   Causes the driver to load a specified file.\r
 \r
@@ -2328,11 +2345,11 @@ EFI_PXE_BASE_CODE_CALLBACK_PROTOCOL gPxeBcCallBackTemplate = {
 EFI_STATUS\r
 EFIAPI\r
 EfiPxeLoadFile (\r
-  IN     EFI_LOAD_FILE_PROTOCOL           *This,\r
-  IN     EFI_DEVICE_PATH_PROTOCOL         *FilePath,\r
-  IN     BOOLEAN                          BootPolicy,\r
-  IN OUT UINTN                            *BufferSize,\r
-  IN     VOID                             *Buffer       OPTIONAL\r
+  IN     EFI_LOAD_FILE_PROTOCOL    *This,\r
+  IN     EFI_DEVICE_PATH_PROTOCOL  *FilePath,\r
+  IN     BOOLEAN                   BootPolicy,\r
+  IN OUT UINTN                     *BufferSize,\r
+  IN     VOID                      *Buffer       OPTIONAL\r
   )\r
 {\r
   PXEBC_PRIVATE_DATA          *Private;\r
@@ -2342,7 +2359,7 @@ EfiPxeLoadFile (
   EFI_STATUS                  Status;\r
   EFI_STATUS                  MediaStatus;\r
 \r
-  if (This == NULL || BufferSize == NULL || FilePath == NULL || !IsDevicePathEnd (FilePath)) {\r
+  if ((This == NULL) || (BufferSize == NULL) || (FilePath == NULL) || !IsDevicePathEnd (FilePath)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -2379,7 +2396,7 @@ EfiPxeLoadFile (
   // Start Pxe Base Code to initialize PXE boot.\r
   //\r
   Status = PxeBc->Start (PxeBc, UsingIpv6);\r
-  if (Status == EFI_ALREADY_STARTED && UsingIpv6 != PxeBc->Mode->UsingIpv6) {\r
+  if ((Status == EFI_ALREADY_STARTED) && (UsingIpv6 != PxeBc->Mode->UsingIpv6)) {\r
     //\r
     // PxeBc protocol has already been started but not on the required IP version, restart it.\r
     //\r
@@ -2388,13 +2405,15 @@ EfiPxeLoadFile (
       Status = PxeBc->Start (PxeBc, UsingIpv6);\r
     }\r
   }\r
-  if (Status == EFI_SUCCESS || Status == EFI_ALREADY_STARTED) {\r
+\r
+  if ((Status == EFI_SUCCESS) || (Status == EFI_ALREADY_STARTED)) {\r
     Status = PxeBcLoadBootFile (Private, BufferSize, Buffer);\r
   }\r
 \r
-  if (Status != EFI_SUCCESS &&\r
-      Status != EFI_UNSUPPORTED &&\r
-      Status != EFI_BUFFER_TOO_SMALL) {\r
+  if ((Status != EFI_SUCCESS) &&\r
+      (Status != EFI_UNSUPPORTED) &&\r
+      (Status != EFI_BUFFER_TOO_SMALL))\r
+  {\r
     //\r
     // There are three cases, which needn't stop pxebc here.\r
     //   1. success to download file.\r