On Thu, Jan 27, 2011 at 04:28:26PM +0100, Joerg Goltermann wrote:
> >Any hints which functions/code should be inspected deeply to find the leak?
> 
> I think i got it, maybe we should replace some more m_free()'s
> with m_freem()'s?
> 
> OK?
> 

yes, absolutely, OK for the diff.

>  - Joerg
> 
> Index: if_ethersubr.c
> ===================================================================
> RCS file: /cvs/src/sys/net/if_ethersubr.c,v
> retrieving revision 1.147
> diff -u -p -r1.147 if_ethersubr.c
> --- if_ethersubr.c      11 Oct 2010 11:31:14 -0000      1.147
> +++ if_ethersubr.c      27 Jan 2011 15:24:26 -0000
> @@ -670,7 +670,7 @@ ether_input(ifp0, eh, m)
>          * If packet has been filtered by the bpf listener, drop it now
>          */
>         if (m->m_flags & M_FILDROP) {
> -               m_free(m);
> +               m_freem(m);
>                 return;
>         }

Reply via email to