Subject: Re: pppoe & mbuf chain
To: YAMAMOTO Takashi <yamt@mwd.biglobe.ne.jp>
From: David Laight <david@l8s.co.uk>
List: tech-net
Date: 06/21/2002 17:39:17
On Sat, Jun 22, 2002 at 01:24:24AM +0900, YAMAMOTO Takashi wrote:
> From: David Laight <david@l8s.co.uk>
> > > +			m = m_pullup(m, m->m_len);
> > 
> > you want to free the original 'm' which you have carefully
> > lost!
> 
> - if m_pullup fails, it frees "original m".
> - m_freem(0) is safe.
> 
> am i missing something?

Ok maybe I didn't check char m_pullup() does, but every
other pullup() I've seen doesn't.
Also discarding the old data block isn't friendly!

	David

-- 
David Laight: david@l8s.co.uk