subject:"\[edk2\] \[Patch 2\/3\] MdeModulePkg\/DxeNetLib\: Add parameter check and ASSERT handling."

Re: [edk2] [Patch 2/3] MdeModulePkg/DxeNetLib: Add parameter check and ASSERT handling.

2018-01-02 Thread Wu, Jiaxin

Reviewed-by: Jiaxin Wu > -Original Message- > From: Wang, Fan > Sent: Wednesday, January 3, 2018 10:32 AM > To: edk2-devel@lists.01.org > Cc: Wang, Fan ; Fu, Siyuan ; Ye, > Ting ; Wu, Jiaxin

[edk2] [Patch 2/3] MdeModulePkg/DxeNetLib: Add parameter check and ASSERT handling.

2018-01-02 Thread fanwang2

From: Wang Fan * Library API should check the input parameters before use, or ASSERT to tell it has to meet some requirements. But in DxeNetLib, not all functions follows this rule. * ASSERT shouldn't be used as error handling, add some handling code for errors. * Add