d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Don't leak mbufs and mbuf clusters in several error-handling situations

Browse Source 
in the if_an receive routine.

Found with:	Coverity Prevent (tm)
MFC after:	1 week
This commit is contained in:
rwatson 2006-01-15 12:09:03 +00:00
parent 428bff839a
commit a447e5c6e2
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
sys/dev/an/if_an.c
View File

@ -937,6 +937,7 @@ an_rxeof(struct an_softc *sc)
/* Read NIC frame header */
if (an_read_data(sc, id, 0, (caddr_t)&rx_frame,
sizeof(rx_frame))) {
m_freem(m);
ifp->if_ierrors++;
return;
}
@ -945,16 +946,19 @@ an_rxeof(struct an_softc *sc)
if (an_read_data(sc, id, 0x34,
(caddr_t)&rx_frame_802_3,
sizeof(rx_frame_802_3))) {
m_freem(m);
ifp->if_ierrors++;
return;
}
if (rx_frame_802_3.an_rx_802_3_status != 0) {
m_freem(m);
ifp->if_ierrors++;
return;
}
/* Check for insane frame length */
len = rx_frame_802_3.an_rx_802_3_payload_len;
if (len > sizeof(sc->buf_802_11)) {
m_freem(m);
printf("an%d: oversized packet "
"received (%d, %d)\n",
sc->an_unit, len, MCLBYTES);