Re: [edk2-devel] [PATCH EDK2 v1 1/1] EmulatorPkg/host: fix overflow in Mult

2020-09-22 Thread wenyi,xie via groups.io
evel@edk2.groups.io On Behalf Of wenyi,xie >> via groups.io >> Sent: Tuesday, September 22, 2020 11:43 AM >> To: devel@edk2.groups.io; Justen, Jordan L ; >> af...@apple.com; Ni, Ray >> Cc: songdongku...@huawei.com >> Subject: Re: [edk2-devel] [PATCH E

Re: [edk2-devel] [PATCH EDK2 v1 1/1] EmulatorPkg/host: fix overflow in Mult

2020-09-22 Thread Ni, Ray
020 11:43 AM > To: devel@edk2.groups.io; Justen, Jordan L ; > af...@apple.com; Ni, Ray > Cc: songdongku...@huawei.com > Subject: Re: [edk2-devel] [PATCH EDK2 v1 1/1] EmulatorPkg/host: fix overflow > in Mult > > Hi all, > > Please review this patch when you're free, t

Re: [edk2-devel] [PATCH EDK2 v1 1/1] EmulatorPkg/host: fix overflow in Mult

2020-09-21 Thread wenyi,xie via groups.io
Hi all, Please review this patch when you're free, thanks. Regards Wenyi On 2020/9/1 18:58, Wenyi Xie wrote: > REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2947 > > When calculating memory regions and store the information in the > gSystemMemory in file WinHost.c, the code below will caus

Re: [edk2-devel] [PATCH EDK2 v1 1/1] EmulatorPkg/host: fix overflow in Mult

2020-09-01 Thread Laszlo Ersek
Ray, On 09/01/20 12:58, wenyi,xie via groups.io wrote: > REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2947 > > When calculating memory regions and store the information in the > gSystemMemory in file WinHost.c, the code below will cause overflow, > because _wtoi (MemorySizeStr) return an in

[edk2-devel] [PATCH EDK2 v1 1/1] EmulatorPkg/host: fix overflow in Mult

2020-09-01 Thread wenyi,xie via groups.io
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2947 When calculating memory regions and store the information in the gSystemMemory in file WinHost.c, the code below will cause overflow, because _wtoi (MemorySizeStr) return an int value and SIZE_1MB is also an int value, if MemorySizeStr is lag