Reviewed-by: Hao Wu <[email protected]> Best Regards, Hao Wu
> -----Original Message----- > From: Zeng, Star > Sent: Monday, January 14, 2019 11:20 PM > To: [email protected] > Cc: Zeng, Star; Wang, Jian J; Wu, Hao A; Laszlo Ersek > Subject: [PATCH V2 07/15] MdeModulePkg Variable: type case VolatileBase > to UINTN directly > > Simplify > ((UINTN) ((UINT8 *) VolatileBase + VolatileBase->Size)) > to > ((UINTN) VolatileBase + VolatileBase->Size) > > This patch prepares for adding emulated variable NV mode > support in VariableRuntimeDxe. > > Cc: Jian J Wang <[email protected]> > Cc: Hao Wu <[email protected]> > Cc: Laszlo Ersek <[email protected]> > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Star Zeng <[email protected]> > --- > MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c > b/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c > index 4d524db30fec..4c8e1d8cf4f1 100644 > --- a/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c > +++ b/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c > @@ -301,7 +301,7 @@ UpdateVariableStore ( > DataPtr += mVariableModuleGlobal- > >VariableGlobal.VolatileVariableBase; > } > > - if ((DataPtr + DataSize) > ((UINTN) ((UINT8 *) VolatileBase + > VolatileBase- > >Size))) { > + if ((DataPtr + DataSize) > ((UINTN) VolatileBase + VolatileBase->Size)) { > return EFI_OUT_OF_RESOURCES; > } > > -- > 2.7.0.windows.1 _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel

