On 03/07/18 16:57, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau <marcandre.lur...@redhat.com> > > Commit 4cc2b63bd829426b05bad0d8952f1855a10d6ed7 fixed an out of bounds > ZeroMem() call. However, as Laszlo Ersek pointed out, the intent was > to clear all but the Identifier (to revert the effect of > RegisterHashInterfaceLib()). For that, it should clear the > SupportedHashMask too. > > Cc: Jiewen Yao <jiewen....@intel.com> > Cc: Chao Zhang <chao.b.zh...@intel.com> > Cc: Star Zeng <star.z...@intel.com> > Cc: Laszlo Ersek <ler...@redhat.com> > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Marc-André Lureau <marcandre.lur...@redhat.com> > --- > .../Library/HashLibBaseCryptoRouter/HashLibBaseCryptoRouterPei.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git > a/SecurityPkg/Library/HashLibBaseCryptoRouter/HashLibBaseCryptoRouterPei.c > b/SecurityPkg/Library/HashLibBaseCryptoRouter/HashLibBaseCryptoRouterPei.c > index 361a4f6508a0..bf6e1336ee76 100644 > --- a/SecurityPkg/Library/HashLibBaseCryptoRouter/HashLibBaseCryptoRouterPei.c > +++ b/SecurityPkg/Library/HashLibBaseCryptoRouter/HashLibBaseCryptoRouterPei.c > @@ -426,6 +426,7 @@ HashLibBaseCryptoRouterPeiConstructor ( > // > ZeroMem (&HashInterfaceHob->HashInterface, sizeof > (HashInterfaceHob->HashInterface)); > HashInterfaceHob->HashInterfaceCount = 0; > + HashInterfaceHob->SupportedHashMask = 0; > } > > // >
Reviewed-by: Laszlo Ersek <ler...@redhat.com> Thanks, Laszlo