Sorry for late reply. I have write a very simple test case for these new APIs. 
And the test result is as expected.
Refer to 
https://github.com/ZhichaoGao/edk2/commit/31938b606c6a6a1fdb560e3d0dd4e41a78e1d7e9
The section to test VerifyEKUsInPkcs7Signature is refer to 
https://github.com/Microsoft/mu_tiano_plus/tree/release/201808/CryptoPkg/UnitTests/VerifyPkcs7EkuUnitTestApp

Thanks,
Zhichao

> -----Original Message-----
> From: Yao, Jiewen
> Sent: Monday, March 25, 2019 4:22 PM
> To: Gao, Zhichao <zhichao....@intel.com>; edk2-devel@lists.01.org
> Cc: Ye, Ting <ting...@intel.com>; Michael Turner
> <michael.tur...@microsoft.com>; Bret Barkelew
> <bret.barke...@microsoft.com>; Gao, Liming <liming....@intel.com>;
> Gang Wei <gang....@intel.com>
> Subject: RE: [edk2] [PATCH 0/6] Add new APIs for BaseCryptLib
> 
> Hi
> Would you please describe what unit test has been run for the new API ?
> 
> 
> > -----Original Message-----
> > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> > Zhichao Gao
> > Sent: Monday, March 25, 2019 12:01 PM
> > To: edk2-devel@lists.01.org
> > Cc: Ye, Ting <ting...@intel.com>; Michael Turner
> > <michael.tur...@microsoft.com>; Bret Barkelew
> > <bret.barke...@microsoft.com>; Gao, Liming <liming....@intel.com>;
> > Gang Wei <gang....@intel.com>
> > Subject: [edk2] [PATCH 0/6] Add new APIs for BaseCryptLib
> >
> > Add new API to get organization name
> > Add new API VerifyEKUsInPkcs7Signature Add PKCS1v2 (RSAES-OAEP)
> > support
> >
> > Contributed-under: TianoCore Contribution Agreement 1.1
> > Signed-off-by: Zhichao Gao <zhichao....@intel.com>
> > Cc: Ting Ye <ting...@intel.com>
> > Cc: Gang Wei <gang....@intel.com>
> > Cc: Wang Jian J <jian.j.w...@intel.com>
> > Cc: Liming Gao <liming....@intel.com>
> > Cc: Sean Brogan <sean.bro...@microsoft.com>
> > Cc: Michael Turner <michael.tur...@microsoft.com>
> > Cc: Bret Barkelew <bret.barke...@microsoft.com>
> >
> > Bret Barkelew (6):
> >   CryptoPkg/BaseCryptLib.h: Add new API to get organization name
> >   CryptoPkg/BaseCryptLib: Add new API to get organization name
> >   CryptoPkg/BaseCryptLib.h: Add new API VerifyEKUsInPkcs7Signature
> >   CryptoPkg/BaseCryptLib: Add new API VerifyEKUsInPkcs7Signature
> >   CryptoPkg/BaseCryptLib.h: Add PKCS1v2 (RSAES-OAEP) support.
> >   CryptoPkg/BaseCryptLib: Add PKCS1v2 (RSAES-OAEP) support.
> >
> >  CryptoPkg/Include/Library/BaseCryptLib.h           | 121 ++++-
> >  CryptoPkg/Library/BaseCryptLib/BaseCryptLib.inf    |   2 +
> >  CryptoPkg/Library/BaseCryptLib/PeiCryptLib.inf     |   4 +-
> >  CryptoPkg/Library/BaseCryptLib/Pk/CryptPkcs1Oaep.c | 218 +++++++++
> >  .../Library/BaseCryptLib/Pk/CryptPkcs1OaepNull.c   |  61 +++
> >  .../Library/BaseCryptLib/Pk/CryptPkcs7VerifyEku.c  | 539
> > +++++++++++++++++++++
> >  .../BaseCryptLib/Pk/CryptPkcs7VerifyEkuRuntime.c   |  75 +++
> >  CryptoPkg/Library/BaseCryptLib/Pk/CryptX509.c      | 102 +++-
> >  CryptoPkg/Library/BaseCryptLib/Pk/CryptX509Null.c  |  32 ++
> >  CryptoPkg/Library/BaseCryptLib/RuntimeCryptLib.inf |   2 +
> >  CryptoPkg/Library/BaseCryptLib/SmmCryptLib.inf     |   2 +
> >  11 files changed, 1144 insertions(+), 14 deletions(-)  create mode
> > 100644 CryptoPkg/Library/BaseCryptLib/Pk/CryptPkcs1Oaep.c
> >  create mode 100644
> > CryptoPkg/Library/BaseCryptLib/Pk/CryptPkcs1OaepNull.c
> >  create mode 100644
> > CryptoPkg/Library/BaseCryptLib/Pk/CryptPkcs7VerifyEku.c
> >  create mode 100644
> > CryptoPkg/Library/BaseCryptLib/Pk/CryptPkcs7VerifyEkuRuntime.c
> >
> > --
> > 2.16.2.windows.1
> >
> > _______________________________________________
> > edk2-devel mailing list
> > edk2-devel@lists.01.org
> > https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to