Subject: Re: CVS commit: syssrc/sys/net
To: None <itojun@iijlab.net>
From: Darren Reed <darrenr@reed.wattle.id.au>
List: tech-net
Date: 09/26/2002 15:54:45
In some email I received from itojun@iijlab.net, sie wrote:
> >On Wed, Sep 25, 2002 at 10:58:02PM +1000, Darren Reed wrote:
> >
> > > Ok, explain to me why we're not fixing the problem of the mbuf being
> > > badly initialised here and instead passing a 'bad mbuf' to bpf_mtap().
> > >
> > > If M_PKTHDR is set, then the contents of m_pkthdr are *meant* to be
> > > valid or at least that's what everyone _tells me_ (hi Jason!).
>
> the change you should have made with sys/net/bpf.c change was
> to update all bpf_mtap() calls from DLT_NULL interfaces (uses "struct
> mbuf" without initialization like those done in MGET). i had no time
> for that, and because of the severety of the bug, i backed the change
> to sys/net/bpf.c out.
Well, I've fixed up ppp (I hope).
Are there any interfaces, besides lo0, that are DLT_NULL ?
I'll look at loopback next.
Darren