Subject: Re: CVS commit: src/sys/dev/qbus
To: None <bouyer@netbsd.org, source-changes@netbsd.org>
From: Matt Thomas <matt@3am-software.com>
List: source-changes
Date: 01/15/2003 15:22:54
At 02:10 PM 1/15/2003, Manuel BOUYER wrote:

>Module Name:    src
>Committed By:   bouyer
>Date:           Wed Jan 15 22:10:26 UTC 2003
>
>Modified Files:
>         src/sys/dev/qbus: if_qe.c
>
>Log Message:
>Zero out the extra space stolen from mbuf to pad packet to ETHER_MIN_LEN
>XXX should it be ETHER_MIN_LEN - ETHER_CRC_LEN ?
>XXX2 we assume there are enouth space in the mbuf for the padding bytes.
>      But other places in the code assumes this already.

Do not assume you can always write into the mbuf.  See M_READONLY(m) or
M_TRAILINGSPACE(m)


-- 
Matt Thomas               Internet:   matt@3am-software.com
3am Software Foundry      WWW URL:    http://www.3am-software.com/bio/matt/
Cupertino, CA             Disclaimer: I avow all knowledge of this message