]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdePkg/Library/BaseLib/SafeString.c
MdePkg: Apply uncrustify changes
[mirror_edk2.git] / MdePkg / Library / BaseLib / SafeString.c
index 417497cbc96a47aae6f54998df8407c742a78e00..f338a32a3a415ece743fd6ee00c3152980d65919 100644 (file)
@@ -1,14 +1,8 @@
 /** @file\r
   Safe String functions.\r
 \r
-  Copyright (c) 2014 - 2018, Intel Corporation. All rights reserved.<BR>\r
-  This program and the accompanying materials\r
-  are licensed and made available under the terms and conditions of the BSD License\r
-  which accompanies this distribution.  The full text of the license may be found at\r
-  http://opensource.org/licenses/bsd-license.php.\r
-\r
-  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
-  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
+  Copyright (c) 2014 - 2019, Intel Corporation. All rights reserved.<BR>\r
+  SPDX-License-Identifier: BSD-2-Clause-Patent\r
 \r
 **/\r
 \r
 \r
 #define SAFE_STRING_CONSTRAINT_CHECK(Expression, Status)  \\r
   do { \\r
-    ASSERT (Expression); \\r
     if (!(Expression)) { \\r
+      DEBUG ((DEBUG_VERBOSE, \\r
+        "%a(%d) %a: SAFE_STRING_CONSTRAINT_CHECK(%a) failed.  Return %r\n", \\r
+        __FILE__, DEBUG_LINE_NUMBER, __FUNCTION__, DEBUG_EXPRESSION_STRING (Expression), Status)); \\r
       return Status; \\r
     } \\r
   } while (FALSE)\r
 **/\r
 BOOLEAN\r
 InternalSafeStringIsOverlap (\r
-  IN VOID    *Base1,\r
-  IN UINTN   Size1,\r
-  IN VOID    *Base2,\r
-  IN UINTN   Size2\r
+  IN VOID   *Base1,\r
+  IN UINTN  Size1,\r
+  IN VOID   *Base2,\r
+  IN UINTN  Size2\r
   )\r
 {\r
   if ((((UINTN)Base1 >= (UINTN)Base2) && ((UINTN)Base1 < (UINTN)Base2 + Size2)) ||\r
-      (((UINTN)Base2 >= (UINTN)Base1) && ((UINTN)Base2 < (UINTN)Base1 + Size1))) {\r
+      (((UINTN)Base2 >= (UINTN)Base1) && ((UINTN)Base2 < (UINTN)Base1 + Size1)))\r
+  {\r
     return TRUE;\r
   }\r
+\r
   return FALSE;\r
 }\r
 \r
@@ -73,7 +71,7 @@ InternalSafeStringNoStrOverlap (
   IN UINTN   Size2\r
   )\r
 {\r
-  return !InternalSafeStringIsOverlap (Str1, Size1 * sizeof(CHAR16), Str2, Size2 * sizeof(CHAR16));\r
+  return !InternalSafeStringIsOverlap (Str1, Size1 * sizeof (CHAR16), Str2, Size2 * sizeof (CHAR16));\r
 }\r
 \r
 /**\r
@@ -91,10 +89,10 @@ InternalSafeStringNoStrOverlap (
 **/\r
 BOOLEAN\r
 InternalSafeStringNoAsciiStrOverlap (\r
-  IN CHAR8   *Str1,\r
-  IN UINTN   Size1,\r
-  IN CHAR8   *Str2,\r
-  IN UINTN   Size2\r
+  IN CHAR8  *Str1,\r
+  IN UINTN  Size1,\r
+  IN CHAR8  *Str2,\r
+  IN UINTN  Size2\r
   )\r
 {\r
   return !InternalSafeStringIsOverlap (Str1, Size1, Str2, Size2);\r
@@ -119,13 +117,13 @@ InternalSafeStringNoAsciiStrOverlap (
 UINTN\r
 EFIAPI\r
 StrnLenS (\r
-  IN CONST CHAR16              *String,\r
-  IN UINTN                     MaxSize\r
+  IN CONST CHAR16  *String,\r
+  IN UINTN         MaxSize\r
   )\r
 {\r
-  UINTN                             Length;\r
+  UINTN  Length;\r
 \r
-  ASSERT (((UINTN) String & BIT0) == 0);\r
+  ASSERT (((UINTN)String & BIT0) == 0);\r
 \r
   //\r
   // If String is a null pointer or MaxSize is 0, then the StrnLenS function returns zero.\r
@@ -145,8 +143,10 @@ StrnLenS (
     if (Length >= MaxSize - 1) {\r
       return MaxSize;\r
     }\r
+\r
     Length++;\r
   }\r
+\r
   return Length;\r
 }\r
 \r
@@ -174,8 +174,8 @@ StrnLenS (
 UINTN\r
 EFIAPI\r
 StrnSizeS (\r
-  IN CONST CHAR16              *String,\r
-  IN UINTN                     MaxSize\r
+  IN CONST CHAR16  *String,\r
+  IN UINTN         MaxSize\r
   )\r
 {\r
   //\r
@@ -203,7 +203,6 @@ StrnSizeS (
 \r
   If Destination is not aligned on a 16-bit boundary, then ASSERT().\r
   If Source is not aligned on a 16-bit boundary, then ASSERT().\r
-  If an error would be returned, then the function will also ASSERT().\r
 \r
   If an error is returned, then the Destination is unmodified.\r
 \r
@@ -225,15 +224,15 @@ StrnSizeS (
 RETURN_STATUS\r
 EFIAPI\r
 StrCpyS (\r
-  OUT CHAR16       *Destination,\r
-  IN  UINTN        DestMax,\r
-  IN  CONST CHAR16 *Source\r
+  OUT CHAR16        *Destination,\r
+  IN  UINTN         DestMax,\r
+  IN  CONST CHAR16  *Source\r
   )\r
 {\r
-  UINTN            SourceLen;\r
+  UINTN  SourceLen;\r
 \r
-  ASSERT (((UINTN) Destination & BIT0) == 0);\r
-  ASSERT (((UINTN) Source & BIT0) == 0);\r
+  ASSERT (((UINTN)Destination & BIT0) == 0);\r
+  ASSERT (((UINTN)Source & BIT0) == 0);\r
 \r
   //\r
   // 1. Neither Destination nor Source shall be a null pointer.\r
@@ -271,6 +270,7 @@ StrCpyS (
   while (*Source != 0) {\r
     *(Destination++) = *(Source++);\r
   }\r
+\r
   *Destination = 0;\r
 \r
   return RETURN_SUCCESS;\r
@@ -285,7 +285,6 @@ StrCpyS (
 \r
   If Length > 0 and Destination is not aligned on a 16-bit boundary, then ASSERT().\r
   If Length > 0 and Source is not aligned on a 16-bit boundary, then ASSERT().\r
-  If an error would be returned, then the function will also ASSERT().\r
 \r
   If an error is returned, then the Destination is unmodified.\r
 \r
@@ -309,16 +308,16 @@ StrCpyS (
 RETURN_STATUS\r
 EFIAPI\r
 StrnCpyS (\r
-  OUT CHAR16       *Destination,\r
-  IN  UINTN        DestMax,\r
-  IN  CONST CHAR16 *Source,\r
-  IN  UINTN        Length\r
+  OUT CHAR16        *Destination,\r
+  IN  UINTN         DestMax,\r
+  IN  CONST CHAR16  *Source,\r
+  IN  UINTN         Length\r
   )\r
 {\r
-  UINTN            SourceLen;\r
+  UINTN  SourceLen;\r
 \r
-  ASSERT (((UINTN) Destination & BIT0) == 0);\r
-  ASSERT (((UINTN) Source & BIT0) == 0);\r
+  ASSERT (((UINTN)Destination & BIT0) == 0);\r
+  ASSERT (((UINTN)Source & BIT0) == 0);\r
 \r
   //\r
   // 1. Neither Destination nor Source shall be a null pointer.\r
@@ -353,6 +352,7 @@ StrnCpyS (
   if (SourceLen > Length) {\r
     SourceLen = Length;\r
   }\r
+\r
   SAFE_STRING_CONSTRAINT_CHECK (InternalSafeStringNoStrOverlap (Destination, DestMax, (CHAR16 *)Source, SourceLen + 1), RETURN_ACCESS_DENIED);\r
 \r
   //\r
@@ -365,6 +365,7 @@ StrnCpyS (
     *(Destination++) = *(Source++);\r
     SourceLen--;\r
   }\r
+\r
   *Destination = 0;\r
 \r
   return RETURN_SUCCESS;\r
@@ -378,7 +379,6 @@ StrnCpyS (
 \r
   If Destination is not aligned on a 16-bit boundary, then ASSERT().\r
   If Source is not aligned on a 16-bit boundary, then ASSERT().\r
-  If an error would be returned, then the function will also ASSERT().\r
 \r
   If an error is returned, then the Destination is unmodified.\r
 \r
@@ -403,17 +403,17 @@ StrnCpyS (
 RETURN_STATUS\r
 EFIAPI\r
 StrCatS (\r
-  IN OUT CHAR16       *Destination,\r
-  IN     UINTN        DestMax,\r
-  IN     CONST CHAR16 *Source\r
+  IN OUT CHAR16        *Destination,\r
+  IN     UINTN         DestMax,\r
+  IN     CONST CHAR16  *Source\r
   )\r
 {\r
-  UINTN               DestLen;\r
-  UINTN               CopyLen;\r
-  UINTN               SourceLen;\r
+  UINTN  DestLen;\r
+  UINTN  CopyLen;\r
+  UINTN  SourceLen;\r
 \r
-  ASSERT (((UINTN) Destination & BIT0) == 0);\r
-  ASSERT (((UINTN) Source & BIT0) == 0);\r
+  ASSERT (((UINTN)Destination & BIT0) == 0);\r
+  ASSERT (((UINTN)Source & BIT0) == 0);\r
 \r
   //\r
   // Let CopyLen denote the value DestMax - StrnLenS(Destination, DestMax) upon entry to StrCatS.\r
@@ -464,6 +464,7 @@ StrCatS (
   while (*Source != 0) {\r
     *(Destination++) = *(Source++);\r
   }\r
+\r
   *Destination = 0;\r
 \r
   return RETURN_SUCCESS;\r
@@ -479,7 +480,6 @@ StrCatS (
 \r
   If Destination is not aligned on a 16-bit boundary, then ASSERT().\r
   If Source is not aligned on a 16-bit boundary, then ASSERT().\r
-  If an error would be returned, then the function will also ASSERT().\r
 \r
   If an error is returned, then the Destination is unmodified.\r
 \r
@@ -505,18 +505,18 @@ StrCatS (
 RETURN_STATUS\r
 EFIAPI\r
 StrnCatS (\r
-  IN OUT CHAR16       *Destination,\r
-  IN     UINTN        DestMax,\r
-  IN     CONST CHAR16 *Source,\r
-  IN     UINTN        Length\r
+  IN OUT CHAR16        *Destination,\r
+  IN     UINTN         DestMax,\r
+  IN     CONST CHAR16  *Source,\r
+  IN     UINTN         Length\r
   )\r
 {\r
-  UINTN               DestLen;\r
-  UINTN               CopyLen;\r
-  UINTN               SourceLen;\r
+  UINTN  DestLen;\r
+  UINTN  CopyLen;\r
+  UINTN  SourceLen;\r
 \r
-  ASSERT (((UINTN) Destination & BIT0) == 0);\r
-  ASSERT (((UINTN) Source & BIT0) == 0);\r
+  ASSERT (((UINTN)Destination & BIT0) == 0);\r
+  ASSERT (((UINTN)Source & BIT0) == 0);\r
 \r
   //\r
   // Let CopyLen denote the value DestMax - StrnLenS(Destination, DestMax) upon entry to StrnCatS.\r
@@ -562,6 +562,7 @@ StrnCatS (
   if (SourceLen > Length) {\r
     SourceLen = Length;\r
   }\r
+\r
   SAFE_STRING_CONSTRAINT_CHECK (InternalSafeStringNoStrOverlap (Destination, DestMax, (CHAR16 *)Source, SourceLen + 1), RETURN_ACCESS_DENIED);\r
 \r
   //\r
@@ -576,6 +577,7 @@ StrnCatS (
     *(Destination++) = *(Source++);\r
     SourceLen--;\r
   }\r
+\r
   *Destination = 0;\r
 \r
   return RETURN_SUCCESS;\r
@@ -596,12 +598,7 @@ StrnCatS (
   be ignored. Then, the function stops at the first character that is a not a\r
   valid decimal character or a Null-terminator, whichever one comes first.\r
 \r
-  If String is NULL, then ASSERT().\r
-  If Data is NULL, then ASSERT().\r
   If String is not aligned in a 16-bit boundary, then ASSERT().\r
-  If PcdMaximumUnicodeStringLength is not zero, and String contains more than\r
-  PcdMaximumUnicodeStringLength Unicode characters, not including the\r
-  Null-terminator, then ASSERT().\r
 \r
   If String has no valid decimal digits in the above format, then 0 is stored\r
   at the location pointed to by Data.\r
@@ -632,12 +629,12 @@ StrnCatS (
 RETURN_STATUS\r
 EFIAPI\r
 StrDecimalToUintnS (\r
-  IN  CONST CHAR16             *String,\r
-  OUT       CHAR16             **EndPointer,  OPTIONAL\r
-  OUT       UINTN              *Data\r
+  IN  CONST CHAR16  *String,\r
+  OUT       CHAR16  **EndPointer   OPTIONAL,\r
+  OUT       UINTN   *Data\r
   )\r
 {\r
-  ASSERT (((UINTN) String & BIT0) == 0);\r
+  ASSERT (((UINTN)String & BIT0) == 0);\r
 \r
   //\r
   // 1. Neither String nor Data shall be a null pointer.\r
@@ -653,7 +650,7 @@ StrDecimalToUintnS (
   }\r
 \r
   if (EndPointer != NULL) {\r
-    *EndPointer = (CHAR16 *) String;\r
+    *EndPointer = (CHAR16 *)String;\r
   }\r
 \r
   //\r
@@ -681,8 +678,9 @@ StrDecimalToUintnS (
     if (*Data > ((MAX_UINTN - (*String - L'0')) / 10)) {\r
       *Data = MAX_UINTN;\r
       if (EndPointer != NULL) {\r
-        *EndPointer = (CHAR16 *) String;\r
+        *EndPointer = (CHAR16 *)String;\r
       }\r
+\r
       return RETURN_UNSUPPORTED;\r
     }\r
 \r
@@ -691,8 +689,9 @@ StrDecimalToUintnS (
   }\r
 \r
   if (EndPointer != NULL) {\r
-    *EndPointer = (CHAR16 *) String;\r
+    *EndPointer = (CHAR16 *)String;\r
   }\r
+\r
   return RETURN_SUCCESS;\r
 }\r
 \r
@@ -711,12 +710,7 @@ StrDecimalToUintnS (
   be ignored. Then, the function stops at the first character that is a not a\r
   valid decimal character or a Null-terminator, whichever one comes first.\r
 \r
-  If String is NULL, then ASSERT().\r
-  If Data is NULL, then ASSERT().\r
   If String is not aligned in a 16-bit boundary, then ASSERT().\r
-  If PcdMaximumUnicodeStringLength is not zero, and String contains more than\r
-  PcdMaximumUnicodeStringLength Unicode characters, not including the\r
-  Null-terminator, then ASSERT().\r
 \r
   If String has no valid decimal digits in the above format, then 0 is stored\r
   at the location pointed to by Data.\r
@@ -747,12 +741,12 @@ StrDecimalToUintnS (
 RETURN_STATUS\r
 EFIAPI\r
 StrDecimalToUint64S (\r
-  IN  CONST CHAR16             *String,\r
-  OUT       CHAR16             **EndPointer,  OPTIONAL\r
-  OUT       UINT64             *Data\r
+  IN  CONST CHAR16  *String,\r
+  OUT       CHAR16  **EndPointer   OPTIONAL,\r
+  OUT       UINT64  *Data\r
   )\r
 {\r
-  ASSERT (((UINTN) String & BIT0) == 0);\r
+  ASSERT (((UINTN)String & BIT0) == 0);\r
 \r
   //\r
   // 1. Neither String nor Data shall be a null pointer.\r
@@ -768,7 +762,7 @@ StrDecimalToUint64S (
   }\r
 \r
   if (EndPointer != NULL) {\r
-    *EndPointer = (CHAR16 *) String;\r
+    *EndPointer = (CHAR16 *)String;\r
   }\r
 \r
   //\r
@@ -796,8 +790,9 @@ StrDecimalToUint64S (
     if (*Data > DivU64x32 (MAX_UINT64 - (*String - L'0'), 10)) {\r
       *Data = MAX_UINT64;\r
       if (EndPointer != NULL) {\r
-        *EndPointer = (CHAR16 *) String;\r
+        *EndPointer = (CHAR16 *)String;\r
       }\r
+\r
       return RETURN_UNSUPPORTED;\r
     }\r
 \r
@@ -806,8 +801,9 @@ StrDecimalToUint64S (
   }\r
 \r
   if (EndPointer != NULL) {\r
-    *EndPointer = (CHAR16 *) String;\r
+    *EndPointer = (CHAR16 *)String;\r
   }\r
+\r
   return RETURN_SUCCESS;\r
 }\r
 \r
@@ -831,12 +827,7 @@ StrDecimalToUint64S (
   the first character that is a not a valid hexadecimal character or NULL,\r
   whichever one comes first.\r
 \r
-  If String is NULL, then ASSERT().\r
-  If Data is NULL, then ASSERT().\r
   If String is not aligned in a 16-bit boundary, then ASSERT().\r
-  If PcdMaximumUnicodeStringLength is not zero, and String contains more than\r
-  PcdMaximumUnicodeStringLength Unicode characters, not including the\r
-  Null-terminator, then ASSERT().\r
 \r
   If String has no valid hexadecimal digits in the above format, then 0 is\r
   stored at the location pointed to by Data.\r
@@ -867,12 +858,12 @@ StrDecimalToUint64S (
 RETURN_STATUS\r
 EFIAPI\r
 StrHexToUintnS (\r
-  IN  CONST CHAR16             *String,\r
-  OUT       CHAR16             **EndPointer,  OPTIONAL\r
-  OUT       UINTN              *Data\r
+  IN  CONST CHAR16  *String,\r
+  OUT       CHAR16  **EndPointer   OPTIONAL,\r
+  OUT       UINTN   *Data\r
   )\r
 {\r
-  ASSERT (((UINTN) String & BIT0) == 0);\r
+  ASSERT (((UINTN)String & BIT0) == 0);\r
 \r
   //\r
   // 1. Neither String nor Data shall be a null pointer.\r
@@ -888,7 +879,7 @@ StrHexToUintnS (
   }\r
 \r
   if (EndPointer != NULL) {\r
-    *EndPointer = (CHAR16 *) String;\r
+    *EndPointer = (CHAR16 *)String;\r
   }\r
 \r
   //\r
@@ -905,11 +896,12 @@ StrHexToUintnS (
     String++;\r
   }\r
 \r
-  if (InternalCharToUpper (*String) == L'X') {\r
+  if (CharToUpper (*String) == L'X') {\r
     if (*(String - 1) != L'0') {\r
       *Data = 0;\r
       return RETURN_SUCCESS;\r
     }\r
+\r
     //\r
     // Skip the 'X'\r
     //\r
@@ -927,8 +919,9 @@ StrHexToUintnS (
     if (*Data > ((MAX_UINTN - InternalHexCharToUintn (*String)) >> 4)) {\r
       *Data = MAX_UINTN;\r
       if (EndPointer != NULL) {\r
-        *EndPointer = (CHAR16 *) String;\r
+        *EndPointer = (CHAR16 *)String;\r
       }\r
+\r
       return RETURN_UNSUPPORTED;\r
     }\r
 \r
@@ -937,8 +930,9 @@ StrHexToUintnS (
   }\r
 \r
   if (EndPointer != NULL) {\r
-    *EndPointer = (CHAR16 *) String;\r
+    *EndPointer = (CHAR16 *)String;\r
   }\r
+\r
   return RETURN_SUCCESS;\r
 }\r
 \r
@@ -962,12 +956,7 @@ StrHexToUintnS (
   the first character that is a not a valid hexadecimal character or NULL,\r
   whichever one comes first.\r
 \r
-  If String is NULL, then ASSERT().\r
-  If Data is NULL, then ASSERT().\r
   If String is not aligned in a 16-bit boundary, then ASSERT().\r
-  If PcdMaximumUnicodeStringLength is not zero, and String contains more than\r
-  PcdMaximumUnicodeStringLength Unicode characters, not including the\r
-  Null-terminator, then ASSERT().\r
 \r
   If String has no valid hexadecimal digits in the above format, then 0 is\r
   stored at the location pointed to by Data.\r
@@ -998,12 +987,12 @@ StrHexToUintnS (
 RETURN_STATUS\r
 EFIAPI\r
 StrHexToUint64S (\r
-  IN  CONST CHAR16             *String,\r
-  OUT       CHAR16             **EndPointer,  OPTIONAL\r
-  OUT       UINT64             *Data\r
+  IN  CONST CHAR16  *String,\r
+  OUT       CHAR16  **EndPointer   OPTIONAL,\r
+  OUT       UINT64  *Data\r
   )\r
 {\r
-  ASSERT (((UINTN) String & BIT0) == 0);\r
+  ASSERT (((UINTN)String & BIT0) == 0);\r
 \r
   //\r
   // 1. Neither String nor Data shall be a null pointer.\r
@@ -1019,7 +1008,7 @@ StrHexToUint64S (
   }\r
 \r
   if (EndPointer != NULL) {\r
-    *EndPointer = (CHAR16 *) String;\r
+    *EndPointer = (CHAR16 *)String;\r
   }\r
 \r
   //\r
@@ -1036,11 +1025,12 @@ StrHexToUint64S (
     String++;\r
   }\r
 \r
-  if (InternalCharToUpper (*String) == L'X') {\r
+  if (CharToUpper (*String) == L'X') {\r
     if (*(String - 1) != L'0') {\r
       *Data = 0;\r
       return RETURN_SUCCESS;\r
     }\r
+\r
     //\r
     // Skip the 'X'\r
     //\r
@@ -1058,8 +1048,9 @@ StrHexToUint64S (
     if (*Data > RShiftU64 (MAX_UINT64 - InternalHexCharToUintn (*String), 4)) {\r
       *Data = MAX_UINT64;\r
       if (EndPointer != NULL) {\r
-        *EndPointer = (CHAR16 *) String;\r
+        *EndPointer = (CHAR16 *)String;\r
       }\r
+\r
       return RETURN_UNSUPPORTED;\r
     }\r
 \r
@@ -1068,8 +1059,9 @@ StrHexToUint64S (
   }\r
 \r
   if (EndPointer != NULL) {\r
-    *EndPointer = (CHAR16 *) String;\r
+    *EndPointer = (CHAR16 *)String;\r
   }\r
+\r
   return RETURN_SUCCESS;\r
 }\r
 \r
@@ -1097,16 +1089,8 @@ StrHexToUint64S (
   "::" can be used to compress one or more groups of X when X contains only 0.\r
   The "::" can only appear once in the String.\r
 \r
-  If String is NULL, then ASSERT().\r
-\r
-  If Address is NULL, then ASSERT().\r
-\r
   If String is not aligned in a 16-bit boundary, then ASSERT().\r
 \r
-  If PcdMaximumUnicodeStringLength is not zero, and String contains more than\r
-  PcdMaximumUnicodeStringLength Unicode characters, not including the\r
-  Null-terminator, then ASSERT().\r
-\r
   If EndPointer is not NULL and Address is translated from String, a pointer\r
   to the character that stopped the scan is stored at the location pointed to\r
   by EndPointer.\r
@@ -1134,27 +1118,27 @@ StrHexToUint64S (
 RETURN_STATUS\r
 EFIAPI\r
 StrToIpv6Address (\r
-  IN  CONST CHAR16       *String,\r
-  OUT CHAR16             **EndPointer, OPTIONAL\r
-  OUT IPv6_ADDRESS       *Address,\r
-  OUT UINT8              *PrefixLength OPTIONAL\r
+  IN  CONST CHAR16  *String,\r
+  OUT CHAR16        **EndPointer  OPTIONAL,\r
+  OUT IPv6_ADDRESS  *Address,\r
+  OUT UINT8         *PrefixLength OPTIONAL\r
   )\r
 {\r
-  RETURN_STATUS          Status;\r
-  UINTN                  AddressIndex;\r
-  UINTN                  Uintn;\r
-  IPv6_ADDRESS           LocalAddress;\r
-  UINT8                  LocalPrefixLength;\r
-  CONST CHAR16           *Pointer;\r
-  CHAR16                 *End;\r
-  UINTN                  CompressStart;\r
-  BOOLEAN                ExpectPrefix;\r
+  RETURN_STATUS  Status;\r
+  UINTN          AddressIndex;\r
+  UINTN          Uintn;\r
+  IPv6_ADDRESS   LocalAddress;\r
+  UINT8          LocalPrefixLength;\r
+  CONST CHAR16   *Pointer;\r
+  CHAR16         *End;\r
+  UINTN          CompressStart;\r
+  BOOLEAN        ExpectPrefix;\r
 \r
   LocalPrefixLength = MAX_UINT8;\r
   CompressStart     = ARRAY_SIZE (Address->Addr);\r
   ExpectPrefix      = FALSE;\r
 \r
-  ASSERT (((UINTN) String & BIT0) == 0);\r
+  ASSERT (((UINTN)String & BIT0) == 0);\r
 \r
   //\r
   // 1. None of String or Guid shall be a null pointer.\r
@@ -1182,7 +1166,7 @@ StrToIpv6Address (
         return RETURN_UNSUPPORTED;\r
       }\r
 \r
-      if (CompressStart != ARRAY_SIZE (Address->Addr) || AddressIndex == ARRAY_SIZE (Address->Addr)) {\r
+      if ((CompressStart != ARRAY_SIZE (Address->Addr)) || (AddressIndex == ARRAY_SIZE (Address->Addr))) {\r
         //\r
         // "::" can only appear once.\r
         // "::" can only appear when address is not full length.\r
@@ -1202,6 +1186,7 @@ StrToIpv6Address (
             //\r
             return RETURN_UNSUPPORTED;\r
           }\r
+\r
           Pointer++;\r
         }\r
       }\r
@@ -1224,33 +1209,35 @@ StrToIpv6Address (
         // Get X.\r
         //\r
         Status = StrHexToUintnS (Pointer, &End, &Uintn);\r
-        if (RETURN_ERROR (Status) || End - Pointer > 4) {\r
+        if (RETURN_ERROR (Status) || (End - Pointer > 4)) {\r
           //\r
           // Number of hexadecimal digit characters is no more than 4.\r
           //\r
           return RETURN_UNSUPPORTED;\r
         }\r
+\r
         Pointer = End;\r
         //\r
         // Uintn won't exceed MAX_UINT16 if number of hexadecimal digit characters is no more than 4.\r
         //\r
         ASSERT (AddressIndex + 1 < ARRAY_SIZE (Address->Addr));\r
-        LocalAddress.Addr[AddressIndex] = (UINT8) ((UINT16) Uintn >> 8);\r
-        LocalAddress.Addr[AddressIndex + 1] = (UINT8) Uintn;\r
-        AddressIndex += 2;\r
+        LocalAddress.Addr[AddressIndex]     = (UINT8)((UINT16)Uintn >> 8);\r
+        LocalAddress.Addr[AddressIndex + 1] = (UINT8)Uintn;\r
+        AddressIndex                       += 2;\r
       } else {\r
         //\r
         // Get P, then exit the loop.\r
         //\r
         Status = StrDecimalToUintnS (Pointer, &End, &Uintn);\r
-        if (RETURN_ERROR (Status) || End == Pointer || Uintn > 128) {\r
+        if (RETURN_ERROR (Status) || (End == Pointer) || (Uintn > 128)) {\r
           //\r
           // Prefix length should not exceed 128.\r
           //\r
           return RETURN_UNSUPPORTED;\r
         }\r
-        LocalPrefixLength = (UINT8) Uintn;\r
-        Pointer = End;\r
+\r
+        LocalPrefixLength = (UINT8)Uintn;\r
+        Pointer           = End;\r
         break;\r
       }\r
     }\r
@@ -1273,18 +1260,21 @@ StrToIpv6Address (
       //\r
       break;\r
     }\r
+\r
     Pointer++;\r
   }\r
 \r
-  if ((AddressIndex == ARRAY_SIZE (Address->Addr) && CompressStart != ARRAY_SIZE (Address->Addr)) ||\r
-    (AddressIndex != ARRAY_SIZE (Address->Addr) && CompressStart == ARRAY_SIZE (Address->Addr))\r
-      ) {\r
+  if (((AddressIndex == ARRAY_SIZE (Address->Addr)) && (CompressStart != ARRAY_SIZE (Address->Addr))) ||\r
+      ((AddressIndex != ARRAY_SIZE (Address->Addr)) && (CompressStart == ARRAY_SIZE (Address->Addr)))\r
+      )\r
+  {\r
     //\r
     // Full length of address shall not have compressing zeros.\r
     // Non-full length of address shall have compressing zeros.\r
     //\r
     return RETURN_UNSUPPORTED;\r
   }\r
+\r
   CopyMem (&Address->Addr[0], &LocalAddress.Addr[0], CompressStart);\r
   ZeroMem (&Address->Addr[CompressStart], ARRAY_SIZE (Address->Addr) - AddressIndex);\r
   if (AddressIndex > CompressStart) {\r
@@ -1298,8 +1288,9 @@ StrToIpv6Address (
   if (PrefixLength != NULL) {\r
     *PrefixLength = LocalPrefixLength;\r
   }\r
+\r
   if (EndPointer != NULL) {\r
-    *EndPointer = (CHAR16 *) Pointer;\r
+    *EndPointer = (CHAR16 *)Pointer;\r
   }\r
 \r
   return RETURN_SUCCESS;\r
@@ -1323,16 +1314,8 @@ StrToIpv6Address (
   When /P is in the String, the function stops at the first character that is not\r
   a valid decimal digit character after P is converted.\r
 \r
-  If String is NULL, then ASSERT().\r
-\r
-  If Address is NULL, then ASSERT().\r
-\r
   If String is not aligned in a 16-bit boundary, then ASSERT().\r
 \r
-  If PcdMaximumUnicodeStringLength is not zero, and String contains more than\r
-  PcdMaximumUnicodeStringLength Unicode characters, not including the\r
-  Null-terminator, then ASSERT().\r
-\r
   If EndPointer is not NULL and Address is translated from String, a pointer\r
   to the character that stopped the scan is stored at the location pointed to\r
   by EndPointer.\r
@@ -1357,22 +1340,22 @@ StrToIpv6Address (
 RETURN_STATUS\r
 EFIAPI\r
 StrToIpv4Address (\r
-  IN  CONST CHAR16       *String,\r
-  OUT CHAR16             **EndPointer, OPTIONAL\r
-  OUT IPv4_ADDRESS       *Address,\r
-  OUT UINT8              *PrefixLength OPTIONAL\r
+  IN  CONST CHAR16  *String,\r
+  OUT CHAR16        **EndPointer  OPTIONAL,\r
+  OUT IPv4_ADDRESS  *Address,\r
+  OUT UINT8         *PrefixLength OPTIONAL\r
   )\r
 {\r
-  RETURN_STATUS          Status;\r
-  UINTN                  AddressIndex;\r
-  UINTN                  Uintn;\r
-  IPv4_ADDRESS           LocalAddress;\r
-  UINT8                  LocalPrefixLength;\r
-  CHAR16                 *Pointer;\r
+  RETURN_STATUS  Status;\r
+  UINTN          AddressIndex;\r
+  UINTN          Uintn;\r
+  IPv4_ADDRESS   LocalAddress;\r
+  UINT8          LocalPrefixLength;\r
+  CHAR16         *Pointer;\r
 \r
   LocalPrefixLength = MAX_UINT8;\r
 \r
-  ASSERT (((UINTN) String & BIT0) == 0);\r
+  ASSERT (((UINTN)String & BIT0) == 0);\r
 \r
   //\r
   // 1. None of String or Guid shall be a null pointer.\r
@@ -1380,7 +1363,7 @@ StrToIpv4Address (
   SAFE_STRING_CONSTRAINT_CHECK ((String != NULL), RETURN_INVALID_PARAMETER);\r
   SAFE_STRING_CONSTRAINT_CHECK ((Address != NULL), RETURN_INVALID_PARAMETER);\r
 \r
-  for (Pointer = (CHAR16 *) String, AddressIndex = 0; AddressIndex < ARRAY_SIZE (Address->Addr) + 1;) {\r
+  for (Pointer = (CHAR16 *)String, AddressIndex = 0; AddressIndex < ARRAY_SIZE (Address->Addr) + 1;) {\r
     if (!InternalIsDecimalDigitCharacter (*Pointer)) {\r
       //\r
       // D or P contains invalid characters.\r
@@ -1391,10 +1374,11 @@ StrToIpv4Address (
     //\r
     // Get D or P.\r
     //\r
-    Status = StrDecimalToUintnS ((CONST CHAR16 *) Pointer, &Pointer, &Uintn);\r
+    Status = StrDecimalToUintnS ((CONST CHAR16 *)Pointer, &Pointer, &Uintn);\r
     if (RETURN_ERROR (Status)) {\r
       return RETURN_UNSUPPORTED;\r
     }\r
+\r
     if (AddressIndex == ARRAY_SIZE (Address->Addr)) {\r
       //\r
       // It's P.\r
@@ -1402,7 +1386,8 @@ StrToIpv4Address (
       if (Uintn > 32) {\r
         return RETURN_UNSUPPORTED;\r
       }\r
-      LocalPrefixLength = (UINT8) Uintn;\r
+\r
+      LocalPrefixLength = (UINT8)Uintn;\r
     } else {\r
       //\r
       // It's D.\r
@@ -1410,7 +1395,8 @@ StrToIpv4Address (
       if (Uintn > MAX_UINT8) {\r
         return RETURN_UNSUPPORTED;\r
       }\r
-      LocalAddress.Addr[AddressIndex] = (UINT8) Uintn;\r
+\r
+      LocalAddress.Addr[AddressIndex] = (UINT8)Uintn;\r
       AddressIndex++;\r
     }\r
 \r
@@ -1450,6 +1436,7 @@ StrToIpv4Address (
   if (PrefixLength != NULL) {\r
     *PrefixLength = LocalPrefixLength;\r
   }\r
+\r
   if (EndPointer != NULL) {\r
     *EndPointer = Pointer;\r
   }\r
@@ -1488,8 +1475,6 @@ StrToIpv4Address (
                   oo          Data4[48:55]\r
                   pp          Data4[56:63]\r
 \r
-  If String is NULL, then ASSERT().\r
-  If Guid is NULL, then ASSERT().\r
   If String is not aligned in a 16-bit boundary, then ASSERT().\r
 \r
   @param  String                   Pointer to a Null-terminated Unicode string.\r
@@ -1504,14 +1489,14 @@ StrToIpv4Address (
 RETURN_STATUS\r
 EFIAPI\r
 StrToGuid (\r
-  IN  CONST CHAR16       *String,\r
-  OUT GUID               *Guid\r
+  IN  CONST CHAR16  *String,\r
+  OUT GUID          *Guid\r
   )\r
 {\r
-  RETURN_STATUS          Status;\r
-  GUID                   LocalGuid;\r
+  RETURN_STATUS  Status;\r
+  GUID           LocalGuid;\r
 \r
-  ASSERT (((UINTN) String & BIT0) == 0);\r
+  ASSERT (((UINTN)String & BIT0) == 0);\r
 \r
   //\r
   // 1. None of String or Guid shall be a null pointer.\r
@@ -1522,49 +1507,53 @@ StrToGuid (
   //\r
   // Get aabbccdd in big-endian.\r
   //\r
-  Status = StrHexToBytes (String, 2 * sizeof (LocalGuid.Data1), (UINT8 *) &LocalGuid.Data1, sizeof (LocalGuid.Data1));\r
-  if (RETURN_ERROR (Status) || String[2 * sizeof (LocalGuid.Data1)] != L'-') {\r
+  Status = StrHexToBytes (String, 2 * sizeof (LocalGuid.Data1), (UINT8 *)&LocalGuid.Data1, sizeof (LocalGuid.Data1));\r
+  if (RETURN_ERROR (Status) || (String[2 * sizeof (LocalGuid.Data1)] != L'-')) {\r
     return RETURN_UNSUPPORTED;\r
   }\r
+\r
   //\r
   // Convert big-endian to little-endian.\r
   //\r
   LocalGuid.Data1 = SwapBytes32 (LocalGuid.Data1);\r
-  String += 2 * sizeof (LocalGuid.Data1) + 1;\r
+  String         += 2 * sizeof (LocalGuid.Data1) + 1;\r
 \r
   //\r
   // Get eeff in big-endian.\r
   //\r
-  Status = StrHexToBytes (String, 2 * sizeof (LocalGuid.Data2), (UINT8 *) &LocalGuid.Data2, sizeof (LocalGuid.Data2));\r
-  if (RETURN_ERROR (Status) || String[2 * sizeof (LocalGuid.Data2)] != L'-') {\r
+  Status = StrHexToBytes (String, 2 * sizeof (LocalGuid.Data2), (UINT8 *)&LocalGuid.Data2, sizeof (LocalGuid.Data2));\r
+  if (RETURN_ERROR (Status) || (String[2 * sizeof (LocalGuid.Data2)] != L'-')) {\r
     return RETURN_UNSUPPORTED;\r
   }\r
+\r
   //\r
   // Convert big-endian to little-endian.\r
   //\r
   LocalGuid.Data2 = SwapBytes16 (LocalGuid.Data2);\r
-  String += 2 * sizeof (LocalGuid.Data2) + 1;\r
+  String         += 2 * sizeof (LocalGuid.Data2) + 1;\r
 \r
   //\r
   // Get gghh in big-endian.\r
   //\r
-  Status = StrHexToBytes (String, 2 * sizeof (LocalGuid.Data3), (UINT8 *) &LocalGuid.Data3, sizeof (LocalGuid.Data3));\r
-  if (RETURN_ERROR (Status) || String[2 * sizeof (LocalGuid.Data3)] != L'-') {\r
+  Status = StrHexToBytes (String, 2 * sizeof (LocalGuid.Data3), (UINT8 *)&LocalGuid.Data3, sizeof (LocalGuid.Data3));\r
+  if (RETURN_ERROR (Status) || (String[2 * sizeof (LocalGuid.Data3)] != L'-')) {\r
     return RETURN_UNSUPPORTED;\r
   }\r
+\r
   //\r
   // Convert big-endian to little-endian.\r
   //\r
   LocalGuid.Data3 = SwapBytes16 (LocalGuid.Data3);\r
-  String += 2 * sizeof (LocalGuid.Data3) + 1;\r
+  String         += 2 * sizeof (LocalGuid.Data3) + 1;\r
 \r
   //\r
   // Get iijj.\r
   //\r
   Status = StrHexToBytes (String, 2 * 2, &LocalGuid.Data4[0], 2);\r
-  if (RETURN_ERROR (Status) || String[2 * 2] != L'-') {\r
+  if (RETURN_ERROR (Status) || (String[2 * 2] != L'-')) {\r
     return RETURN_UNSUPPORTED;\r
   }\r
+\r
   String += 2 * 2 + 1;\r
 \r
   //\r
@@ -1595,17 +1584,6 @@ StrToGuid (
 \r
   If String is not aligned in a 16-bit boundary, then ASSERT().\r
 \r
-  If String is NULL, then ASSERT().\r
-\r
-  If Buffer is NULL, then ASSERT().\r
-\r
-  If Length is not multiple of 2, then ASSERT().\r
-\r
-  If PcdMaximumUnicodeStringLength is not zero and Length is greater than\r
-  PcdMaximumUnicodeStringLength, then ASSERT().\r
-\r
-  If MaxBufferSize is less than (Length / 2), then ASSERT().\r
-\r
   @param  String                   Pointer to a Null-terminated Unicode string.\r
   @param  Length                   The number of Unicode characters to decode.\r
   @param  Buffer                   Pointer to the converted bytes array.\r
@@ -1626,15 +1604,15 @@ StrToGuid (
 RETURN_STATUS\r
 EFIAPI\r
 StrHexToBytes (\r
-  IN  CONST CHAR16       *String,\r
-  IN  UINTN              Length,\r
-  OUT UINT8              *Buffer,\r
-  IN  UINTN              MaxBufferSize\r
+  IN  CONST CHAR16  *String,\r
+  IN  UINTN         Length,\r
+  OUT UINT8         *Buffer,\r
+  IN  UINTN         MaxBufferSize\r
   )\r
 {\r
-  UINTN                  Index;\r
+  UINTN  Index;\r
 \r
-  ASSERT (((UINTN) String & BIT0) == 0);\r
+  ASSERT (((UINTN)String & BIT0) == 0);\r
 \r
   //\r
   // 1. None of String or Buffer shall be a null pointer.\r
@@ -1667,6 +1645,7 @@ StrHexToBytes (
       break;\r
     }\r
   }\r
+\r
   if (Index != Length) {\r
     return RETURN_UNSUPPORTED;\r
   }\r
@@ -1674,18 +1653,18 @@ StrHexToBytes (
   //\r
   // Convert the hex string to bytes.\r
   //\r
-  for(Index = 0; Index < Length; Index++) {\r
-\r
+  for (Index = 0; Index < Length; Index++) {\r
     //\r
     // For even characters, write the upper nibble for each buffer byte,\r
     // and for even characters, the lower nibble.\r
     //\r
     if ((Index & BIT0) == 0) {\r
-      Buffer[Index / 2]  = (UINT8) InternalHexCharToUintn (String[Index]) << 4;\r
+      Buffer[Index / 2] = (UINT8)InternalHexCharToUintn (String[Index]) << 4;\r
     } else {\r
-      Buffer[Index / 2] |= (UINT8) InternalHexCharToUintn (String[Index]);\r
+      Buffer[Index / 2] |= (UINT8)InternalHexCharToUintn (String[Index]);\r
     }\r
   }\r
+\r
   return RETURN_SUCCESS;\r
 }\r
 \r
@@ -1706,11 +1685,11 @@ StrHexToBytes (
 UINTN\r
 EFIAPI\r
 AsciiStrnLenS (\r
-  IN CONST CHAR8               *String,\r
-  IN UINTN                     MaxSize\r
+  IN CONST CHAR8  *String,\r
+  IN UINTN        MaxSize\r
   )\r
 {\r
-  UINTN                             Length;\r
+  UINTN  Length;\r
 \r
   //\r
   // If String is a null pointer or MaxSize is 0, then the AsciiStrnLenS function returns zero.\r
@@ -1730,8 +1709,10 @@ AsciiStrnLenS (
     if (Length >= MaxSize - 1) {\r
       return MaxSize;\r
     }\r
+\r
     Length++;\r
   }\r
+\r
   return Length;\r
 }\r
 \r
@@ -1757,8 +1738,8 @@ AsciiStrnLenS (
 UINTN\r
 EFIAPI\r
 AsciiStrnSizeS (\r
-  IN CONST CHAR8               *String,\r
-  IN UINTN                     MaxSize\r
+  IN CONST CHAR8  *String,\r
+  IN UINTN        MaxSize\r
   )\r
 {\r
   //\r
@@ -1785,8 +1766,6 @@ AsciiStrnSizeS (
 \r
   This function is similar as strcpy_s defined in C11.\r
 \r
-  If an error would be returned, then the function will also ASSERT().\r
-\r
   If an error is returned, then the Destination is unmodified.\r
 \r
   @param  Destination              A pointer to a Null-terminated Ascii string.\r
@@ -1812,7 +1791,7 @@ AsciiStrCpyS (
   IN  CONST CHAR8  *Source\r
   )\r
 {\r
-  UINTN            SourceLen;\r
+  UINTN  SourceLen;\r
 \r
   //\r
   // 1. Neither Destination nor Source shall be a null pointer.\r
@@ -1850,6 +1829,7 @@ AsciiStrCpyS (
   while (*Source != 0) {\r
     *(Destination++) = *(Source++);\r
   }\r
+\r
   *Destination = 0;\r
 \r
   return RETURN_SUCCESS;\r
@@ -1862,8 +1842,6 @@ AsciiStrCpyS (
 \r
   This function is similar as strncpy_s defined in C11.\r
 \r
-  If an error would be returned, then the function will also ASSERT().\r
-\r
   If an error is returned, then the Destination is unmodified.\r
 \r
   @param  Destination              A pointer to a Null-terminated Ascii string.\r
@@ -1892,7 +1870,7 @@ AsciiStrnCpyS (
   IN  UINTN        Length\r
   )\r
 {\r
-  UINTN            SourceLen;\r
+  UINTN  SourceLen;\r
 \r
   //\r
   // 1. Neither Destination nor Source shall be a null pointer.\r
@@ -1927,6 +1905,7 @@ AsciiStrnCpyS (
   if (SourceLen > Length) {\r
     SourceLen = Length;\r
   }\r
+\r
   SAFE_STRING_CONSTRAINT_CHECK (InternalSafeStringNoAsciiStrOverlap (Destination, DestMax, (CHAR8 *)Source, SourceLen + 1), RETURN_ACCESS_DENIED);\r
 \r
   //\r
@@ -1939,6 +1918,7 @@ AsciiStrnCpyS (
     *(Destination++) = *(Source++);\r
     SourceLen--;\r
   }\r
+\r
   *Destination = 0;\r
 \r
   return RETURN_SUCCESS;\r
@@ -1950,8 +1930,6 @@ AsciiStrnCpyS (
 \r
   This function is similar as strcat_s defined in C11.\r
 \r
-  If an error would be returned, then the function will also ASSERT().\r
-\r
   If an error is returned, then the Destination is unmodified.\r
 \r
   @param  Destination              A pointer to a Null-terminated Ascii string.\r
@@ -1980,9 +1958,9 @@ AsciiStrCatS (
   IN     CONST CHAR8  *Source\r
   )\r
 {\r
-  UINTN               DestLen;\r
-  UINTN               CopyLen;\r
-  UINTN               SourceLen;\r
+  UINTN  DestLen;\r
+  UINTN  CopyLen;\r
+  UINTN  SourceLen;\r
 \r
   //\r
   // Let CopyLen denote the value DestMax - AsciiStrnLenS(Destination, DestMax) upon entry to AsciiStrCatS.\r
@@ -2033,6 +2011,7 @@ AsciiStrCatS (
   while (*Source != 0) {\r
     *(Destination++) = *(Source++);\r
   }\r
+\r
   *Destination = 0;\r
 \r
   return RETURN_SUCCESS;\r
@@ -2046,8 +2025,6 @@ AsciiStrCatS (
 \r
   This function is similar as strncat_s defined in C11.\r
 \r
-  If an error would be returned, then the function will also ASSERT().\r
-\r
   If an error is returned, then the Destination is unmodified.\r
 \r
   @param  Destination              A pointer to a Null-terminated Ascii string.\r
@@ -2078,9 +2055,9 @@ AsciiStrnCatS (
   IN     UINTN        Length\r
   )\r
 {\r
-  UINTN               DestLen;\r
-  UINTN               CopyLen;\r
-  UINTN               SourceLen;\r
+  UINTN  DestLen;\r
+  UINTN  CopyLen;\r
+  UINTN  SourceLen;\r
 \r
   //\r
   // Let CopyLen denote the value DestMax - AsciiStrnLenS(Destination, DestMax) upon entry to AsciiStrnCatS.\r
@@ -2126,6 +2103,7 @@ AsciiStrnCatS (
   if (SourceLen > Length) {\r
     SourceLen = Length;\r
   }\r
+\r
   SAFE_STRING_CONSTRAINT_CHECK (InternalSafeStringNoAsciiStrOverlap (Destination, DestMax, (CHAR8 *)Source, SourceLen + 1), RETURN_ACCESS_DENIED);\r
 \r
   //\r
@@ -2140,6 +2118,7 @@ AsciiStrnCatS (
     *(Destination++) = *(Source++);\r
     SourceLen--;\r
   }\r
+\r
   *Destination = 0;\r
 \r
   return RETURN_SUCCESS;\r
@@ -2160,12 +2139,6 @@ AsciiStrnCatS (
   be ignored. Then, the function stops at the first character that is a not a\r
   valid decimal character or a Null-terminator, whichever one comes first.\r
 \r
-  If String is NULL, then ASSERT().\r
-  If Data is NULL, then ASSERT().\r
-  If PcdMaximumAsciiStringLength is not zero, and String contains more than\r
-  PcdMaximumAsciiStringLength Ascii characters, not including the\r
-  Null-terminator, then ASSERT().\r
-\r
   If String has no valid decimal digits in the above format, then 0 is stored\r
   at the location pointed to by Data.\r
   If the number represented by String exceeds the range defined by UINTN, then\r
@@ -2195,9 +2168,9 @@ AsciiStrnCatS (
 RETURN_STATUS\r
 EFIAPI\r
 AsciiStrDecimalToUintnS (\r
-  IN  CONST CHAR8              *String,\r
-  OUT       CHAR8              **EndPointer,  OPTIONAL\r
-  OUT       UINTN              *Data\r
+  IN  CONST CHAR8  *String,\r
+  OUT       CHAR8  **EndPointer   OPTIONAL,\r
+  OUT       UINTN  *Data\r
   )\r
 {\r
   //\r
@@ -2214,7 +2187,7 @@ AsciiStrDecimalToUintnS (
   }\r
 \r
   if (EndPointer != NULL) {\r
-    *EndPointer = (CHAR8 *) String;\r
+    *EndPointer = (CHAR8 *)String;\r
   }\r
 \r
   //\r
@@ -2242,8 +2215,9 @@ AsciiStrDecimalToUintnS (
     if (*Data > ((MAX_UINTN - (*String - '0')) / 10)) {\r
       *Data = MAX_UINTN;\r
       if (EndPointer != NULL) {\r
-        *EndPointer = (CHAR8 *) String;\r
+        *EndPointer = (CHAR8 *)String;\r
       }\r
+\r
       return RETURN_UNSUPPORTED;\r
     }\r
 \r
@@ -2252,8 +2226,9 @@ AsciiStrDecimalToUintnS (
   }\r
 \r
   if (EndPointer != NULL) {\r
-    *EndPointer = (CHAR8 *) String;\r
+    *EndPointer = (CHAR8 *)String;\r
   }\r
+\r
   return RETURN_SUCCESS;\r
 }\r
 \r
@@ -2272,12 +2247,6 @@ AsciiStrDecimalToUintnS (
   be ignored. Then, the function stops at the first character that is a not a\r
   valid decimal character or a Null-terminator, whichever one comes first.\r
 \r
-  If String is NULL, then ASSERT().\r
-  If Data is NULL, then ASSERT().\r
-  If PcdMaximumAsciiStringLength is not zero, and String contains more than\r
-  PcdMaximumAsciiStringLength Ascii characters, not including the\r
-  Null-terminator, then ASSERT().\r
-\r
   If String has no valid decimal digits in the above format, then 0 is stored\r
   at the location pointed to by Data.\r
   If the number represented by String exceeds the range defined by UINT64, then\r
@@ -2307,9 +2276,9 @@ AsciiStrDecimalToUintnS (
 RETURN_STATUS\r
 EFIAPI\r
 AsciiStrDecimalToUint64S (\r
-  IN  CONST CHAR8              *String,\r
-  OUT       CHAR8              **EndPointer,  OPTIONAL\r
-  OUT       UINT64             *Data\r
+  IN  CONST CHAR8   *String,\r
+  OUT       CHAR8   **EndPointer   OPTIONAL,\r
+  OUT       UINT64  *Data\r
   )\r
 {\r
   //\r
@@ -2326,7 +2295,7 @@ AsciiStrDecimalToUint64S (
   }\r
 \r
   if (EndPointer != NULL) {\r
-    *EndPointer = (CHAR8 *) String;\r
+    *EndPointer = (CHAR8 *)String;\r
   }\r
 \r
   //\r
@@ -2354,8 +2323,9 @@ AsciiStrDecimalToUint64S (
     if (*Data > DivU64x32 (MAX_UINT64 - (*String - '0'), 10)) {\r
       *Data = MAX_UINT64;\r
       if (EndPointer != NULL) {\r
-        *EndPointer = (CHAR8 *) String;\r
+        *EndPointer = (CHAR8 *)String;\r
       }\r
+\r
       return RETURN_UNSUPPORTED;\r
     }\r
 \r
@@ -2364,8 +2334,9 @@ AsciiStrDecimalToUint64S (
   }\r
 \r
   if (EndPointer != NULL) {\r
-    *EndPointer = (CHAR8 *) String;\r
+    *EndPointer = (CHAR8 *)String;\r
   }\r
+\r
   return RETURN_SUCCESS;\r
 }\r
 \r
@@ -2388,12 +2359,6 @@ AsciiStrDecimalToUint64S (
   character that is a not a valid hexadecimal character or Null-terminator,\r
   whichever on comes first.\r
 \r
-  If String is NULL, then ASSERT().\r
-  If Data is NULL, then ASSERT().\r
-  If PcdMaximumAsciiStringLength is not zero, and String contains more than\r
-  PcdMaximumAsciiStringLength Ascii characters, not including the\r
-  Null-terminator, then ASSERT().\r
-\r
   If String has no valid hexadecimal digits in the above format, then 0 is\r
   stored at the location pointed to by Data.\r
   If the number represented by String exceeds the range defined by UINTN, then\r
@@ -2423,9 +2388,9 @@ AsciiStrDecimalToUint64S (
 RETURN_STATUS\r
 EFIAPI\r
 AsciiStrHexToUintnS (\r
-  IN  CONST CHAR8              *String,\r
-  OUT       CHAR8              **EndPointer,  OPTIONAL\r
-  OUT       UINTN              *Data\r
+  IN  CONST CHAR8  *String,\r
+  OUT       CHAR8  **EndPointer   OPTIONAL,\r
+  OUT       UINTN  *Data\r
   )\r
 {\r
   //\r
@@ -2442,7 +2407,7 @@ AsciiStrHexToUintnS (
   }\r
 \r
   if (EndPointer != NULL) {\r
-    *EndPointer = (CHAR8 *) String;\r
+    *EndPointer = (CHAR8 *)String;\r
   }\r
 \r
   //\r
@@ -2459,11 +2424,12 @@ AsciiStrHexToUintnS (
     String++;\r
   }\r
 \r
-  if (InternalBaseLibAsciiToUpper (*String) == 'X') {\r
+  if (AsciiCharToUpper (*String) == 'X') {\r
     if (*(String - 1) != '0') {\r
       *Data = 0;\r
       return RETURN_SUCCESS;\r
     }\r
+\r
     //\r
     // Skip the 'X'\r
     //\r
@@ -2481,8 +2447,9 @@ AsciiStrHexToUintnS (
     if (*Data > ((MAX_UINTN - InternalAsciiHexCharToUintn (*String)) >> 4)) {\r
       *Data = MAX_UINTN;\r
       if (EndPointer != NULL) {\r
-        *EndPointer = (CHAR8 *) String;\r
+        *EndPointer = (CHAR8 *)String;\r
       }\r
+\r
       return RETURN_UNSUPPORTED;\r
     }\r
 \r
@@ -2491,8 +2458,9 @@ AsciiStrHexToUintnS (
   }\r
 \r
   if (EndPointer != NULL) {\r
-    *EndPointer = (CHAR8 *) String;\r
+    *EndPointer = (CHAR8 *)String;\r
   }\r
+\r
   return RETURN_SUCCESS;\r
 }\r
 \r
@@ -2515,12 +2483,6 @@ AsciiStrHexToUintnS (
   character that is a not a valid hexadecimal character or Null-terminator,\r
   whichever on comes first.\r
 \r
-  If String is NULL, then ASSERT().\r
-  If Data is NULL, then ASSERT().\r
-  If PcdMaximumAsciiStringLength is not zero, and String contains more than\r
-  PcdMaximumAsciiStringLength Ascii characters, not including the\r
-  Null-terminator, then ASSERT().\r
-\r
   If String has no valid hexadecimal digits in the above format, then 0 is\r
   stored at the location pointed to by Data.\r
   If the number represented by String exceeds the range defined by UINT64, then\r
@@ -2550,9 +2512,9 @@ AsciiStrHexToUintnS (
 RETURN_STATUS\r
 EFIAPI\r
 AsciiStrHexToUint64S (\r
-  IN  CONST CHAR8              *String,\r
-  OUT       CHAR8              **EndPointer,  OPTIONAL\r
-  OUT       UINT64             *Data\r
+  IN  CONST CHAR8   *String,\r
+  OUT       CHAR8   **EndPointer   OPTIONAL,\r
+  OUT       UINT64  *Data\r
   )\r
 {\r
   //\r
@@ -2569,7 +2531,7 @@ AsciiStrHexToUint64S (
   }\r
 \r
   if (EndPointer != NULL) {\r
-    *EndPointer = (CHAR8 *) String;\r
+    *EndPointer = (CHAR8 *)String;\r
   }\r
 \r
   //\r
@@ -2586,11 +2548,12 @@ AsciiStrHexToUint64S (
     String++;\r
   }\r
 \r
-  if (InternalBaseLibAsciiToUpper (*String) == 'X') {\r
+  if (AsciiCharToUpper (*String) == 'X') {\r
     if (*(String - 1) != '0') {\r
       *Data = 0;\r
       return RETURN_SUCCESS;\r
     }\r
+\r
     //\r
     // Skip the 'X'\r
     //\r
@@ -2608,8 +2571,9 @@ AsciiStrHexToUint64S (
     if (*Data > RShiftU64 (MAX_UINT64 - InternalAsciiHexCharToUintn (*String), 4)) {\r
       *Data = MAX_UINT64;\r
       if (EndPointer != NULL) {\r
-        *EndPointer = (CHAR8 *) String;\r
+        *EndPointer = (CHAR8 *)String;\r
       }\r
+\r
       return RETURN_UNSUPPORTED;\r
     }\r
 \r
@@ -2618,8 +2582,9 @@ AsciiStrHexToUint64S (
   }\r
 \r
   if (EndPointer != NULL) {\r
-    *EndPointer = (CHAR8 *) String;\r
+    *EndPointer = (CHAR8 *)String;\r
   }\r
+\r
   return RETURN_SUCCESS;\r
 }\r
 \r
@@ -2641,7 +2606,6 @@ AsciiStrHexToUint64S (
   the upper 8 bits, then ASSERT().\r
 \r
   If Source is not aligned on a 16-bit boundary, then ASSERT().\r
-  If an error would be returned, then the function will also ASSERT().\r
 \r
   If an error is returned, then the Destination is unmodified.\r
 \r
@@ -2667,14 +2631,14 @@ AsciiStrHexToUint64S (
 RETURN_STATUS\r
 EFIAPI\r
 UnicodeStrToAsciiStrS (\r
-  IN      CONST CHAR16              *Source,\r
-  OUT     CHAR8                     *Destination,\r
-  IN      UINTN                     DestMax\r
+  IN      CONST CHAR16  *Source,\r
+  OUT     CHAR8         *Destination,\r
+  IN      UINTN         DestMax\r
   )\r
 {\r
-  UINTN            SourceLen;\r
+  UINTN  SourceLen;\r
 \r
-  ASSERT (((UINTN) Source & BIT0) == 0);\r
+  ASSERT (((UINTN)Source & BIT0) == 0);\r
 \r
   //\r
   // 1. Neither Destination nor Source shall be a null pointer.\r
@@ -2688,6 +2652,7 @@ UnicodeStrToAsciiStrS (
   if (ASCII_RSIZE_MAX != 0) {\r
     SAFE_STRING_CONSTRAINT_CHECK ((DestMax <= ASCII_RSIZE_MAX), RETURN_INVALID_PARAMETER);\r
   }\r
+\r
   if (RSIZE_MAX != 0) {\r
     SAFE_STRING_CONSTRAINT_CHECK ((DestMax <= RSIZE_MAX), RETURN_INVALID_PARAMETER);\r
   }\r
@@ -2706,7 +2671,7 @@ UnicodeStrToAsciiStrS (
   //\r
   // 5. Copying shall not take place between objects that overlap.\r
   //\r
-  SAFE_STRING_CONSTRAINT_CHECK (!InternalSafeStringIsOverlap (Destination, DestMax, (VOID *)Source, (SourceLen + 1) * sizeof(CHAR16)), RETURN_ACCESS_DENIED);\r
+  SAFE_STRING_CONSTRAINT_CHECK (!InternalSafeStringIsOverlap (Destination, DestMax, (VOID *)Source, (SourceLen + 1) * sizeof (CHAR16)), RETURN_ACCESS_DENIED);\r
 \r
   //\r
   // convert string\r
@@ -2717,8 +2682,9 @@ UnicodeStrToAsciiStrS (
     // non-zero value in the upper 8 bits, then ASSERT().\r
     //\r
     ASSERT (*Source < 0x100);\r
-    *(Destination++) = (CHAR8) *(Source++);\r
+    *(Destination++) = (CHAR8)*(Source++);\r
   }\r
+\r
   *Destination = '\0';\r
 \r
   return RETURN_SUCCESS;\r
@@ -2741,7 +2707,6 @@ UnicodeStrToAsciiStrS (
   If any Unicode characters in Source contain non-zero value in the upper 8\r
   bits, then ASSERT().\r
   If Source is not aligned on a 16-bit boundary, then ASSERT().\r
-  If an error would be returned, then the function will also ASSERT().\r
 \r
   If an error is returned, then Destination and DestinationLength are\r
   unmodified.\r
@@ -2773,16 +2738,16 @@ UnicodeStrToAsciiStrS (
 RETURN_STATUS\r
 EFIAPI\r
 UnicodeStrnToAsciiStrS (\r
-  IN      CONST CHAR16              *Source,\r
-  IN      UINTN                     Length,\r
-  OUT     CHAR8                     *Destination,\r
-  IN      UINTN                     DestMax,\r
-  OUT     UINTN                     *DestinationLength\r
+  IN      CONST CHAR16  *Source,\r
+  IN      UINTN         Length,\r
+  OUT     CHAR8         *Destination,\r
+  IN      UINTN         DestMax,\r
+  OUT     UINTN         *DestinationLength\r
   )\r
 {\r
-  UINTN            SourceLen;\r
+  UINTN  SourceLen;\r
 \r
-  ASSERT (((UINTN) Source & BIT0) == 0);\r
+  ASSERT (((UINTN)Source & BIT0) == 0);\r
 \r
   //\r
   // 1. None of Destination, Source or DestinationLength shall be a null\r
@@ -2800,6 +2765,7 @@ UnicodeStrnToAsciiStrS (
     SAFE_STRING_CONSTRAINT_CHECK ((Length <= ASCII_RSIZE_MAX), RETURN_INVALID_PARAMETER);\r
     SAFE_STRING_CONSTRAINT_CHECK ((DestMax <= ASCII_RSIZE_MAX), RETURN_INVALID_PARAMETER);\r
   }\r
+\r
   if (RSIZE_MAX != 0) {\r
     SAFE_STRING_CONSTRAINT_CHECK ((Length <= RSIZE_MAX), RETURN_INVALID_PARAMETER);\r
     SAFE_STRING_CONSTRAINT_CHECK ((DestMax <= RSIZE_MAX), RETURN_INVALID_PARAMETER);\r
@@ -2825,7 +2791,8 @@ UnicodeStrnToAsciiStrS (
   if (SourceLen > Length) {\r
     SourceLen = Length;\r
   }\r
-  SAFE_STRING_CONSTRAINT_CHECK (!InternalSafeStringIsOverlap (Destination, DestMax, (VOID *)Source, (SourceLen + 1) * sizeof(CHAR16)), RETURN_ACCESS_DENIED);\r
+\r
+  SAFE_STRING_CONSTRAINT_CHECK (!InternalSafeStringIsOverlap (Destination, DestMax, (VOID *)Source, (SourceLen + 1) * sizeof (CHAR16)), RETURN_ACCESS_DENIED);\r
 \r
   *DestinationLength = 0;\r
 \r
@@ -2838,10 +2805,11 @@ UnicodeStrnToAsciiStrS (
     // 8 bits, then ASSERT().\r
     //\r
     ASSERT (*Source < 0x100);\r
-    *(Destination++) = (CHAR8) *(Source++);\r
+    *(Destination++) = (CHAR8)*(Source++);\r
     SourceLen--;\r
     (*DestinationLength)++;\r
   }\r
+\r
   *Destination = 0;\r
 \r
   return RETURN_SUCCESS;\r
@@ -2861,7 +2829,6 @@ UnicodeStrnToAsciiStrS (
   equal or greater than ((AsciiStrLen (Source) + 1) * sizeof (CHAR16)) in bytes.\r
 \r
   If Destination is not aligned on a 16-bit boundary, then ASSERT().\r
-  If an error would be returned, then the function will also ASSERT().\r
 \r
   If an error is returned, then the Destination is unmodified.\r
 \r
@@ -2887,14 +2854,14 @@ UnicodeStrnToAsciiStrS (
 RETURN_STATUS\r
 EFIAPI\r
 AsciiStrToUnicodeStrS (\r
-  IN      CONST CHAR8               *Source,\r
-  OUT     CHAR16                    *Destination,\r
-  IN      UINTN                     DestMax\r
+  IN      CONST CHAR8  *Source,\r
+  OUT     CHAR16       *Destination,\r
+  IN      UINTN        DestMax\r
   )\r
 {\r
-  UINTN            SourceLen;\r
+  UINTN  SourceLen;\r
 \r
-  ASSERT (((UINTN) Destination & BIT0) == 0);\r
+  ASSERT (((UINTN)Destination & BIT0) == 0);\r
 \r
   //\r
   // 1. Neither Destination nor Source shall be a null pointer.\r
@@ -2908,6 +2875,7 @@ AsciiStrToUnicodeStrS (
   if (RSIZE_MAX != 0) {\r
     SAFE_STRING_CONSTRAINT_CHECK ((DestMax <= RSIZE_MAX), RETURN_INVALID_PARAMETER);\r
   }\r
+\r
   if (ASCII_RSIZE_MAX != 0) {\r
     SAFE_STRING_CONSTRAINT_CHECK ((DestMax <= ASCII_RSIZE_MAX), RETURN_INVALID_PARAMETER);\r
   }\r
@@ -2926,7 +2894,7 @@ AsciiStrToUnicodeStrS (
   //\r
   // 5. Copying shall not take place between objects that overlap.\r
   //\r
-  SAFE_STRING_CONSTRAINT_CHECK (!InternalSafeStringIsOverlap (Destination, DestMax * sizeof(CHAR16), (VOID *)Source, SourceLen + 1), RETURN_ACCESS_DENIED);\r
+  SAFE_STRING_CONSTRAINT_CHECK (!InternalSafeStringIsOverlap (Destination, DestMax * sizeof (CHAR16), (VOID *)Source, SourceLen + 1), RETURN_ACCESS_DENIED);\r
 \r
   //\r
   // Convert string\r
@@ -2934,6 +2902,7 @@ AsciiStrToUnicodeStrS (
   while (*Source != '\0') {\r
     *(Destination++) = (CHAR16)(UINT8)*(Source++);\r
   }\r
+\r
   *Destination = '\0';\r
 \r
   return RETURN_SUCCESS;\r
@@ -2954,7 +2923,6 @@ AsciiStrToUnicodeStrS (
   ((MIN(AsciiStrLen(Source), Length) + 1) * sizeof (CHAR8)) in bytes.\r
 \r
   If Destination is not aligned on a 16-bit boundary, then ASSERT().\r
-  If an error would be returned, then the function will also ASSERT().\r
 \r
   If an error is returned, then Destination and DestinationLength are\r
   unmodified.\r
@@ -2985,16 +2953,16 @@ AsciiStrToUnicodeStrS (
 RETURN_STATUS\r
 EFIAPI\r
 AsciiStrnToUnicodeStrS (\r
-  IN      CONST CHAR8               *Source,\r
-  IN      UINTN                     Length,\r
-  OUT     CHAR16                    *Destination,\r
-  IN      UINTN                     DestMax,\r
-  OUT     UINTN                     *DestinationLength\r
+  IN      CONST CHAR8  *Source,\r
+  IN      UINTN        Length,\r
+  OUT     CHAR16       *Destination,\r
+  IN      UINTN        DestMax,\r
+  OUT     UINTN        *DestinationLength\r
   )\r
 {\r
-  UINTN            SourceLen;\r
+  UINTN  SourceLen;\r
 \r
-  ASSERT (((UINTN) Destination & BIT0) == 0);\r
+  ASSERT (((UINTN)Destination & BIT0) == 0);\r
 \r
   //\r
   // 1. None of Destination, Source or DestinationLength shall be a null\r
@@ -3012,6 +2980,7 @@ AsciiStrnToUnicodeStrS (
     SAFE_STRING_CONSTRAINT_CHECK ((Length <= RSIZE_MAX), RETURN_INVALID_PARAMETER);\r
     SAFE_STRING_CONSTRAINT_CHECK ((DestMax <= RSIZE_MAX), RETURN_INVALID_PARAMETER);\r
   }\r
+\r
   if (ASCII_RSIZE_MAX != 0) {\r
     SAFE_STRING_CONSTRAINT_CHECK ((Length <= ASCII_RSIZE_MAX), RETURN_INVALID_PARAMETER);\r
     SAFE_STRING_CONSTRAINT_CHECK ((DestMax <= ASCII_RSIZE_MAX), RETURN_INVALID_PARAMETER);\r
@@ -3037,7 +3006,8 @@ AsciiStrnToUnicodeStrS (
   if (SourceLen > Length) {\r
     SourceLen = Length;\r
   }\r
-  SAFE_STRING_CONSTRAINT_CHECK (!InternalSafeStringIsOverlap (Destination, DestMax * sizeof(CHAR16), (VOID *)Source, SourceLen + 1), RETURN_ACCESS_DENIED);\r
+\r
+  SAFE_STRING_CONSTRAINT_CHECK (!InternalSafeStringIsOverlap (Destination, DestMax * sizeof (CHAR16), (VOID *)Source, SourceLen + 1), RETURN_ACCESS_DENIED);\r
 \r
   *DestinationLength = 0;\r
 \r
@@ -3049,6 +3019,7 @@ AsciiStrnToUnicodeStrS (
     SourceLen--;\r
     (*DestinationLength)++;\r
   }\r
+\r
   *Destination = 0;\r
 \r
   return RETURN_SUCCESS;\r
@@ -3078,10 +3049,6 @@ AsciiStrnToUnicodeStrS (
   "::" can be used to compress one or more groups of X when X contains only 0.\r
   The "::" can only appear once in the String.\r
 \r
-  If String is NULL, then ASSERT().\r
-\r
-  If Address is NULL, then ASSERT().\r
-\r
   If EndPointer is not NULL and Address is translated from String, a pointer\r
   to the character that stopped the scan is stored at the location pointed to\r
   by EndPointer.\r
@@ -3109,21 +3076,21 @@ AsciiStrnToUnicodeStrS (
 RETURN_STATUS\r
 EFIAPI\r
 AsciiStrToIpv6Address (\r
-  IN  CONST CHAR8        *String,\r
-  OUT CHAR8              **EndPointer, OPTIONAL\r
-  OUT IPv6_ADDRESS       *Address,\r
-  OUT UINT8              *PrefixLength OPTIONAL\r
+  IN  CONST CHAR8   *String,\r
+  OUT CHAR8         **EndPointer  OPTIONAL,\r
+  OUT IPv6_ADDRESS  *Address,\r
+  OUT UINT8         *PrefixLength OPTIONAL\r
   )\r
 {\r
-  RETURN_STATUS          Status;\r
-  UINTN                  AddressIndex;\r
-  UINTN                  Uintn;\r
-  IPv6_ADDRESS           LocalAddress;\r
-  UINT8                  LocalPrefixLength;\r
-  CONST CHAR8            *Pointer;\r
-  CHAR8                  *End;\r
-  UINTN                  CompressStart;\r
-  BOOLEAN                ExpectPrefix;\r
+  RETURN_STATUS  Status;\r
+  UINTN          AddressIndex;\r
+  UINTN          Uintn;\r
+  IPv6_ADDRESS   LocalAddress;\r
+  UINT8          LocalPrefixLength;\r
+  CONST CHAR8    *Pointer;\r
+  CHAR8          *End;\r
+  UINTN          CompressStart;\r
+  BOOLEAN        ExpectPrefix;\r
 \r
   LocalPrefixLength = MAX_UINT8;\r
   CompressStart     = ARRAY_SIZE (Address->Addr);\r
@@ -3155,7 +3122,7 @@ AsciiStrToIpv6Address (
         return RETURN_UNSUPPORTED;\r
       }\r
 \r
-      if (CompressStart != ARRAY_SIZE (Address->Addr) || AddressIndex == ARRAY_SIZE (Address->Addr)) {\r
+      if ((CompressStart != ARRAY_SIZE (Address->Addr)) || (AddressIndex == ARRAY_SIZE (Address->Addr))) {\r
         //\r
         // "::" can only appear once.\r
         // "::" can only appear when address is not full length.\r
@@ -3175,6 +3142,7 @@ AsciiStrToIpv6Address (
             //\r
             return RETURN_UNSUPPORTED;\r
           }\r
+\r
           Pointer++;\r
         }\r
       }\r
@@ -3197,33 +3165,35 @@ AsciiStrToIpv6Address (
         // Get X.\r
         //\r
         Status = AsciiStrHexToUintnS (Pointer, &End, &Uintn);\r
-        if (RETURN_ERROR (Status) || End - Pointer > 4) {\r
+        if (RETURN_ERROR (Status) || (End - Pointer > 4)) {\r
           //\r
           // Number of hexadecimal digit characters is no more than 4.\r
           //\r
           return RETURN_UNSUPPORTED;\r
         }\r
+\r
         Pointer = End;\r
         //\r
         // Uintn won't exceed MAX_UINT16 if number of hexadecimal digit characters is no more than 4.\r
         //\r
         ASSERT (AddressIndex + 1 < ARRAY_SIZE (Address->Addr));\r
-        LocalAddress.Addr[AddressIndex] = (UINT8) ((UINT16) Uintn >> 8);\r
-        LocalAddress.Addr[AddressIndex + 1] = (UINT8) Uintn;\r
-        AddressIndex += 2;\r
+        LocalAddress.Addr[AddressIndex]     = (UINT8)((UINT16)Uintn >> 8);\r
+        LocalAddress.Addr[AddressIndex + 1] = (UINT8)Uintn;\r
+        AddressIndex                       += 2;\r
       } else {\r
         //\r
         // Get P, then exit the loop.\r
         //\r
         Status = AsciiStrDecimalToUintnS (Pointer, &End, &Uintn);\r
-        if (RETURN_ERROR (Status) || End == Pointer || Uintn > 128) {\r
+        if (RETURN_ERROR (Status) || (End == Pointer) || (Uintn > 128)) {\r
           //\r
           // Prefix length should not exceed 128.\r
           //\r
           return RETURN_UNSUPPORTED;\r
         }\r
-        LocalPrefixLength = (UINT8) Uintn;\r
-        Pointer = End;\r
+\r
+        LocalPrefixLength = (UINT8)Uintn;\r
+        Pointer           = End;\r
         break;\r
       }\r
     }\r
@@ -3246,18 +3216,21 @@ AsciiStrToIpv6Address (
       //\r
       break;\r
     }\r
+\r
     Pointer++;\r
   }\r
 \r
-  if ((AddressIndex == ARRAY_SIZE (Address->Addr) && CompressStart != ARRAY_SIZE (Address->Addr)) ||\r
-    (AddressIndex != ARRAY_SIZE (Address->Addr) && CompressStart == ARRAY_SIZE (Address->Addr))\r
-      ) {\r
+  if (((AddressIndex == ARRAY_SIZE (Address->Addr)) && (CompressStart != ARRAY_SIZE (Address->Addr))) ||\r
+      ((AddressIndex != ARRAY_SIZE (Address->Addr)) && (CompressStart == ARRAY_SIZE (Address->Addr)))\r
+      )\r
+  {\r
     //\r
     // Full length of address shall not have compressing zeros.\r
     // Non-full length of address shall have compressing zeros.\r
     //\r
     return RETURN_UNSUPPORTED;\r
   }\r
+\r
   CopyMem (&Address->Addr[0], &LocalAddress.Addr[0], CompressStart);\r
   ZeroMem (&Address->Addr[CompressStart], ARRAY_SIZE (Address->Addr) - AddressIndex);\r
   if (AddressIndex > CompressStart) {\r
@@ -3266,14 +3239,14 @@ AsciiStrToIpv6Address (
       &LocalAddress.Addr[CompressStart],\r
       AddressIndex - CompressStart\r
       );\r
-\r
   }\r
 \r
   if (PrefixLength != NULL) {\r
     *PrefixLength = LocalPrefixLength;\r
   }\r
+\r
   if (EndPointer != NULL) {\r
-    *EndPointer = (CHAR8 *) Pointer;\r
+    *EndPointer = (CHAR8 *)Pointer;\r
   }\r
 \r
   return RETURN_SUCCESS;\r
@@ -3297,10 +3270,6 @@ AsciiStrToIpv6Address (
   When /P is in the String, the function stops at the first character that is not\r
   a valid decimal digit character after P is converted.\r
 \r
-  If String is NULL, then ASSERT().\r
-\r
-  If Address is NULL, then ASSERT().\r
-\r
   If EndPointer is not NULL and Address is translated from String, a pointer\r
   to the character that stopped the scan is stored at the location pointed to\r
   by EndPointer.\r
@@ -3325,18 +3294,18 @@ AsciiStrToIpv6Address (
 RETURN_STATUS\r
 EFIAPI\r
 AsciiStrToIpv4Address (\r
-  IN  CONST CHAR8        *String,\r
-  OUT CHAR8              **EndPointer, OPTIONAL\r
-  OUT IPv4_ADDRESS       *Address,\r
-  OUT UINT8              *PrefixLength OPTIONAL\r
+  IN  CONST CHAR8   *String,\r
+  OUT CHAR8         **EndPointer  OPTIONAL,\r
+  OUT IPv4_ADDRESS  *Address,\r
+  OUT UINT8         *PrefixLength OPTIONAL\r
   )\r
 {\r
-  RETURN_STATUS          Status;\r
-  UINTN                  AddressIndex;\r
-  UINTN                  Uintn;\r
-  IPv4_ADDRESS           LocalAddress;\r
-  UINT8                  LocalPrefixLength;\r
-  CHAR8                  *Pointer;\r
+  RETURN_STATUS  Status;\r
+  UINTN          AddressIndex;\r
+  UINTN          Uintn;\r
+  IPv4_ADDRESS   LocalAddress;\r
+  UINT8          LocalPrefixLength;\r
+  CHAR8          *Pointer;\r
 \r
   LocalPrefixLength = MAX_UINT8;\r
 \r
@@ -3346,7 +3315,7 @@ AsciiStrToIpv4Address (
   SAFE_STRING_CONSTRAINT_CHECK ((String != NULL), RETURN_INVALID_PARAMETER);\r
   SAFE_STRING_CONSTRAINT_CHECK ((Address != NULL), RETURN_INVALID_PARAMETER);\r
 \r
-  for (Pointer = (CHAR8 *) String, AddressIndex = 0; AddressIndex < ARRAY_SIZE (Address->Addr) + 1;) {\r
+  for (Pointer = (CHAR8 *)String, AddressIndex = 0; AddressIndex < ARRAY_SIZE (Address->Addr) + 1;) {\r
     if (!InternalAsciiIsDecimalDigitCharacter (*Pointer)) {\r
       //\r
       // D or P contains invalid characters.\r
@@ -3357,10 +3326,11 @@ AsciiStrToIpv4Address (
     //\r
     // Get D or P.\r
     //\r
-    Status = AsciiStrDecimalToUintnS ((CONST CHAR8 *) Pointer, &Pointer, &Uintn);\r
+    Status = AsciiStrDecimalToUintnS ((CONST CHAR8 *)Pointer, &Pointer, &Uintn);\r
     if (RETURN_ERROR (Status)) {\r
       return RETURN_UNSUPPORTED;\r
     }\r
+\r
     if (AddressIndex == ARRAY_SIZE (Address->Addr)) {\r
       //\r
       // It's P.\r
@@ -3368,7 +3338,8 @@ AsciiStrToIpv4Address (
       if (Uintn > 32) {\r
         return RETURN_UNSUPPORTED;\r
       }\r
-      LocalPrefixLength = (UINT8) Uintn;\r
+\r
+      LocalPrefixLength = (UINT8)Uintn;\r
     } else {\r
       //\r
       // It's D.\r
@@ -3376,7 +3347,8 @@ AsciiStrToIpv4Address (
       if (Uintn > MAX_UINT8) {\r
         return RETURN_UNSUPPORTED;\r
       }\r
-      LocalAddress.Addr[AddressIndex] = (UINT8) Uintn;\r
+\r
+      LocalAddress.Addr[AddressIndex] = (UINT8)Uintn;\r
       AddressIndex++;\r
     }\r
 \r
@@ -3416,6 +3388,7 @@ AsciiStrToIpv4Address (
   if (PrefixLength != NULL) {\r
     *PrefixLength = LocalPrefixLength;\r
   }\r
+\r
   if (EndPointer != NULL) {\r
     *EndPointer = Pointer;\r
   }\r
@@ -3454,9 +3427,6 @@ AsciiStrToIpv4Address (
                   oo          Data4[48:55]\r
                   pp          Data4[56:63]\r
 \r
-  If String is NULL, then ASSERT().\r
-  If Guid is NULL, then ASSERT().\r
-\r
   @param  String                   Pointer to a Null-terminated ASCII string.\r
   @param  Guid                     Pointer to the converted GUID.\r
 \r
@@ -3469,12 +3439,12 @@ AsciiStrToIpv4Address (
 RETURN_STATUS\r
 EFIAPI\r
 AsciiStrToGuid (\r
-  IN  CONST CHAR8        *String,\r
-  OUT GUID               *Guid\r
+  IN  CONST CHAR8  *String,\r
+  OUT GUID         *Guid\r
   )\r
 {\r
-  RETURN_STATUS          Status;\r
-  GUID                   LocalGuid;\r
+  RETURN_STATUS  Status;\r
+  GUID           LocalGuid;\r
 \r
   //\r
   // None of String or Guid shall be a null pointer.\r
@@ -3485,49 +3455,53 @@ AsciiStrToGuid (
   //\r
   // Get aabbccdd in big-endian.\r
   //\r
-  Status = AsciiStrHexToBytes (String, 2 * sizeof (LocalGuid.Data1), (UINT8 *) &LocalGuid.Data1, sizeof (LocalGuid.Data1));\r
-  if (RETURN_ERROR (Status) || String[2 * sizeof (LocalGuid.Data1)] != '-') {\r
+  Status = AsciiStrHexToBytes (String, 2 * sizeof (LocalGuid.Data1), (UINT8 *)&LocalGuid.Data1, sizeof (LocalGuid.Data1));\r
+  if (RETURN_ERROR (Status) || (String[2 * sizeof (LocalGuid.Data1)] != '-')) {\r
     return RETURN_UNSUPPORTED;\r
   }\r
+\r
   //\r
   // Convert big-endian to little-endian.\r
   //\r
   LocalGuid.Data1 = SwapBytes32 (LocalGuid.Data1);\r
-  String += 2 * sizeof (LocalGuid.Data1) + 1;\r
+  String         += 2 * sizeof (LocalGuid.Data1) + 1;\r
 \r
   //\r
   // Get eeff in big-endian.\r
   //\r
-  Status = AsciiStrHexToBytes (String, 2 * sizeof (LocalGuid.Data2), (UINT8 *) &LocalGuid.Data2, sizeof (LocalGuid.Data2));\r
-  if (RETURN_ERROR (Status) || String[2 * sizeof (LocalGuid.Data2)] != '-') {\r
+  Status = AsciiStrHexToBytes (String, 2 * sizeof (LocalGuid.Data2), (UINT8 *)&LocalGuid.Data2, sizeof (LocalGuid.Data2));\r
+  if (RETURN_ERROR (Status) || (String[2 * sizeof (LocalGuid.Data2)] != '-')) {\r
     return RETURN_UNSUPPORTED;\r
   }\r
+\r
   //\r
   // Convert big-endian to little-endian.\r
   //\r
   LocalGuid.Data2 = SwapBytes16 (LocalGuid.Data2);\r
-  String += 2 * sizeof (LocalGuid.Data2) + 1;\r
+  String         += 2 * sizeof (LocalGuid.Data2) + 1;\r
 \r
   //\r
   // Get gghh in big-endian.\r
   //\r
-  Status = AsciiStrHexToBytes (String, 2 * sizeof (LocalGuid.Data3), (UINT8 *) &LocalGuid.Data3, sizeof (LocalGuid.Data3));\r
-  if (RETURN_ERROR (Status) || String[2 * sizeof (LocalGuid.Data3)] != '-') {\r
+  Status = AsciiStrHexToBytes (String, 2 * sizeof (LocalGuid.Data3), (UINT8 *)&LocalGuid.Data3, sizeof (LocalGuid.Data3));\r
+  if (RETURN_ERROR (Status) || (String[2 * sizeof (LocalGuid.Data3)] != '-')) {\r
     return RETURN_UNSUPPORTED;\r
   }\r
+\r
   //\r
   // Convert big-endian to little-endian.\r
   //\r
   LocalGuid.Data3 = SwapBytes16 (LocalGuid.Data3);\r
-  String += 2 * sizeof (LocalGuid.Data3) + 1;\r
+  String         += 2 * sizeof (LocalGuid.Data3) + 1;\r
 \r
   //\r
   // Get iijj.\r
   //\r
   Status = AsciiStrHexToBytes (String, 2 * 2, &LocalGuid.Data4[0], 2);\r
-  if (RETURN_ERROR (Status) || String[2 * 2] != '-') {\r
+  if (RETURN_ERROR (Status) || (String[2 * 2] != '-')) {\r
     return RETURN_UNSUPPORTED;\r
   }\r
+\r
   String += 2 * 2 + 1;\r
 \r
   //\r
@@ -3556,17 +3530,6 @@ AsciiStrToGuid (
   decoding stops after Length of characters and outputs Buffer containing\r
   (Length / 2) bytes.\r
 \r
-  If String is NULL, then ASSERT().\r
-\r
-  If Buffer is NULL, then ASSERT().\r
-\r
-  If Length is not multiple of 2, then ASSERT().\r
-\r
-  If PcdMaximumAsciiStringLength is not zero and Length is greater than\r
-  PcdMaximumAsciiStringLength, then ASSERT().\r
-\r
-  If MaxBufferSize is less than (Length / 2), then ASSERT().\r
-\r
   @param  String                   Pointer to a Null-terminated ASCII string.\r
   @param  Length                   The number of ASCII characters to decode.\r
   @param  Buffer                   Pointer to the converted bytes array.\r
@@ -3587,13 +3550,13 @@ AsciiStrToGuid (
 RETURN_STATUS\r
 EFIAPI\r
 AsciiStrHexToBytes (\r
-  IN  CONST CHAR8        *String,\r
-  IN  UINTN              Length,\r
-  OUT UINT8              *Buffer,\r
-  IN  UINTN              MaxBufferSize\r
+  IN  CONST CHAR8  *String,\r
+  IN  UINTN        Length,\r
+  OUT UINT8        *Buffer,\r
+  IN  UINTN        MaxBufferSize\r
   )\r
 {\r
-  UINTN                  Index;\r
+  UINTN  Index;\r
 \r
   //\r
   // 1. None of String or Buffer shall be a null pointer.\r
@@ -3626,6 +3589,7 @@ AsciiStrHexToBytes (
       break;\r
     }\r
   }\r
+\r
   if (Index != Length) {\r
     return RETURN_UNSUPPORTED;\r
   }\r
@@ -3633,18 +3597,17 @@ AsciiStrHexToBytes (
   //\r
   // Convert the hex string to bytes.\r
   //\r
-  for(Index = 0; Index < Length; Index++) {\r
-\r
+  for (Index = 0; Index < Length; Index++) {\r
     //\r
     // For even characters, write the upper nibble for each buffer byte,\r
     // and for even characters, the lower nibble.\r
     //\r
     if ((Index & BIT0) == 0) {\r
-      Buffer[Index / 2]  = (UINT8) InternalAsciiHexCharToUintn (String[Index]) << 4;\r
+      Buffer[Index / 2] = (UINT8)InternalAsciiHexCharToUintn (String[Index]) << 4;\r
     } else {\r
-      Buffer[Index / 2] |= (UINT8) InternalAsciiHexCharToUintn (String[Index]);\r
+      Buffer[Index / 2] |= (UINT8)InternalAsciiHexCharToUintn (String[Index]);\r
     }\r
   }\r
+\r
   return RETURN_SUCCESS;\r
 }\r
-\r