On 03/07/18 16:57, [email protected] wrote: > From: Marc-André Lureau <[email protected]> > > 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 <[email protected]> > Cc: Chao Zhang <[email protected]> > Cc: Star Zeng <[email protected]> > Cc: Laszlo Ersek <[email protected]> > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Marc-André Lureau <[email protected]> > --- > .../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 <[email protected]> Thanks, Laszlo _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
