Harti Brandt (Mon, Feb 24, 2003 at 08:46:13PM +0100) wrote:
> On Mon, 24 Feb 2003, Hiten Pandya wrote:
> 
> HP>Craig Rodrigues (Mon, Feb 24, 2003 at 12:07:02PM -0500) wrote:
> HP>> The code in question looks like:
> HP>> =========================================================================
> HP>> struct mbuf *
> HP>> copy_mbuf(struct mbuf *m)
> HP>> {
> HP>>         struct mbuf *new;
> HP>>
> HP>>         MGET(new, M_DONTWAIT, MT_DATA);
> HP>>         if(new == NULL)
> HP>>                 return NULL;
> HP>>         if(m->m_flags & M_PKTHDR)
> HP>>                 M_COPY_PKTHDR(new, m);
> HP>
> HP>What you need, is m_dup_pkthdr().  M_COPY_PKTHDR has been
> HP>deprecated for several reasons, that are outlined in the
> HP>commit log of rev. 1.109 of sys/sys/mbuf.h.
> 
> I wrote that code. It must be a M_MOVE_PKTHDR, because m is just disposed
> afterwards.

OK.  This was mentioned in the comment in sys/mbuf.h and the commit  log
too. :-)

-- 
Hiten Pandya ([EMAIL PROTECTED], [EMAIL PROTECTED])
http://www.unixdaemons.com/~hiten/

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message

Reply via email to