Hi Guomin, Where are the typos? Please point out all the typos you found; it makes easy for Zhichao to fix all the typos. Thanks, Eric From: devel@edk2.groups.io On Behalf Of Guomin Jiang Sent: Friday, May 8, 2020 3:24 PM To: devel@edk2.groups.io; Gao, Zhichao Cc: Wang, Jian J ; Lu, XiaoyuX ; Fu, Siyuan ; Kinney, Michael D ; Yao, Jiewen Subject: Re: [edk2-devel] [PATCH V3 0/8] CryptoPkg: Retire the deprecated functions It seem ok except some typo, please modify it. Reviewed-by: Guomin Jiang > > -----Original Message----- > From: devel@edk2.groups.io > On Behalf Of Gao, > Zhichao > Sent: Thursday, May 7, 2020 7:58 AM > To: devel@edk2.groups.io > Cc: Wang, Jian J >; Lu, XiaoyuX >; > Fu, Siyuan >; Kinney, Michael D > >; Yao, Jiewen > > Subject: [edk2-devel] [PATCH V3 0/8] CryptoPkg: Retire the deprecated > functions > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1898 > > Retire below deprecated functions: > MD4, ARC4, TDES, AES ECB MODE, HMAC MD5, HMAC SHA1. > > V2: > Seprate the MD5 and SHA1 changes to other patch set. > Change the deprecated function wrapper implementatuion in > EDKII_CRYPTO_PROTOCOL to call a dummy function to print debug message > and assert. > > V3: > Remove the missing part of deprecated functions in V2. > Change the name field in the EDKII_CRYPTO_PROTOCOL to indicate the > function is deprecated and unsupported. > > Cc: Jian J Wang > > Cc: Xiaoyu Lu > > Cc: Siyuan Fu > > Cc: Michael D Kinney > > Cc: Jiewen Yao > > Signed-off-by: Zhichao Gao > > > Zhichao Gao (8): > CryptoPkg/CryptoDxe: Add function to indicate the deprecated algorithm > CryptoPkg/BaseCrpytLib: Retire MD4 algorithm > CryptoPkg/BaseCryptLib: Retire ARC4 algorithm > CryptoPkg/BaseCryptLib: Retire the Tdes algorithm > CryptoPkg/BaseCryptLib: Retire Aes Ecb mode algorithm > CryptoPkg/BaseCryptLib: Retire HMAC MD5 algorithm > CryptoPkg/BaseCryptLib: Retire HMAC SHA1 algorithm > CryptoPkg/Crypto.h: Update the version of Crypto Driver > > CryptoPkg/CryptoPkg.dsc | 46 +- > CryptoPkg/Driver/Crypto.c | 780 ++--------- > CryptoPkg/Include/Library/BaseCryptLib.h | 809 ------------ > .../Library/BaseCryptLib/BaseCryptLib.inf | 7 +- > .../Library/BaseCryptLib/Cipher/CryptAes.c | 114 -- > .../BaseCryptLib/Cipher/CryptAesNull.c | 52 - > .../Library/BaseCryptLib/Cipher/CryptArc4.c | 205 --- > .../BaseCryptLib/Cipher/CryptArc4Null.c | 124 -- > .../Library/BaseCryptLib/Cipher/CryptTdes.c | 364 ------ > .../BaseCryptLib/Cipher/CryptTdesNull.c | 160 --- > .../Library/BaseCryptLib/Hash/CryptMd4.c | 223 ---- > .../Library/BaseCryptLib/Hash/CryptMd4Null.c | 143 -- > .../Library/BaseCryptLib/Hmac/CryptHmacMd5.c | 216 --- > .../BaseCryptLib/Hmac/CryptHmacMd5Null.c | 139 -- > .../Library/BaseCryptLib/Hmac/CryptHmacSha1.c | 216 --- > .../BaseCryptLib/Hmac/CryptHmacSha1Null.c | 139 -- > .../Library/BaseCryptLib/PeiCryptLib.inf | 11 +- > .../Library/BaseCryptLib/PeiCryptLib.uni | 4 +- > CryptoPkg/Library/BaseCryptLib/Pem/CryptPem.c | 7 +- > .../Library/BaseCryptLib/RuntimeCryptLib.inf | 11 +- > .../Library/BaseCryptLib/RuntimeCryptLib.uni | 4 +- > .../Library/BaseCryptLib/SmmCryptLib.inf | 12 +- > .../Library/BaseCryptLib/SmmCryptLib.uni | 4 +- > .../BaseCryptLibNull/BaseCryptLibNull.inf | 5 - > .../BaseCryptLibNull/Cipher/CryptAesNull.c | 52 - > .../BaseCryptLibNull/Cipher/CryptArc4Null.c | 124 -- > .../BaseCryptLibNull/Cipher/CryptTdesNull.c | 160 --- > .../BaseCryptLibNull/Hash/CryptMd4Null.c | 143 -- > .../BaseCryptLibNull/Hmac/CryptHmacMd5Null.c | 139 -- > .../BaseCryptLibNull/Hmac/CryptHmacSha1Null.c | 139 -- > .../BaseCryptLibOnProtocolPpi/CryptLib.c | 1157 ++--------------- > .../Library/Include/openssl/opensslconf.h | 9 + > CryptoPkg/Library/OpensslLib/OpensslLib.inf | 28 - > .../Library/OpensslLib/OpensslLibCrypto.inf | 28 - > CryptoPkg/Private/Protocol/Crypto.h | 708 ++-------- > 35 files changed, 389 insertions(+), 6093 deletions(-) delete mode 100644 > CryptoPkg/Library/BaseCryptLib/Cipher/CryptArc4.c > delete mode 100644 > CryptoPkg/Library/BaseCryptLib/Cipher/CryptArc4Null.c > delete mode 100644 CryptoPkg/Library/BaseCryptLib/Cipher/CryptTdes.c > delete mode 100644 > CryptoPkg/Library/BaseCryptLib/Cipher/CryptTdesNull.c > delete mode 100644 CryptoPkg/Library/BaseCryptLib/Hash/CryptMd4.c > delete mode 100644 CryptoPkg/Library/BaseCryptLib/Hash/CryptMd4Null.c > delete mode 100644 > CryptoPkg/Library/BaseCryptLib/Hmac/CryptHmacMd5.c > delete mode 100644 > CryptoPkg/Library/BaseCryptLib/Hmac/CryptHmacMd5Null.c > delete mode 100644 > CryptoPkg/Library/BaseCryptLib/Hmac/CryptHmacSha1.c > delete mode 100644 > CryptoPkg/Library/BaseCryptLib/Hmac/CryptHmacSha1Null.c > delete mode 100644 > CryptoPkg/Library/BaseCryptLibNull/Cipher/CryptArc4Null.c > delete mode 100644 > CryptoPkg/Library/BaseCryptLibNull/Cipher/CryptTdesNull.c > delete mode 100644 > CryptoPkg/Library/BaseCryptLibNull/Hash/CryptMd4Null.c > delete mode 100644 > CryptoPkg/Library/BaseCryptLibNull/Hmac/CryptHmacMd5Null.c > delete mode 100644 > CryptoPkg/Library/BaseCryptLibNull/Hmac/CryptHmacSha1Null.c > > -- > 2.21.0.windows.1 > > >