Re: [edk2] [Patch] MdeModulePkg: Free NET_BUF data after it is sent out to avoid memory leak

2017-11-22 Thread Wu, Jiaxin
Hi Fan, Looks this patch will cause the null pointer dereference issue, see below analysis: With this patch, the NET_BUF will be freed and the corresponding Arg (Packet) will also be freed in DhcpReleasePacket. Wrap = NetbufFromExt (, 1, 0, 0, DhcpReleasePacket, Packet); That's

Re: [edk2] [Patch] MdeModulePkg: Free NET_BUF data after it is sent out to avoid memory leak

2017-11-21 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan > -Original Message- > From: Wang, Fan > Sent: Wednesday, November 22, 2017 2:56 PM > To: edk2-devel@lists.01.org > Cc: Wu, Jiaxin ; Ye, Ting ; Fu, > Siyuan ; Wang, Fan

[edk2] [Patch] MdeModulePkg: Free NET_BUF data after it is sent out to avoid memory leak

2017-11-21 Thread fanwang2
When build a DHCP message in function DhcpSendMessage() or DhcpRetransmit(), a new NET_BUF is created by the library of NetbufFromExt, but it's not freed after it is sent out. This patch is to fix this memory leak issue. Cc: Jiaxin Wu Cc: Ye Ting Cc: Fu