Re: [edk2] [PATCH v2] NetworkPkg:Fix Network memory leak when calling GetModeData interface

2016-01-21 Thread Fu, Siyuan
Hi, Lubo 1. Below lines in Ip6Dxe/ComponentName.c changes original If condition is incorrect. The first else means Status is an EFI error, so we shouldn't check the IsStarted flag. - if (!EFI_ERROR (Status) && Ip6ModeData.IsStarted) { -Status = NetLibIp6ToStr (&Ip6ModeData.ConfigData.Statio

[edk2] [PATCH v2] NetworkPkg:Fix Network memory leak when calling GetModeData interface

2016-01-21 Thread Zhang Lubo
v2: * Check for NULL pointer before Free them. Multiple network protocols have a GetModeData() interface, which may allocate memory resource in the return mode data structure. It's callers responsibility to free these buffers. Cc: Fu Siyuan Cc: Ye Ting Cc: Wu Jiaxin Contributed-under: TianoCo