diff options
author | stas <stas@FreeBSD.org> | 2008-12-06 14:23:45 +0000 |
---|---|---|
committer | stas <stas@FreeBSD.org> | 2008-12-06 14:23:45 +0000 |
commit | 278fe65d806dc84125dade5682e8b6d551ab23ca (patch) | |
tree | abf5e99ccfdc7f90ac11f5e78823bb6f395f3eb8 /sys/dev/ae | |
parent | 19922f97dbfdbb70fa3ccc3a18c0145c52d56da0 (diff) | |
download | FreeBSD-src-278fe65d806dc84125dade5682e8b6d551ab23ca.zip FreeBSD-src-278fe65d806dc84125dade5682e8b6d551ab23ca.tar.gz |
- Eliminate unused variable. [1]
- Check for runt frames entering the stack. [2]
Suggested by: ganbold[1], yongari[2]
Approved by: kib (mentor)
MFC after: 2 weeks
Diffstat (limited to 'sys/dev/ae')
-rw-r--r-- | sys/dev/ae/if_ae.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/sys/dev/ae/if_ae.c b/sys/dev/ae/if_ae.c index da2a420..345a01f 100644 --- a/sys/dev/ae/if_ae.c +++ b/sys/dev/ae/if_ae.c @@ -1105,12 +1105,9 @@ ae_dmamap_cb(void *arg, bus_dma_segment_t *segs, int nsegs, int error) static int ae_alloc_rings(ae_softc_t *sc) { - bus_dma_tag_t bustag; bus_addr_t busaddr; int error; - bustag = bus_get_dma_tag(sc->dev); - /* * Create parent DMA tag. */ @@ -1903,8 +1900,8 @@ ae_rxeof(ae_softc_t *sc, ae_rxd_t *rxd) if_printf(ifp, "Rx interrupt occuried.\n"); #endif size = le16toh(rxd->len) - ETHER_CRC_LEN; - if (size < 0) { - if_printf(ifp, "Negative length packet received."); + if (size < (ETHER_MIN_LEN - ETHER_CRC_LEN - ETHER_VLAN_ENCAP_LEN)) { + if_printf(ifp, "Runt frame received."); return (EIO); } |