Update return FALSE to ASSERT() for code consistent.
authortye1 <tye1@6f19259b-4bc3-4df7-8a09-765794883524>
Fri, 4 Nov 2011 02:55:30 +0000 (02:55 +0000)
committertye1 <tye1@6f19259b-4bc3-4df7-8a09-765794883524>
Fri, 4 Nov 2011 02:55:30 +0000 (02:55 +0000)
Signed-off-by: tye
Reviewed-by: jyao1
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@12659 6f19259b-4bc3-4df7-8a09-765794883524

CryptoPkg/Library/BaseCryptLib/Pem/CryptPem.c
CryptoPkg/Library/BaseCryptLib/Pk/CryptAuthenticode.c
CryptoPkg/Library/BaseCryptLib/Pk/CryptPkcs7.c
CryptoPkg/Library/BaseCryptLib/Pk/CryptX509.c

index ed53b2d..0b37adc 100644 (file)
@@ -84,10 +84,7 @@ RsaGetPrivateKeyFromPem (
   //\r
   ASSERT (PemData    != NULL);\r
   ASSERT (RsaContext != NULL);\r
-\r
-  if (PemSize > INT_MAX) {\r
-    return FALSE;\r
-  }\r
+  ASSERT (PemSize    <= INT_MAX);\r
 \r
   Status = FALSE;\r
   PemBio = NULL;\r
index 0370fd7..0321b20 100644 (file)
@@ -60,14 +60,11 @@ AuthenticodeVerify (
   UINTN        ContentSize;\r
 \r
   //\r
-  // ASSERT if Authenticode Signature Data or PE Image Hash is NULL\r
+  // ASSERT if Authenticode Signature Data or PE Image Hash is NULL.\r
   //\r
   ASSERT (AuthData  != NULL);\r
   ASSERT (ImageHash != NULL);\r
-\r
-  if (DataSize > INT_MAX) {\r
-    return FALSE;\r
-  }\r
+  ASSERT (DataSize  <= INT_MAX);\r
 \r
   Status       = FALSE;\r
   Pkcs7        = NULL;\r
index c00d3bb..47bab10 100644 (file)
@@ -150,17 +150,13 @@ Pkcs7Sign (
   //\r
   // Check input parameters.\r
   //\r
-  if ((PrivateKey == NULL) || (KeyPassword == NULL) || (InData == NULL)) {\r
-    return FALSE;\r
-  }\r
-  \r
-  if ((SignCert == NULL) || (SignedData == NULL) || (SignedDataSize == NULL)) {\r
-    return FALSE;\r
-  }\r
-\r
-  if (InDataSize > INT_MAX) {\r
-    return FALSE;\r
-  }\r
+  ASSERT (PrivateKey     != NULL);\r
+  ASSERT (KeyPassword    != NULL);\r
+  ASSERT (InData         != NULL);\r
+  ASSERT (SignCert       != NULL);\r
+  ASSERT (SignedData     != NULL);\r
+  ASSERT (SignedDataSize != NULL);\r
+  ASSERT (InDataSize     <= INT_MAX);\r
 \r
   RsaContext = NULL;\r
   Key        = NULL;\r
@@ -311,13 +307,14 @@ Pkcs7Verify (
   BOOLEAN     Wrapped;\r
 \r
   //\r
-  // ASSERT if P7Data is NULL or P7Length is not larger than 19 bytes.\r
+  // ASSERT if any input parameter is invalid.\r
   //\r
-  ASSERT ((P7Data != NULL) || (P7Length <= 19));\r
-\r
-  if ((CertLength > INT_MAX) || (DataLength > INT_MAX)) {\r
-    return FALSE;\r
-  }\r
+  ASSERT (P7Data      != NULL);\r
+  ASSERT (TrustedCert != NULL);\r
+  ASSERT (InData      != NULL);\r
+  ASSERT (P7Length    <= INT_MAX);\r
+  ASSERT (CertLength  <= INT_MAX);\r
+  ASSERT (DataLength  <= INT_MAX);\r
 \r
   Status    = FALSE;\r
   Pkcs7     = NULL;\r
index 88c2136..3a5485e 100644 (file)
@@ -45,12 +45,9 @@ X509ConstructCertificate (
   //\r
   // ASSERT if Cert is NULL or SingleX509Cert is NULL.\r
   //\r
-  ASSERT (Cert != NULL);\r
+  ASSERT (Cert           != NULL);\r
   ASSERT (SingleX509Cert != NULL);\r
-\r
-  if (CertSize > INT_MAX) {\r
-    return FALSE;\r
-  }\r
+  ASSERT (CertSize       <= INT_MAX);\r
 \r
   Status = FALSE;\r
 \r