On 01/14/19 16:19, Star Zeng wrote: > 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; > } > >
Reviewed-by: Laszlo Ersek <[email protected]> Thanks, Laszlo _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
