Re: [edk2] [PATCH] MdeModulePkg: DxeUdpIoLib: fix non-empty payload path in UDP reception

2016-04-04 Thread Fu, Siyuan
Laszlo, Thanks for the quick fix. Siyuan From: Laszlo Ersek [mailto:ler...@redhat.com] Sent: Monday, April 4, 2016 7:56 PM To: Subramanian, Sriram (EG Servers Platform SW) Cc: edk2-devel-01 ; Fu, Siyuan ; Wu, Jiaxin

Re: [edk2] [PATCH] MdeModulePkg: DxeUdpIoLib: fix non-empty payload path in UDP reception

2016-04-04 Thread Laszlo Ersek
On 04/04/16 05:45, Subramanian, Sriram (EG Servers Platform SW) wrote: > Thanks for testing that, Laszlo. > > Reviewed-by: Sriram Subramanian Thanks a lot for the quick review! Since this bug makes OVMF more or less unusable, I pushed the patch (which is also very simple)

Re: [edk2] [PATCH] MdeModulePkg: DxeUdpIoLib: fix non-empty payload path in UDP reception

2016-04-03 Thread Subramanian, Sriram (EG Servers Platform SW)
Thanks for testing that, Laszlo. Reviewed-by: Sriram Subramanian Thanks, Sriram. -Original Message- From: Laszlo Ersek [mailto:ler...@redhat.com] Sent: Sunday, April 3, 2016 3:44 PM To: edk2-devel-01 Cc: Siyuan Fu ;

[edk2] [PATCH] MdeModulePkg: DxeUdpIoLib: fix non-empty payload path in UDP reception

2016-04-03 Thread Laszlo Ersek
Commit 1b31acb66c02 ("MdeModulePkg: Check received packet size before use it.") introduced a chunk of code under the new "Resume" label, in function UdpIoOnDgramRcvdDpc(). The new code is supposed to run only when the received packet has zero-length payload, but a "return" statement was forgotten,