回复: [edk2-devel] 回复: [edk2-stable202108] Package: CryptoPkg/BaseCryptLib: fix incorrect param order

[gaoliming]

Sure. I will merge this patch. 

Thanks
Liming

> -----邮件原件-----
> 发件人: devel at edk2.groups.io <devel at edk2.groups.io> 代表 Yao, Jiewen
> 发送时间: 2021年8月18日 9:20
> 收件人: devel at edk2.groups.io; gaoliming at byosoft.com.cn; 'Stewart, Chris
> (SW Architect)' <chris.stewart at hp.com>
> 抄送: Wang, Jian J <jian.j.wang at intel.com>; Lu, XiaoyuX
> <xiaoyux.lu at intel.com>; Jiang, Guomin <guomin.jiang at intel.com>; 'Leif
> Lindholm' <leif at nuviainc.com>; 'Andrew Fish' <afish at apple.com>; Kinney,
> Michael D <michael.d.kinney at intel.com>; Yao, Jiewen
> <jiewen.yao at intel.com>
> 主题: Re: [edk2-devel] 回复: [edk2-stable202108] Package:
> CryptoPkg/BaseCryptLib: fix incorrect param order
> 
> Reviewed-by: Jiewen Yao <Jiewen.yao at intel.com>
> 
> Thanks Liming. Yes. I agree to merge it for stable tag.
> Will you help to do that?
> 
> > -----Original Message-----
> > From: devel at edk2.groups.io <devel at edk2.groups.io> On Behalf Of
> gaoliming
> > Sent: Wednesday, August 18, 2021 9:18 AM
> > To: 'Stewart, Chris (SW Architect)' <chris.stewart at hp.com>;
> > devel at edk2.groups.io
> > Cc: Yao, Jiewen <jiewen.yao at intel.com>; Wang, Jian J
> <jian.j.wang at intel.com>;
> > Lu, XiaoyuX <xiaoyux.lu at intel.com>; Jiang, Guomin
> <guomin.jiang at intel.com>;
> > 'Leif Lindholm' <leif at nuviainc.com>; 'Andrew Fish' <afish at apple.com>;
> Kinney,
> > Michael D <michael.d.kinney at intel.com>
> > Subject: [edk2-devel] 回复: [edk2-stable202108] Package:
> > CryptoPkg/BaseCryptLib: fix incorrect param order
> >
> > Reviewed-by: Liming Gao <gaoliming at byosoft.com.cn>
> >
> > This is a clear bug fix. I suggest to merge it for this stable tag 202108.
> >
> > Thanks
> > Liming
> > > -----邮件原件-----
> > > 发件人: Stewart, Chris (SW Architect) <chris.stewart at hp.com>
> > > 发送时间: 2021年8月17日 20:48
> > > 收件人: devel at edk2.groups.io
> > > 抄送: jiewen.yao at intel.com; jian.j.wang at intel.com;
> xiaoyux.lu at intel.com;
> > > guomin.jiang at intel.com; gaoliming at byosoft.com.cn; Stewart, Chris (SW
> > > Architect) <chris.stewart at hp.com>
> > > 主题: [PATCH] Package: CryptoPkg/BaseCryptLib: fix incorrect param
> order
> > >
> > > [PATCH] Package: CryptoPkg/BaseCryptLib: fix incorrect param order
> > >
> > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3550
> > >
> > > Function ConvertAsn1TimeToEfiTime initializes timestamp to zeroes with
> > > SetMem, but the actual parameters are out of order.
> > > The result is the SetMem operation has no effect. The fix is to put the
> > actual
> > > parameters in the correct order.
> > >
> > > Signed-off-by: Chris Stewart <chris.stewart at hp.com>
> > > Reviewed-by: Liming Gao <gaoliming at byosoft.com.cn>
> > > CC: Jiewen Yao <jiewen.yao at intel.com>
> > > CC: Jian J Wang <jian.j.wang at intel.com>
> > > CC: Xiaoyu Lu <xiaoyux.lu at intel.com>
> > > CC: Guomin Jiang <guomin.jiang at intel.com>
> > > ---
> > >  CryptoPkg/Library/BaseCryptLib/Pk/CryptTs.c | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/CryptoPkg/Library/BaseCryptLib/Pk/CryptTs.c
> > > b/CryptoPkg/Library/BaseCryptLib/Pk/CryptTs.c
> > > index ff7f6488f2..970e9b94a0 100644
> > > --- a/CryptoPkg/Library/BaseCryptLib/Pk/CryptTs.c
> > > +++ b/CryptoPkg/Library/BaseCryptLib/Pk/CryptTs.c
> > > @@ -155,7 +155,7 @@ ConvertAsn1TimeToEfiTime (
> > >    }
> > >
> > >    Str = (CONST CHAR8*)Asn1Time->data;
> > > -  SetMem (EfiTime, 0, sizeof (EFI_TIME));
> > > +  SetMem (EfiTime, sizeof (EFI_TIME), 0);
> > >
> > >    Index = 0;
> > >    if (Asn1Time->type == V_ASN1_UTCTIME) {               /* two
> digit
> > > year */
> > > --
> > > 2.19.2.windows.1
> >
> >
> >
> >
> >
> >
> >
> 
> 
> 
> 
> 





-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#79549): https://edk2.groups.io/g/devel/message/79549
Mute This Topic: https://groups.io/mt/84988513/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-