Re: Mbuf leak in if_lagg.c

From: Alexandre Martins <alexandre.martins_at_stormshield.eu>
Date: Fri, 27 Mar 2015 13:11:07 +0100
Le jeudi 26 mars 2015, 23:10:49 Andrey V. Elsukov a écrit :
> On 26.03.2015 22:42, Andrey V. Elsukov wrote:
> >> If lp_detaching is non 0, the mbuf pointer is set to NULL without m_freem
> >> it.
> >> 
> >> Can you look at this ?
> > 
> > Hi,
> > 
> > what you thing about this patch?
> > lp_detaching can be non zero in case of parent interface departure.
> > So I don't see the reason to call ETHER_BPF_MTAP() in this case.
> 
> Now I see the reason - to capture all received packets before interface
> departure. New is attached.

Sounds good for me :)

-- 
Alexandre Martins
STORMSHIELD


Received on Fri Mar 27 2015 - 11:10:16 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:56 UTC