Subject: Re: CVS commit: src/sys/dist/ipf/netinet
To: Martti Kuparinen <martti@netbsd.org>
From: Pavel Cahyna <pavel@netbsd.org>
List: source-changes
Date: 09/17/2007 21:58:24
On Mon, Sep 17, 2007 at 06:25:24AM +0000, Martti Kuparinen wrote:
>
> Module Name: src
> Committed By: martti
> Date: Mon Sep 17 06:25:24 UTC 2007
>
> Modified Files:
> src/sys/dist/ipf/netinet: fil.c ip_fil_netbsd.c
>
> Log Message:
> Returning from m_pulldown can leave the first mbut with no data (m_len = 0).
> The mbuf chain wasn't being walked and thus fin_m was not updated to point
> to the new first mbuf with data in it.
Don't you leak the first mbuf in the chain when walking it like that?
Pavel