On December 14, 2021 10:01 PM, Ard Biesheuvel wrote:
> On Tue, 14 Dec 2021 at 14:42, Min Xu <min.m...@intel.com> wrote:
> > --- a/EmbeddedPkg/Library/PrePiLib/FwVol.c
> > +++ b/EmbeddedPkg/Library/PrePiLib/FwVol.c
> > @@ -335,7 +335,7 @@ FfsProcessSection (
> >
> >          Status = UefiDecompressGetInfo (
> >                     CompressedData,
> > -                   CompressedDataLength,
> > +                   (UINT32)CompressedDataLength,
> 
> How is this change related to the items listed in the commit log?
> 
> If there are code issues affecting this library, please fix them in a separate
> patch.
> 
CompressedDataLength is declared as UINTN which is UINT64 in X64 arch. But the 
second parameter of UefiDecompressGetInfo() is declared as UINT32. So a build 
error is triggered.
  RETURN_STATUS
  EFIAPI
  UefiDecompressGetInfo (
    IN  CONST VOID  *Source,
    IN  UINT32      SourceSize,
    OUT UINT32      *DestinationSize,
    OUT UINT32      *ScratchSize
    );

I will fix it in a separate patch.

Thanks
Min


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#84959): https://edk2.groups.io/g/devel/message/84959
Mute This Topic: https://groups.io/mt/87720796/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to