Re: [edk2] [Patch] MdeModulePkg: Add wrong/invalid subnet check

2016-11-10 Thread Wu, Jiaxin
Thanks Laszlo, your suggestions make sense. Best Regards! Jiaxin > -Original Message- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Thursday, November 10, 2016 5:19 PM > To: Wu, Jiaxin ; edk2-de...@ml01.01.org > Cc: Santhapur Naveen ; Ye, Ting > ; Fu, Siyuan > Subject: Re: [Pat

Re: [edk2] [Patch] MdeModulePkg: Add wrong/invalid subnet check

2016-11-10 Thread Laszlo Ersek
I have a few comments: On 11/10/16 09:45, Jiaxin Wu wrote: > This patch is used to add the wrong/invalid subnet check. > Meanwhile, correct the the return status. (1) I propose to split this patch into three patches, with the following subjects: MdeModulePkg/Ip4Dxe: Catch invalid subnet early in

[edk2] [Patch] MdeModulePkg: Add wrong/invalid subnet check

2016-11-10 Thread Jiaxin Wu
This patch is used to add the wrong/invalid subnet check. Meanwhile, correct the the return status. Cc: Santhapur Naveen Cc: Laszlo Ersek Cc: Ye Ting Cc: Fu Siyuan Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiaxin Wu --- MdeModulePkg/Universal/Network/Ip4Dxe/Ip4Co