diff options
| author | yongari <yongari@FreeBSD.org> | 2006-08-12 01:24:37 +0000 |
|---|---|---|
| committer | yongari <yongari@FreeBSD.org> | 2006-08-12 01:24:37 +0000 |
| commit | 88c94f7a066113a020df0ee72d4030f30217047f (patch) | |
| tree | 273bc48388a0b6c71c62c8a94ea1f562a646ca05 /sys/dev/stge | |
| parent | c9f13fce81bceabdc304be6565e5ca3d7c7195d7 (diff) | |
| download | FreeBSD-src-88c94f7a066113a020df0ee72d4030f30217047f.zip FreeBSD-src-88c94f7a066113a020df0ee72d4030f30217047f.tar.gz | |
Fix invalid reference of mbuf chains.
Use proper pointer dereference to inform modified mbuf chains to
caller.
In collaboration with: glebius
Diffstat (limited to 'sys/dev/stge')
| -rw-r--r-- | sys/dev/stge/if_stge.c | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/sys/dev/stge/if_stge.c b/sys/dev/stge/if_stge.c index f346ca3..10dddae 100644 --- a/sys/dev/stge/if_stge.c +++ b/sys/dev/stge/if_stge.c @@ -1206,7 +1206,7 @@ stge_encap(struct stge_softc *sc, struct mbuf **m_head) { struct stge_txdesc *txd; struct stge_tfd *tfd; - struct mbuf *m, *n; + struct mbuf *m; struct m_tag *mtag; bus_dma_segment_t txsegs[STGE_MAXTXSEGS]; int error, i, nsegs, si; @@ -1217,32 +1217,32 @@ stge_encap(struct stge_softc *sc, struct mbuf **m_head) if ((txd = STAILQ_FIRST(&sc->sc_cdata.stge_txfreeq)) == NULL) return (ENOBUFS); - m = *m_head; error = bus_dmamap_load_mbuf_sg(sc->sc_cdata.stge_tx_tag, - txd->tx_dmamap, m, txsegs, &nsegs, 0); + txd->tx_dmamap, *m_head, txsegs, &nsegs, 0); if (error == EFBIG) { - n = m_defrag(m, M_DONTWAIT); - if (n == NULL) { - m_freem(m); - m = NULL; + m = m_defrag(*m_head, M_DONTWAIT); + if (m == NULL) { + m_freem(*m_head); + *m_head = NULL; return (ENOMEM); } - m = n; + *m_head = m; error = bus_dmamap_load_mbuf_sg(sc->sc_cdata.stge_tx_tag, - txd->tx_dmamap, m, txsegs, &nsegs, 0); + txd->tx_dmamap, *m_head, txsegs, &nsegs, 0); if (error != 0) { - m_freem(m); - m = NULL; + m_freem(*m_head); + *m_head = NULL; return (error); } } else if (error != 0) return (error); if (nsegs == 0) { - m_freem(m); - m = NULL; + m_freem(*m_head); + *m_head = NULL; return (EIO); } + m = *m_head; csum_flags = 0; if ((m->m_pkthdr.csum_flags & STGE_CSUM_FEATURES) != 0) { if (m->m_pkthdr.csum_flags & CSUM_IP) |
