]> git.proxmox.com Git - mirror_edk2.git/blobdiff - NetworkPkg/HttpDxe/HttpImpl.c
NetworkPkg: Apply uncrustify changes
[mirror_edk2.git] / NetworkPkg / HttpDxe / HttpImpl.c
index 702482151516ac90dbaa958170f5bc69676bb9a1..d64cd9e965c0d890ea5ddc2e1b2f96e73a3c0924 100644 (file)
@@ -45,11 +45,11 @@ EFI_HTTP_PROTOCOL  mEfiHttpTemplate = {
 EFI_STATUS\r
 EFIAPI\r
 EfiHttpGetModeData (\r
-  IN  EFI_HTTP_PROTOCOL         *This,\r
-  OUT EFI_HTTP_CONFIG_DATA      *HttpConfigData\r
+  IN  EFI_HTTP_PROTOCOL     *This,\r
+  OUT EFI_HTTP_CONFIG_DATA  *HttpConfigData\r
   )\r
 {\r
-  HTTP_PROTOCOL                 *HttpInstance;\r
+  HTTP_PROTOCOL  *HttpInstance;\r
 \r
   //\r
   // Check input parameters.\r
@@ -61,7 +61,8 @@ EfiHttpGetModeData (
   HttpInstance = HTTP_INSTANCE_FROM_PROTOCOL (This);\r
 \r
   if ((HttpConfigData->AccessPoint.IPv6Node == NULL) ||\r
-      (HttpConfigData->AccessPoint.IPv4Node == NULL)) {\r
+      (HttpConfigData->AccessPoint.IPv4Node == NULL))\r
+  {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -78,7 +79,7 @@ EfiHttpGetModeData (
       HttpConfigData->AccessPoint.IPv6Node,\r
       &HttpInstance->Ipv6Node,\r
       sizeof (HttpInstance->Ipv6Node)\r
-    );\r
+      );\r
   } else {\r
     CopyMem (\r
       HttpConfigData->AccessPoint.IPv4Node,\r
@@ -124,20 +125,21 @@ EfiHttpGetModeData (
 EFI_STATUS\r
 EFIAPI\r
 EfiHttpConfigure (\r
-  IN  EFI_HTTP_PROTOCOL         *This,\r
-  IN  EFI_HTTP_CONFIG_DATA      *HttpConfigData OPTIONAL\r
+  IN  EFI_HTTP_PROTOCOL     *This,\r
+  IN  EFI_HTTP_CONFIG_DATA  *HttpConfigData OPTIONAL\r
   )\r
 {\r
-  HTTP_PROTOCOL                 *HttpInstance;\r
-  EFI_STATUS                    Status;\r
+  HTTP_PROTOCOL  *HttpInstance;\r
+  EFI_STATUS     Status;\r
 \r
   //\r
   // Check input parameters.\r
   //\r
-  if (This == NULL ||\r
-      (HttpConfigData != NULL &&\r
-       ((HttpConfigData->LocalAddressIsIPv6 && HttpConfigData->AccessPoint.IPv6Node == NULL) ||\r
-        (!HttpConfigData->LocalAddressIsIPv6 && HttpConfigData->AccessPoint.IPv4Node == NULL)))) {\r
+  if ((This == NULL) ||\r
+      ((HttpConfigData != NULL) &&\r
+       ((HttpConfigData->LocalAddressIsIPv6 && (HttpConfigData->AccessPoint.IPv6Node == NULL)) ||\r
+        (!HttpConfigData->LocalAddressIsIPv6 && (HttpConfigData->AccessPoint.IPv4Node == NULL)))))\r
+  {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -145,7 +147,6 @@ EfiHttpConfigure (
   ASSERT (HttpInstance->Service != NULL);\r
 \r
   if (HttpConfigData != NULL) {\r
-\r
     if (HttpConfigData->HttpVersion >= HttpVersionUnsupported) {\r
       return EFI_UNSUPPORTED;\r
     }\r
@@ -185,7 +186,6 @@ EfiHttpConfigure (
 \r
     HttpInstance->State = HTTP_STATE_HTTP_CONFIGED;\r
     return EFI_SUCCESS;\r
-\r
   } else {\r
     //\r
     // Reset all the resources related to HttpInstance.\r
@@ -196,7 +196,6 @@ EfiHttpConfigure (
   }\r
 }\r
 \r
-\r
 /**\r
   The Request() function queues an HTTP request to this HTTP instance.\r
 \r
@@ -226,41 +225,41 @@ EfiHttpConfigure (
 EFI_STATUS\r
 EFIAPI\r
 EfiHttpRequest (\r
-  IN  EFI_HTTP_PROTOCOL         *This,\r
-  IN  EFI_HTTP_TOKEN            *Token\r
+  IN  EFI_HTTP_PROTOCOL  *This,\r
+  IN  EFI_HTTP_TOKEN     *Token\r
   )\r
 {\r
-  EFI_HTTP_MESSAGE              *HttpMsg;\r
-  EFI_HTTP_REQUEST_DATA         *Request;\r
-  VOID                          *UrlParser;\r
-  EFI_STATUS                    Status;\r
-  CHAR8                         *HostName;\r
-  UINTN                         HostNameSize;\r
-  UINT16                        RemotePort;\r
-  HTTP_PROTOCOL                 *HttpInstance;\r
-  BOOLEAN                       Configure;\r
-  BOOLEAN                       ReConfigure;\r
-  BOOLEAN                       TlsConfigure;\r
-  CHAR8                         *RequestMsg;\r
-  CHAR8                         *Url;\r
-  UINTN                         UrlLen;\r
-  CHAR16                        *HostNameStr;\r
-  HTTP_TOKEN_WRAP               *Wrap;\r
-  CHAR8                         *FileUrl;\r
-  UINTN                         RequestMsgSize;\r
-  EFI_HANDLE                    ImageHandle;\r
+  EFI_HTTP_MESSAGE       *HttpMsg;\r
+  EFI_HTTP_REQUEST_DATA  *Request;\r
+  VOID                   *UrlParser;\r
+  EFI_STATUS             Status;\r
+  CHAR8                  *HostName;\r
+  UINTN                  HostNameSize;\r
+  UINT16                 RemotePort;\r
+  HTTP_PROTOCOL          *HttpInstance;\r
+  BOOLEAN                Configure;\r
+  BOOLEAN                ReConfigure;\r
+  BOOLEAN                TlsConfigure;\r
+  CHAR8                  *RequestMsg;\r
+  CHAR8                  *Url;\r
+  UINTN                  UrlLen;\r
+  CHAR16                 *HostNameStr;\r
+  HTTP_TOKEN_WRAP        *Wrap;\r
+  CHAR8                  *FileUrl;\r
+  UINTN                  RequestMsgSize;\r
+  EFI_HANDLE             ImageHandle;\r
 \r
   //\r
   // Initializations\r
   //\r
-  Url = NULL;\r
-  UrlParser = NULL;\r
-  RemotePort = 0;\r
-  HostName = NULL;\r
-  RequestMsg = NULL;\r
-  HostNameStr = NULL;\r
-  Wrap = NULL;\r
-  FileUrl = NULL;\r
+  Url          = NULL;\r
+  UrlParser    = NULL;\r
+  RemotePort   = 0;\r
+  HostName     = NULL;\r
+  RequestMsg   = NULL;\r
+  HostNameStr  = NULL;\r
+  Wrap         = NULL;\r
+  FileUrl      = NULL;\r
   TlsConfigure = FALSE;\r
 \r
   if ((This == NULL) || (Token == NULL)) {\r
@@ -280,7 +279,8 @@ EfiHttpRequest (
   if ((Request != NULL) && (Request->Method != HttpMethodGet) &&\r
       (Request->Method != HttpMethodHead) && (Request->Method != HttpMethodDelete) &&\r
       (Request->Method != HttpMethodPut) && (Request->Method != HttpMethodPost) &&\r
-      (Request->Method != HttpMethodPatch)) {\r
+      (Request->Method != HttpMethodPatch))\r
+  {\r
     return EFI_UNSUPPORTED;\r
   }\r
 \r
@@ -303,7 +303,8 @@ EfiHttpRequest (
     //\r
     if ((HttpInstance->Method != HttpMethodPut) &&\r
         (HttpInstance->Method != HttpMethodPost) &&\r
-        (HttpInstance->Method != HttpMethodPatch)) {\r
+        (HttpInstance->Method != HttpMethodPatch))\r
+    {\r
       return EFI_INVALID_PARAMETER;\r
     }\r
 \r
@@ -317,7 +318,7 @@ EfiHttpRequest (
     //\r
     // We need to have the Message Body for sending the HTTP message across in these cases.\r
     //\r
-    if (HttpMsg->Body == NULL || HttpMsg->BodyLength == 0) {\r
+    if ((HttpMsg->Body == NULL) || (HttpMsg->BodyLength == 0)) {\r
       return EFI_INVALID_PARAMETER;\r
     }\r
 \r
@@ -337,18 +338,18 @@ EfiHttpRequest (
     //\r
     // Parse the URI of the remote host.\r
     //\r
-    Url = HttpInstance->Url;\r
+    Url    = HttpInstance->Url;\r
     UrlLen = StrLen (Request->Url) + 1;\r
     if (UrlLen > HTTP_URL_BUFFER_LEN) {\r
       Url = AllocateZeroPool (UrlLen);\r
       if (Url == NULL) {\r
         return EFI_OUT_OF_RESOURCES;\r
       }\r
+\r
       FreePool (HttpInstance->Url);\r
       HttpInstance->Url = Url;\r
     }\r
 \r
-\r
     UnicodeStrToAsciiStrS (Request->Url, Url, UrlLen);\r
 \r
     //\r
@@ -361,7 +362,6 @@ EfiHttpRequest (
     // HTTP is disabled, return directly if the URI is not HTTPS.\r
     //\r
     if (!PcdGetBool (PcdAllowHttpConnections) && !(HttpInstance->UseHttps)) {\r
-\r
       DEBUG ((DEBUG_ERROR, "EfiHttpRequest: HTTP is disabled.\n"));\r
 \r
       return EFI_ACCESS_DENIED;\r
@@ -370,7 +370,7 @@ EfiHttpRequest (
     //\r
     // Check whether we need to create Tls child and open the TLS protocol.\r
     //\r
-    if (HttpInstance->UseHttps && HttpInstance->TlsChildHandle == NULL) {\r
+    if (HttpInstance->UseHttps && (HttpInstance->TlsChildHandle == NULL)) {\r
       //\r
       // Use TlsSb to create Tls child and open the TLS protocol.\r
       //\r
@@ -394,7 +394,7 @@ EfiHttpRequest (
     }\r
 \r
     UrlParser = NULL;\r
-    Status = HttpParseUrl (Url, (UINT32) AsciiStrLen (Url), FALSE, &UrlParser);\r
+    Status    = HttpParseUrl (Url, (UINT32)AsciiStrLen (Url), FALSE, &UrlParser);\r
     if (EFI_ERROR (Status)) {\r
       goto Error1;\r
     }\r
@@ -407,7 +407,7 @@ EfiHttpRequest (
     if (HttpInstance->LocalAddressIsIPv6) {\r
       HostNameSize = AsciiStrSize (HostName);\r
 \r
-      if (HostNameSize > 2 && HostName[0] == '[' && HostName[HostNameSize - 2] == ']') {\r
+      if ((HostNameSize > 2) && (HostName[0] == '[') && (HostName[HostNameSize - 2] == ']')) {\r
         //\r
         // HostName format is expressed as IPv6, so, remove '[' and ']'.\r
         //\r
@@ -425,6 +425,7 @@ EfiHttpRequest (
         RemotePort = HTTP_DEFAULT_PORT;\r
       }\r
     }\r
+\r
     //\r
     // If Configure is TRUE, it indicates the first time to call Request();\r
     // If ReConfigure is TRUE, it indicates the request URL is not same\r
@@ -443,7 +444,8 @@ EfiHttpRequest (
           (AsciiStrCmp (HttpInstance->RemoteHost, HostName) == 0) &&\r
           (!HttpInstance->UseHttps || (HttpInstance->UseHttps &&\r
                                        !TlsConfigure &&\r
-                                       HttpInstance->TlsSessionState == EfiTlsSessionDataTransferring))) {\r
+                                       (HttpInstance->TlsSessionState == EfiTlsSessionDataTransferring))))\r
+      {\r
         //\r
         // Host Name and port number of the request URL are the same with previous call to Request().\r
         // If Https protocol used, the corresponding SessionState is EfiTlsSessionDataTransferring.\r
@@ -515,7 +517,7 @@ EfiHttpRequest (
 \r
     if (EFI_ERROR (Status)) {\r
       HostNameSize = AsciiStrSize (HostName);\r
-      HostNameStr = AllocateZeroPool (HostNameSize * sizeof (CHAR16));\r
+      HostNameStr  = AllocateZeroPool (HostNameSize * sizeof (CHAR16));\r
       if (HostNameStr == NULL) {\r
         Status = EFI_OUT_OF_RESOURCES;\r
         goto Error1;\r
@@ -527,6 +529,7 @@ EfiHttpRequest (
       } else {\r
         Status = HttpDns6 (HttpInstance, HostNameStr, &HttpInstance->RemoteIpv6Addr);\r
       }\r
+\r
       HttpNotify (HttpEventDns, Status);\r
 \r
       FreePool (HostNameStr);\r
@@ -542,7 +545,7 @@ EfiHttpRequest (
     ASSERT (HttpInstance->RemoteHost == NULL);\r
     HttpInstance->RemotePort = RemotePort;\r
     HttpInstance->RemoteHost = HostName;\r
-    HostName = NULL;\r
+    HostName                 = NULL;\r
   }\r
 \r
   if (ReConfigure) {\r
@@ -577,8 +580,8 @@ EfiHttpRequest (
     goto Error1;\r
   }\r
 \r
-  Wrap->HttpToken      = Token;\r
-  Wrap->HttpInstance   = HttpInstance;\r
+  Wrap->HttpToken    = Token;\r
+  Wrap->HttpInstance = HttpInstance;\r
   if (Request != NULL) {\r
     Wrap->TcpWrap.Method = Request->Method;\r
   }\r
@@ -608,13 +611,14 @@ EfiHttpRequest (
   // Create request message.\r
   //\r
   FileUrl = Url;\r
-  if (Url != NULL && *FileUrl != '/') {\r
+  if ((Url != NULL) && (*FileUrl != '/')) {\r
     //\r
     // Convert the absolute-URI to the absolute-path\r
     //\r
     while (*FileUrl != ':') {\r
       FileUrl++;\r
     }\r
+\r
     if ((*(FileUrl+1) == '/') && (*(FileUrl+2) == '/')) {\r
       FileUrl += 3;\r
       while (*FileUrl != '/') {\r
@@ -628,7 +632,7 @@ EfiHttpRequest (
 \r
   Status = HttpGenRequestMessage (HttpMsg, FileUrl, &RequestMsg, &RequestMsgSize);\r
 \r
-  if (EFI_ERROR (Status) || NULL == RequestMsg) {\r
+  if (EFI_ERROR (Status) || (NULL == RequestMsg)) {\r
     goto Error3;\r
   }\r
 \r
@@ -651,7 +655,7 @@ EfiHttpRequest (
   Status = HttpTransmitTcp (\r
              HttpInstance,\r
              Wrap,\r
-             (UINT8*) RequestMsg,\r
+             (UINT8 *)RequestMsg,\r
              RequestMsgSize\r
              );\r
   if (EFI_ERROR (Status)) {\r
@@ -698,6 +702,7 @@ Error2:
     gBS->CloseEvent (Wrap->TcpWrap.Tx4Token.CompletionToken.Event);\r
     Wrap->TcpWrap.Tx4Token.CompletionToken.Event = NULL;\r
   }\r
+\r
   if (NULL != Wrap->TcpWrap.Tx6Token.CompletionToken.Event) {\r
     gBS->CloseEvent (Wrap->TcpWrap.Tx6Token.CompletionToken.Event);\r
     Wrap->TcpWrap.Tx6Token.CompletionToken.Event = NULL;\r
@@ -707,15 +712,16 @@ Error1:
   if (HostName != NULL) {\r
     FreePool (HostName);\r
   }\r
+\r
   if (Wrap != NULL) {\r
     FreePool (Wrap);\r
   }\r
+\r
   if (UrlParser != NULL) {\r
     HttpUrlFreeParser (UrlParser);\r
   }\r
 \r
   return Status;\r
-\r
 }\r
 \r
 /**\r
@@ -732,16 +738,16 @@ Error1:
 EFI_STATUS\r
 EFIAPI\r
 HttpCancelTokens (\r
-  IN NET_MAP                *Map,\r
-  IN NET_MAP_ITEM           *Item,\r
-  IN VOID                   *Context\r
+  IN NET_MAP       *Map,\r
+  IN NET_MAP_ITEM  *Item,\r
+  IN VOID          *Context\r
   )\r
 {\r
-  EFI_HTTP_TOKEN            *Token;\r
-  HTTP_TOKEN_WRAP           *Wrap;\r
-  HTTP_PROTOCOL             *HttpInstance;\r
+  EFI_HTTP_TOKEN   *Token;\r
+  HTTP_TOKEN_WRAP  *Wrap;\r
+  HTTP_PROTOCOL    *HttpInstance;\r
 \r
-  Token = (EFI_HTTP_TOKEN *) Context;\r
+  Token = (EFI_HTTP_TOKEN *)Context;\r
 \r
   //\r
   // Return EFI_SUCCESS to check the next item in the map if\r
@@ -751,7 +757,7 @@ HttpCancelTokens (
     return EFI_SUCCESS;\r
   }\r
 \r
-  Wrap = (HTTP_TOKEN_WRAP *) Item->Value;\r
+  Wrap = (HTTP_TOKEN_WRAP *)Item->Value;\r
   ASSERT (Wrap != NULL);\r
   HttpInstance = Wrap->HttpInstance;\r
 \r
@@ -808,11 +814,11 @@ HttpCancelTokens (
 **/\r
 EFI_STATUS\r
 HttpCancel (\r
-  IN  HTTP_PROTOCOL             *HttpInstance,\r
-  IN  EFI_HTTP_TOKEN            *Token\r
+  IN  HTTP_PROTOCOL   *HttpInstance,\r
+  IN  EFI_HTTP_TOKEN  *Token\r
   )\r
 {\r
-  EFI_STATUS                    Status;\r
+  EFI_STATUS  Status;\r
 \r
   //\r
   // First check the tokens queued by EfiHttpRequest().\r
@@ -855,7 +861,6 @@ HttpCancel (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Abort an asynchronous HTTP request or response token.\r
 \r
@@ -881,11 +886,11 @@ HttpCancel (
 EFI_STATUS\r
 EFIAPI\r
 EfiHttpCancel (\r
-  IN  EFI_HTTP_PROTOCOL         *This,\r
-  IN  EFI_HTTP_TOKEN            *Token\r
+  IN  EFI_HTTP_PROTOCOL  *This,\r
+  IN  EFI_HTTP_TOKEN     *Token\r
   )\r
 {\r
-  HTTP_PROTOCOL                 *HttpInstance;\r
+  HTTP_PROTOCOL  *HttpInstance;\r
 \r
   if (This == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
@@ -898,7 +903,6 @@ EfiHttpCancel (
   }\r
 \r
   return HttpCancel (HttpInstance, Token);\r
-\r
 }\r
 \r
 /**\r
@@ -918,28 +922,28 @@ EfiHttpCancel (
 EFI_STATUS\r
 EFIAPI\r
 HttpBodyParserCallback (\r
-  IN HTTP_BODY_PARSE_EVENT      EventType,\r
-  IN CHAR8                      *Data,\r
-  IN UINTN                      Length,\r
-  IN VOID                       *Context\r
+  IN HTTP_BODY_PARSE_EVENT  EventType,\r
+  IN CHAR8                  *Data,\r
+  IN UINTN                  Length,\r
+  IN VOID                   *Context\r
   )\r
 {\r
-  HTTP_CALLBACK_DATA            *CallbackData;\r
-  HTTP_TOKEN_WRAP               *Wrap;\r
-  UINTN                         BodyLength;\r
-  CHAR8                         *Body;\r
+  HTTP_CALLBACK_DATA  *CallbackData;\r
+  HTTP_TOKEN_WRAP     *Wrap;\r
+  UINTN               BodyLength;\r
+  CHAR8               *Body;\r
 \r
   if (EventType != BodyParseEventOnComplete) {\r
     return EFI_SUCCESS;\r
   }\r
 \r
-  if (Data == NULL || Length != 0 || Context == NULL) {\r
+  if ((Data == NULL) || (Length != 0) || (Context == NULL)) {\r
     return EFI_SUCCESS;\r
   }\r
 \r
-  CallbackData = (HTTP_CALLBACK_DATA *) Context;\r
+  CallbackData = (HTTP_CALLBACK_DATA *)Context;\r
 \r
-  Wrap       = (HTTP_TOKEN_WRAP *) (CallbackData->Wrap);\r
+  Wrap       = (HTTP_TOKEN_WRAP *)(CallbackData->Wrap);\r
   Body       = CallbackData->ParseData;\r
   BodyLength = CallbackData->ParseDataLength;\r
 \r
@@ -965,35 +969,35 @@ HttpBodyParserCallback (
 **/\r
 EFI_STATUS\r
 HttpResponseWorker (\r
-  IN  HTTP_TOKEN_WRAP           *Wrap\r
+  IN  HTTP_TOKEN_WRAP  *Wrap\r
   )\r
 {\r
-  EFI_STATUS                    Status;\r
-  EFI_HTTP_MESSAGE              *HttpMsg;\r
-  CHAR8                         *EndofHeader;\r
-  CHAR8                         *HttpHeaders;\r
-  UINTN                         SizeofHeaders;\r
-  UINTN                         BufferSize;\r
-  UINTN                         StatusCode;\r
-  CHAR8                         *Tmp;\r
-  CHAR8                         *HeaderTmp;\r
-  CHAR8                         *StatusCodeStr;\r
-  UINTN                         BodyLen;\r
-  HTTP_PROTOCOL                 *HttpInstance;\r
-  EFI_HTTP_TOKEN                *Token;\r
-  NET_MAP_ITEM                  *Item;\r
-  HTTP_TOKEN_WRAP               *ValueInItem;\r
-  UINTN                         HdrLen;\r
-  NET_FRAGMENT                  Fragment;\r
-  UINT32                        TimeoutValue;\r
-\r
-  if (Wrap == NULL || Wrap->HttpInstance == NULL) {\r
+  EFI_STATUS        Status;\r
+  EFI_HTTP_MESSAGE  *HttpMsg;\r
+  CHAR8             *EndofHeader;\r
+  CHAR8             *HttpHeaders;\r
+  UINTN             SizeofHeaders;\r
+  UINTN             BufferSize;\r
+  UINTN             StatusCode;\r
+  CHAR8             *Tmp;\r
+  CHAR8             *HeaderTmp;\r
+  CHAR8             *StatusCodeStr;\r
+  UINTN             BodyLen;\r
+  HTTP_PROTOCOL     *HttpInstance;\r
+  EFI_HTTP_TOKEN    *Token;\r
+  NET_MAP_ITEM      *Item;\r
+  HTTP_TOKEN_WRAP   *ValueInItem;\r
+  UINTN             HdrLen;\r
+  NET_FRAGMENT      Fragment;\r
+  UINT32            TimeoutValue;\r
+\r
+  if ((Wrap == NULL) || (Wrap->HttpInstance == NULL)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
   HttpInstance = Wrap->HttpInstance;\r
-  Token = Wrap->HttpToken;\r
-  HttpMsg = Token->Message;\r
+  Token        = Wrap->HttpToken;\r
+  HttpMsg      = Token->Message;\r
 \r
   HttpInstance->EndofHeader = NULL;\r
   HttpInstance->HttpHeaders = NULL;\r
@@ -1014,7 +1018,7 @@ HttpResponseWorker (
       //\r
       // The data is stored at [NextMsg, CacheBody + CacheLen].\r
       //\r
-      HdrLen = HttpInstance->CacheBody + HttpInstance->CacheLen - HttpInstance->NextMsg;\r
+      HdrLen      = HttpInstance->CacheBody + HttpInstance->CacheLen - HttpInstance->NextMsg;\r
       HttpHeaders = AllocateZeroPool (HdrLen);\r
       if (HttpHeaders == NULL) {\r
         Status = EFI_OUT_OF_RESOURCES;\r
@@ -1026,8 +1030,8 @@ HttpResponseWorker (
       HttpInstance->CacheBody   = NULL;\r
       HttpInstance->NextMsg     = NULL;\r
       HttpInstance->CacheOffset = 0;\r
-      SizeofHeaders = HdrLen;\r
-      BufferSize = HttpInstance->CacheLen;\r
+      SizeofHeaders             = HdrLen;\r
+      BufferSize                = HttpInstance->CacheLen;\r
 \r
       //\r
       // Check whether we cached the whole HTTP headers.\r
@@ -1038,7 +1042,6 @@ HttpResponseWorker (
     HttpInstance->EndofHeader = &EndofHeader;\r
     HttpInstance->HttpHeaders = &HttpHeaders;\r
 \r
-\r
     if (HttpInstance->TimeoutEvent == NULL) {\r
       //\r
       // Create TimeoutEvent for response\r
@@ -1131,9 +1134,9 @@ HttpResponseWorker (
     }\r
 \r
     HttpMsg->Data.Response->StatusCode = HttpMappingToStatusCode (StatusCode);\r
-    HttpInstance->StatusCode = StatusCode;\r
+    HttpInstance->StatusCode           = StatusCode;\r
 \r
-    Status = EFI_NOT_READY;\r
+    Status      = EFI_NOT_READY;\r
     ValueInItem = NULL;\r
 \r
     //\r
@@ -1144,8 +1147,8 @@ HttpResponseWorker (
     // Hence, check that case before doing a NetMapRemoveHead.\r
     //\r
     if (!NetMapIsEmpty (&HttpInstance->TxTokens)) {\r
-      NetMapRemoveHead (&HttpInstance->TxTokens, (VOID**) &ValueInItem);\r
-      if (ValueInItem == NULL)  {\r
+      NetMapRemoveHead (&HttpInstance->TxTokens, (VOID **)&ValueInItem);\r
+      if (ValueInItem == NULL) {\r
         goto Error;\r
       }\r
 \r
@@ -1193,7 +1196,6 @@ HttpResponseWorker (
       FreePool (HttpHeaders);\r
       HttpHeaders = NULL;\r
 \r
-\r
       //\r
       // Init message-body parser by header information.\r
       //\r
@@ -1203,7 +1205,7 @@ HttpResponseWorker (
                  HttpMsg->HeaderCount,\r
                  HttpMsg->Headers,\r
                  HttpBodyParserCallback,\r
-                 (VOID *) (&HttpInstance->CallbackData),\r
+                 (VOID *)(&HttpInstance->CallbackData),\r
                  &HttpInstance->MsgParser\r
                  );\r
       if (EFI_ERROR (Status)) {\r
@@ -1217,8 +1219,8 @@ HttpResponseWorker (
         //\r
         // Record the CallbackData data.\r
         //\r
-        HttpInstance->CallbackData.Wrap = (VOID *) Wrap;\r
-        HttpInstance->CallbackData.ParseData = (VOID *) HttpInstance->CacheBody;\r
+        HttpInstance->CallbackData.Wrap            = (VOID *)Wrap;\r
+        HttpInstance->CallbackData.ParseData       = (VOID *)HttpInstance->CacheBody;\r
         HttpInstance->CallbackData.ParseDataLength = HttpInstance->CacheLen;\r
 \r
         //\r
@@ -1279,7 +1281,7 @@ HttpResponseWorker (
         //\r
         CopyMem (HttpMsg->Body, HttpInstance->CacheBody + HttpInstance->CacheOffset, BodyLen);\r
         HttpInstance->CacheOffset = BodyLen + HttpInstance->CacheOffset;\r
-        HttpMsg->BodyLength = BodyLen;\r
+        HttpMsg->BodyLength       = BodyLen;\r
 \r
         if (HttpInstance->NextMsg == NULL) {\r
           //\r
@@ -1291,6 +1293,7 @@ HttpResponseWorker (
           HttpInstance->CacheOffset = 0;\r
         }\r
       }\r
+\r
       //\r
       // Return since we already received required data.\r
       //\r
@@ -1298,12 +1301,12 @@ HttpResponseWorker (
       goto Exit;\r
     }\r
 \r
-    if (BodyLen == 0 && HttpInstance->MsgParser == NULL) {\r
+    if ((BodyLen == 0) && (HttpInstance->MsgParser == NULL)) {\r
       //\r
       // We received a complete HTTP message, and we don't have more data to return to caller.\r
       //\r
       HttpMsg->BodyLength = 0;\r
-      Status = EFI_SUCCESS;\r
+      Status              = EFI_SUCCESS;\r
       goto Exit;\r
     }\r
   }\r
@@ -1319,7 +1322,6 @@ HttpResponseWorker (
     if (EFI_ERROR (Status)) {\r
       goto Error2;\r
     }\r
-\r
   } else {\r
     if (HttpInstance->TimeoutEvent == NULL) {\r
       //\r
@@ -1361,15 +1363,15 @@ HttpResponseWorker (
     //\r
     // Process the received the body packet.\r
     //\r
-    HttpMsg->BodyLength = MIN ((UINTN) Fragment.Len, HttpMsg->BodyLength);\r
+    HttpMsg->BodyLength = MIN ((UINTN)Fragment.Len, HttpMsg->BodyLength);\r
 \r
     CopyMem (HttpMsg->Body, Fragment.Bulk, HttpMsg->BodyLength);\r
 \r
     //\r
     // Record the CallbackData data.\r
     //\r
-    HttpInstance->CallbackData.Wrap = (VOID *) Wrap;\r
-    HttpInstance->CallbackData.ParseData = HttpMsg->Body;\r
+    HttpInstance->CallbackData.Wrap            = (VOID *)Wrap;\r
+    HttpInstance->CallbackData.ParseData       = HttpMsg->Body;\r
     HttpInstance->CallbackData.ParseDataLength = HttpMsg->BodyLength;\r
 \r
     //\r
@@ -1396,7 +1398,7 @@ HttpResponseWorker (
     // Check whether there is the next message header in the HttpMsg->Body.\r
     //\r
     if (HttpInstance->NextMsg != NULL) {\r
-      HttpMsg->BodyLength = HttpInstance->NextMsg - (CHAR8 *) HttpMsg->Body;\r
+      HttpMsg->BodyLength = HttpInstance->NextMsg - (CHAR8 *)HttpMsg->Body;\r
     }\r
 \r
     HttpInstance->CacheLen = Fragment.Len - HttpMsg->BodyLength;\r
@@ -1491,10 +1493,8 @@ Error:
   gBS->SignalEvent (Token->Event);\r
 \r
   return Status;\r
-\r
 }\r
 \r
-\r
 /**\r
   The Response() function queues an HTTP response to this HTTP instance, similar to\r
   Receive() function in the EFI TCP driver. When the HTTP response is received successfully,\r
@@ -1545,14 +1545,14 @@ Error:
 EFI_STATUS\r
 EFIAPI\r
 EfiHttpResponse (\r
-  IN  EFI_HTTP_PROTOCOL         *This,\r
-  IN  EFI_HTTP_TOKEN            *Token\r
+  IN  EFI_HTTP_PROTOCOL  *This,\r
+  IN  EFI_HTTP_TOKEN     *Token\r
   )\r
 {\r
-  EFI_STATUS                    Status;\r
-  EFI_HTTP_MESSAGE              *HttpMsg;\r
-  HTTP_PROTOCOL                 *HttpInstance;\r
-  HTTP_TOKEN_WRAP               *Wrap;\r
+  EFI_STATUS        Status;\r
+  EFI_HTTP_MESSAGE  *HttpMsg;\r
+  HTTP_PROTOCOL     *HttpInstance;\r
+  HTTP_TOKEN_WRAP   *Wrap;\r
 \r
   if ((This == NULL) || (Token == NULL)) {\r
     return EFI_INVALID_PARAMETER;\r
@@ -1619,6 +1619,7 @@ Error:
     if (Wrap->TcpWrap.Rx6Token.CompletionToken.Event != NULL) {\r
       gBS->CloseEvent (Wrap->TcpWrap.Rx6Token.CompletionToken.Event);\r
     }\r
+\r
     FreePool (Wrap);\r
   }\r
 \r
@@ -1648,11 +1649,11 @@ Error:
 EFI_STATUS\r
 EFIAPI\r
 EfiHttpPoll (\r
-  IN  EFI_HTTP_PROTOCOL         *This\r
+  IN  EFI_HTTP_PROTOCOL  *This\r
   )\r
 {\r
-  EFI_STATUS                    Status;\r
-  HTTP_PROTOCOL                 *HttpInstance;\r
+  EFI_STATUS     Status;\r
+  HTTP_PROTOCOL  *HttpInstance;\r
 \r
   if (This == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
@@ -1668,11 +1669,13 @@ EfiHttpPoll (
     if (HttpInstance->Tcp6 == NULL) {\r
       return EFI_NOT_STARTED;\r
     }\r
+\r
     Status = HttpInstance->Tcp6->Poll (HttpInstance->Tcp6);\r
   } else {\r
     if (HttpInstance->Tcp4 == NULL) {\r
       return EFI_NOT_STARTED;\r
     }\r
+\r
     Status = HttpInstance->Tcp4->Poll (HttpInstance->Tcp4);\r
   }\r
 \r