I am surprised DuetPkg is still being used. Reviewed-by: Ruiyu Ni <[email protected]>
Regards, Ray >-----Original Message----- >From: edk2-devel [mailto:[email protected]] On Behalf Of >Yonghong Zhu >Sent: Wednesday, August 2, 2017 5:28 PM >To: [email protected] >Cc: Ni, Ruiyu <[email protected]>; Wu, Hao A <[email protected]>; Andrew >Fish <[email protected]> >Subject: [edk2] [Patch 2/4] DuetPkg: Fix Xcode 9 Beta treating 32-bit left >shift as undefined > >Bug: https://bugzilla.tianocore.org/show_bug.cgi?id=635 > >Cc: Ruiyu Ni <[email protected]> >Cc: Hao Wu <[email protected]> >Contributed-under: TianoCore Contribution Agreement 1.0 >Signed-off-by: Andrew Fish <[email protected]> >--- > DuetPkg/EfiLdr/TianoDecompress.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > >diff --git a/DuetPkg/EfiLdr/TianoDecompress.c >b/DuetPkg/EfiLdr/TianoDecompress.c >index b504e8e..86d82fc 100644 >--- a/DuetPkg/EfiLdr/TianoDecompress.c >+++ b/DuetPkg/EfiLdr/TianoDecompress.c >@@ -91,15 +91,15 @@ Arguments: > > Returns: (VOID) > > --*/ > { >- Sd->mBitBuf = (UINT32) (Sd->mBitBuf << NumOfBits); >+ Sd->mBitBuf = (UINT32) (((UINT64)Sd->mBitBuf) << NumOfBits); > > while (NumOfBits > Sd->mBitCount) { > >- Sd->mBitBuf |= (UINT32) (Sd->mSubBitBuf << (NumOfBits = (UINT16) >(NumOfBits - Sd->mBitCount))); >+ Sd->mBitBuf |= (UINT32) (((UINT64)Sd->mSubBitBuf) << (NumOfBits = >(UINT16) (NumOfBits - Sd->mBitCount))); > > if (Sd->mCompSize > 0) { > // > // Get 1 byte into SubBitBuf > // >-- >2.6.1.windows.1 > >_______________________________________________ >edk2-devel mailing list >[email protected] >https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
