Re: [edk2-devel] [PATCH v1 1/1] CryptoPkg: Redefinition bug in CrtLibSupport.h.
Merged: https://github.com/tianocore/edk2/pull/2692 > -Original Message- > From: devel@edk2.groups.io On Behalf Of Yao, Jiewen > Sent: Saturday, March 26, 2022 2:50 PM > To: devel@edk2.groups.io; Li, Zhihao > Subject: Re: [edk2-devel] [PATCH v1 1/1] CryptoPkg: Redefinition bug in > CrtLibSupport.h. > > Reviewed-by: Jiewen Yao > > > -Original Message- > > From: devel@edk2.groups.io On Behalf Of Li, Zhihao > > Sent: Friday, March 25, 2022 3:41 PM > > To: devel@edk2.groups.io > > Subject: [edk2-devel] [PATCH v1 1/1] CryptoPkg: Redefinition bug in > > CrtLibSupport.h. > > > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3885 > > > > Parallel hash patch redefines uint_64 type in CrtLibSupport.h > > which has been defined in openssl/include/openssl/e_os2.h. > > CryptMd5.c including e_os2.h cause redefinition bug. > > > > Cc: Jiewen Yao jiewen@intel.com > > Cc: Jian J Wang jian.j.w...@intel.com > > Cc: Xiaoyu Lu xiaoyu1...@intel.com > > Cc: Guomin Jiang guomin.ji...@intel.com > > Cc: Siyuan Fu siyuan...@intel.com > > > > Signed-off-by: Zhihao Li > > --- > > CryptoPkg/Library/BaseCryptLib/Hash/CryptParallelHash.h | 2 ++ > > CryptoPkg/Library/Include/CrtLibSupport.h | 1 - > > 2 files changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/CryptoPkg/Library/BaseCryptLib/Hash/CryptParallelHash.h > > b/CryptoPkg/Library/BaseCryptLib/Hash/CryptParallelHash.h > > index fe08d4928e8d..dcfe200e5829 100644 > > --- a/CryptoPkg/Library/BaseCryptLib/Hash/CryptParallelHash.h > > +++ b/CryptoPkg/Library/BaseCryptLib/Hash/CryptParallelHash.h > > @@ -25,6 +25,8 @@ http://creativecommons.org/publicdomain/zero/1.0/ > > > > > > #define KECCAK1600_WIDTH 1600 > > > > > > > > +typedef UINT64 uint64_t; > > > > + > > > > // > > > > // This struct referring to m_sha3.c from opessl and modified its type > > name. > > > > // > > > > diff --git a/CryptoPkg/Library/Include/CrtLibSupport.h > > b/CryptoPkg/Library/Include/CrtLibSupport.h > > index b76b140a7acf..75172b920b67 100644 > > --- a/CryptoPkg/Library/Include/CrtLibSupport.h > > +++ b/CryptoPkg/Library/Include/CrtLibSupport.h > > @@ -111,7 +111,6 @@ typedef UINT8 u_char; > > typedef UINT32 uid_t; > > > > typedef UINT32 gid_t; > > > > typedef CHAR16 wchar_t; > > > > -typedef UINT64 uint64_t; > > > > > > > > // > > > > // File operations are not required for EFI building, > > > > -- > > 2.26.2.windows.1 > > > > > > > > > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#88064): https://edk2.groups.io/g/devel/message/88064 Mute This Topic: https://groups.io/mt/90017880/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v1 1/1] CryptoPkg: Redefinition bug in CrtLibSupport.h.
Reviewed-by: Jiewen Yao > -Original Message- > From: devel@edk2.groups.io On Behalf Of Li, Zhihao > Sent: Friday, March 25, 2022 3:41 PM > To: devel@edk2.groups.io > Subject: [edk2-devel] [PATCH v1 1/1] CryptoPkg: Redefinition bug in > CrtLibSupport.h. > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3885 > > Parallel hash patch redefines uint_64 type in CrtLibSupport.h > which has been defined in openssl/include/openssl/e_os2.h. > CryptMd5.c including e_os2.h cause redefinition bug. > > Cc: Jiewen Yao jiewen@intel.com > Cc: Jian J Wang jian.j.w...@intel.com > Cc: Xiaoyu Lu xiaoyu1...@intel.com > Cc: Guomin Jiang guomin.ji...@intel.com > Cc: Siyuan Fu siyuan...@intel.com > > Signed-off-by: Zhihao Li > --- > CryptoPkg/Library/BaseCryptLib/Hash/CryptParallelHash.h | 2 ++ > CryptoPkg/Library/Include/CrtLibSupport.h | 1 - > 2 files changed, 2 insertions(+), 1 deletion(-) > > diff --git a/CryptoPkg/Library/BaseCryptLib/Hash/CryptParallelHash.h > b/CryptoPkg/Library/BaseCryptLib/Hash/CryptParallelHash.h > index fe08d4928e8d..dcfe200e5829 100644 > --- a/CryptoPkg/Library/BaseCryptLib/Hash/CryptParallelHash.h > +++ b/CryptoPkg/Library/BaseCryptLib/Hash/CryptParallelHash.h > @@ -25,6 +25,8 @@ http://creativecommons.org/publicdomain/zero/1.0/ > > > #define KECCAK1600_WIDTH 1600 > > > > +typedef UINT64 uint64_t; > > + > > // > > // This struct referring to m_sha3.c from opessl and modified its type name. > > // > > diff --git a/CryptoPkg/Library/Include/CrtLibSupport.h > b/CryptoPkg/Library/Include/CrtLibSupport.h > index b76b140a7acf..75172b920b67 100644 > --- a/CryptoPkg/Library/Include/CrtLibSupport.h > +++ b/CryptoPkg/Library/Include/CrtLibSupport.h > @@ -111,7 +111,6 @@ typedef UINT8 u_char; > typedef UINT32 uid_t; > > typedef UINT32 gid_t; > > typedef CHAR16 wchar_t; > > -typedef UINT64 uint64_t; > > > > // > > // File operations are not required for EFI building, > > -- > 2.26.2.windows.1 > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#88061): https://edk2.groups.io/g/devel/message/88061 Mute This Topic: https://groups.io/mt/90017880/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[edk2-devel] [PATCH v1 1/1] CryptoPkg: Redefinition bug in CrtLibSupport.h.
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3885 Parallel hash patch redefines uint_64 type in CrtLibSupport.h which has been defined in openssl/include/openssl/e_os2.h. CryptMd5.c including e_os2.h cause redefinition bug. Cc: Jiewen Yao jiewen@intel.com Cc: Jian J Wang jian.j.w...@intel.com Cc: Xiaoyu Lu xiaoyu1...@intel.com Cc: Guomin Jiang guomin.ji...@intel.com Cc: Siyuan Fu siyuan...@intel.com Signed-off-by: Zhihao Li --- CryptoPkg/Library/BaseCryptLib/Hash/CryptParallelHash.h | 2 ++ CryptoPkg/Library/Include/CrtLibSupport.h | 1 - 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/CryptoPkg/Library/BaseCryptLib/Hash/CryptParallelHash.h b/CryptoPkg/Library/BaseCryptLib/Hash/CryptParallelHash.h index fe08d4928e8d..dcfe200e5829 100644 --- a/CryptoPkg/Library/BaseCryptLib/Hash/CryptParallelHash.h +++ b/CryptoPkg/Library/BaseCryptLib/Hash/CryptParallelHash.h @@ -25,6 +25,8 @@ http://creativecommons.org/publicdomain/zero/1.0/ #define KECCAK1600_WIDTH 1600 +typedef UINT64 uint64_t; + // // This struct referring to m_sha3.c from opessl and modified its type name. // diff --git a/CryptoPkg/Library/Include/CrtLibSupport.h b/CryptoPkg/Library/Include/CrtLibSupport.h index b76b140a7acf..75172b920b67 100644 --- a/CryptoPkg/Library/Include/CrtLibSupport.h +++ b/CryptoPkg/Library/Include/CrtLibSupport.h @@ -111,7 +111,6 @@ typedef UINT8 u_char; typedef UINT32 uid_t; typedef UINT32 gid_t; typedef CHAR16 wchar_t; -typedef UINT64 uint64_t; // // File operations are not required for EFI building, -- 2.26.2.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#88023): https://edk2.groups.io/g/devel/message/88023 Mute This Topic: https://groups.io/mt/90017880/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-