Fix the following bpf(4) race condition which can result in a panic:
(1) bpf peer attaches to interface netif0 (2) Packet is received by netif0 (3) ifp->if_bpf pointer is checked and handed off to bpf (4) bpf peer detaches from netif0 resulting in ifp->if_bpf being initialized to NULL. (5) ifp->if_bpf is dereferenced by bpf machinery (6) Kaboom This race condition likely explains the various different kernel panics reported around sending SIGINT to tcpdump or dhclient processes. But really this race can result in kernel panics anywhere you have frequent bpf attach and detach operations with high packet per second load. Summary of changes: - Remove the bpf interface's "driverp" member - When we attach bpf interfaces, we now set the ifp->if_bpf member to the bpf interface structure. Once this is done, ifp->if_bpf should never be NULL. [1] - Introduce bpf_peers_present function, an inline operation which will do a lockless read bpf peer list associated with the interface. It should be noted that the bpf code will pickup the bpf_interface lock before adding or removing bpf peers. This should serialize the access to the bpf descriptor list, removing the race. - Expose the bpf_if structure in bpf.h so that the bpf_peers_present function can use it. This also removes the struct bpf_if; hack that was there. - Adjust all consumers of the raw if_bpf structure to use bpf_peers_present Now what happens is: (1) Packet is received by netif0 (2) Check to see if bpf descriptor list is empty (3) Pickup the bpf interface lock (4) Hand packet off to process From the attach/detach side: (1) Pickup the bpf interface lock (2) Add/remove from bpf descriptor list Now that we are storing the bpf interface structure with the ifnet, there is is no need to walk the bpf interface list to locate the correct bpf interface. We now simply look up the interface, and initialize the pointer. This has a nice side effect of changing a bpf interface attach operation from O(N) (where N is the number of bpf interfaces), to O(1). [1] From now on, we can no longer check ifp->if_bpf to tell us whether or not we have any bpf peers that might be interested in receiving packets. In collaboration with: sam@ MFC after: 1 month
This commit is contained in:
parent
6dea56bafc
commit
16d878cc99
Notes:
svn2git
2020-12-20 02:59:44 +00:00
svn path=/head/; revision=159180
@ -2905,7 +2905,7 @@ ath_rx_proc(void *arg, int npending)
|
||||
* pass decrypt+mic errors but others may be
|
||||
* interesting (e.g. crc).
|
||||
*/
|
||||
if (sc->sc_drvbpf != NULL &&
|
||||
if (bpf_peers_present(sc->sc_drvbpf) &&
|
||||
(ds->ds_rxstat.rs_status & sc->sc_monpass)) {
|
||||
bus_dmamap_sync(sc->sc_dmat, bf->bf_dmamap,
|
||||
BUS_DMASYNC_POSTREAD);
|
||||
@ -2936,7 +2936,8 @@ ath_rx_proc(void *arg, int npending)
|
||||
|
||||
sc->sc_stats.ast_ant_rx[ds->ds_rxstat.rs_antenna]++;
|
||||
|
||||
if (sc->sc_drvbpf != NULL && !ath_rx_tap(sc, m, ds, tsf, nf)) {
|
||||
if (bpf_peers_present(sc->sc_drvbpf) &&
|
||||
!ath_rx_tap(sc, m, ds, tsf, nf)) {
|
||||
m_freem(m); /* XXX reclaim */
|
||||
goto rx_next;
|
||||
}
|
||||
@ -3636,7 +3637,7 @@ ath_tx_start(struct ath_softc *sc, struct ieee80211_node *ni, struct ath_buf *bf
|
||||
|
||||
if (ic->ic_rawbpf)
|
||||
bpf_mtap(ic->ic_rawbpf, m0);
|
||||
if (sc->sc_drvbpf) {
|
||||
if (bpf_peers_present(sc->sc_drvbpf)) {
|
||||
u_int64_t tsf = ath_hal_gettsf64(ah);
|
||||
|
||||
sc->sc_tx_th.wt_tsf = htole64(tsf);
|
||||
|
@ -1054,7 +1054,7 @@ ipw_data_intr(struct ipw_softc *sc, struct ipw_status *status,
|
||||
m->m_pkthdr.rcvif = ifp;
|
||||
m->m_pkthdr.len = m->m_len = le32toh(status->len);
|
||||
|
||||
if (sc->sc_drvbpf != NULL) {
|
||||
if (bpf_peers_present(sc->sc_drvbpf)) {
|
||||
struct ipw_rx_radiotap_header *tap = &sc->sc_rxtap;
|
||||
|
||||
tap->wr_flags = 0;
|
||||
@ -1328,7 +1328,7 @@ ipw_tx_start(struct ifnet *ifp, struct mbuf *m0, struct ieee80211_node *ni)
|
||||
wh = mtod(m0, struct ieee80211_frame *);
|
||||
}
|
||||
|
||||
if (sc->sc_drvbpf != NULL) {
|
||||
if (bpf_peers_present(sc->sc_drvbpf)) {
|
||||
struct ipw_tx_radiotap_header *tap = &sc->sc_txtap;
|
||||
|
||||
tap->wt_flags = 0;
|
||||
|
@ -1300,7 +1300,7 @@ iwi_frame_intr(struct iwi_softc *sc, struct iwi_rx_data *data, int i,
|
||||
|
||||
m_adj(m, sizeof (struct iwi_hdr) + sizeof (struct iwi_frame));
|
||||
|
||||
if (sc->sc_drvbpf != NULL) {
|
||||
if (bpf_peers_present(sc->sc_drvbpf)) {
|
||||
struct iwi_rx_radiotap_header *tap = &sc->sc_rxtap;
|
||||
|
||||
tap->wr_flags = 0;
|
||||
@ -1829,7 +1829,7 @@ iwi_tx_start(struct ifnet *ifp, struct mbuf *m0, struct ieee80211_node *ni,
|
||||
wh = mtod(m0, struct ieee80211_frame *);
|
||||
}
|
||||
|
||||
if (sc->sc_drvbpf != NULL) {
|
||||
if (bpf_peers_present(sc->sc_drvbpf)) {
|
||||
struct iwi_tx_radiotap_header *tap = &sc->sc_txtap;
|
||||
|
||||
tap->wt_flags = 0;
|
||||
|
@ -1225,7 +1225,7 @@ rt2560_decryption_intr(struct rt2560_softc *sc)
|
||||
m->m_pkthdr.len = m->m_len =
|
||||
(le32toh(desc->flags) >> 16) & 0xfff;
|
||||
|
||||
if (sc->sc_drvbpf != NULL) {
|
||||
if (bpf_peers_present(sc->sc_drvbpf)) {
|
||||
struct rt2560_rx_radiotap_header *tap = &sc->sc_rxtap;
|
||||
uint32_t tsf_lo, tsf_hi;
|
||||
|
||||
@ -1346,7 +1346,7 @@ rt2560_beacon_expire(struct rt2560_softc *sc)
|
||||
|
||||
ieee80211_beacon_update(ic, data->ni, &sc->sc_bo, data->m, 1);
|
||||
|
||||
if (ic->ic_rawbpf != NULL)
|
||||
if (bpf_peers_present(ic->ic_rawbpf))
|
||||
bpf_mtap(ic->ic_rawbpf, data->m);
|
||||
|
||||
rt2560_tx_bcn(sc, data->m, data->ni);
|
||||
@ -1606,7 +1606,7 @@ rt2560_tx_bcn(struct rt2560_softc *sc, struct mbuf *m0,
|
||||
return error;
|
||||
}
|
||||
|
||||
if (sc->sc_drvbpf != NULL) {
|
||||
if (bpf_peers_present(sc->sc_drvbpf)) {
|
||||
struct rt2560_tx_radiotap_header *tap = &sc->sc_txtap;
|
||||
|
||||
tap->wt_flags = 0;
|
||||
@ -1663,7 +1663,7 @@ rt2560_tx_mgt(struct rt2560_softc *sc, struct mbuf *m0,
|
||||
return error;
|
||||
}
|
||||
|
||||
if (sc->sc_drvbpf != NULL) {
|
||||
if (bpf_peers_present(sc->sc_drvbpf)) {
|
||||
struct rt2560_tx_radiotap_header *tap = &sc->sc_txtap;
|
||||
|
||||
tap->wt_flags = 0;
|
||||
@ -1882,7 +1882,7 @@ rt2560_tx_data(struct rt2560_softc *sc, struct mbuf *m0,
|
||||
wh = mtod(m0, struct ieee80211_frame *);
|
||||
}
|
||||
|
||||
if (sc->sc_drvbpf != NULL) {
|
||||
if (bpf_peers_present(sc->sc_drvbpf)) {
|
||||
struct rt2560_tx_radiotap_header *tap = &sc->sc_txtap;
|
||||
|
||||
tap->wt_flags = 0;
|
||||
@ -1961,7 +1961,7 @@ rt2560_start(struct ifnet *ifp)
|
||||
ni = (struct ieee80211_node *)m0->m_pkthdr.rcvif;
|
||||
m0->m_pkthdr.rcvif = NULL;
|
||||
|
||||
if (ic->ic_rawbpf != NULL)
|
||||
if (bpf_peers_present(ic->ic_rawbpf))
|
||||
bpf_mtap(ic->ic_rawbpf, m0);
|
||||
|
||||
if (rt2560_tx_mgt(sc, m0, ni) != 0)
|
||||
@ -1997,7 +1997,7 @@ rt2560_start(struct ifnet *ifp)
|
||||
continue;
|
||||
}
|
||||
|
||||
if (ic->ic_rawbpf != NULL)
|
||||
if (bpf_peers_present(ic->ic_rawbpf))
|
||||
bpf_mtap(ic->ic_rawbpf, m0);
|
||||
|
||||
if (rt2560_tx_data(sc, m0, ni) != 0) {
|
||||
|
@ -1132,7 +1132,7 @@ rt2661_rx_intr(struct rt2661_softc *sc)
|
||||
m->m_pkthdr.len = m->m_len =
|
||||
(le32toh(desc->flags) >> 16) & 0xfff;
|
||||
|
||||
if (sc->sc_drvbpf != NULL) {
|
||||
if (bpf_peers_present(sc->sc_drvbpf)) {
|
||||
struct rt2661_rx_radiotap_header *tap = &sc->sc_rxtap;
|
||||
uint32_t tsf_lo, tsf_hi;
|
||||
|
||||
@ -1480,7 +1480,7 @@ rt2661_tx_mgt(struct rt2661_softc *sc, struct mbuf *m0,
|
||||
return error;
|
||||
}
|
||||
|
||||
if (sc->sc_drvbpf != NULL) {
|
||||
if (bpf_peers_present(sc->sc_drvbpf)) {
|
||||
struct rt2661_tx_radiotap_header *tap = &sc->sc_txtap;
|
||||
|
||||
tap->wt_flags = 0;
|
||||
@ -1704,7 +1704,7 @@ rt2661_tx_data(struct rt2661_softc *sc, struct mbuf *m0,
|
||||
wh = mtod(m0, struct ieee80211_frame *);
|
||||
}
|
||||
|
||||
if (sc->sc_drvbpf != NULL) {
|
||||
if (bpf_peers_present(sc->sc_drvbpf)) {
|
||||
struct rt2661_tx_radiotap_header *tap = &sc->sc_txtap;
|
||||
|
||||
tap->wt_flags = 0;
|
||||
|
@ -956,7 +956,7 @@ ural_rxeof(usbd_xfer_handle xfer, usbd_private_handle priv, usbd_status status)
|
||||
m->m_pkthdr.len = m->m_len = (le32toh(desc->flags) >> 16) & 0xfff;
|
||||
m->m_flags |= M_HASFCS; /* h/w leaves FCS */
|
||||
|
||||
if (sc->sc_drvbpf != NULL) {
|
||||
if (bpf_peers_present(sc->sc_drvbpf)) {
|
||||
struct ural_rx_radiotap_header *tap = &sc->sc_rxtap;
|
||||
|
||||
tap->wr_flags = IEEE80211_RADIOTAP_F_FCS;
|
||||
@ -1204,7 +1204,7 @@ ural_tx_mgt(struct ural_softc *sc, struct mbuf *m0, struct ieee80211_node *ni)
|
||||
flags |= RAL_TX_TIMESTAMP;
|
||||
}
|
||||
|
||||
if (sc->sc_drvbpf != NULL) {
|
||||
if (bpf_peers_present(sc->sc_drvbpf)) {
|
||||
struct ural_tx_radiotap_header *tap = &sc->sc_txtap;
|
||||
|
||||
tap->wt_flags = 0;
|
||||
@ -1293,7 +1293,7 @@ ural_tx_data(struct ural_softc *sc, struct mbuf *m0, struct ieee80211_node *ni)
|
||||
*(uint16_t *)wh->i_dur = htole16(dur);
|
||||
}
|
||||
|
||||
if (sc->sc_drvbpf != NULL) {
|
||||
if (bpf_peers_present(sc->sc_drvbpf)) {
|
||||
struct ural_tx_radiotap_header *tap = &sc->sc_txtap;
|
||||
|
||||
tap->wt_flags = 0;
|
||||
|
@ -961,7 +961,7 @@ wi_start(struct ifnet *ifp)
|
||||
wh = mtod(m0, struct ieee80211_frame *);
|
||||
}
|
||||
#if NBPFILTER > 0
|
||||
if (ic->ic_rawbpf)
|
||||
if (bpf_peers_present(ic->ic_rawbpf))
|
||||
bpf_mtap(ic->ic_rawbpf, m0);
|
||||
#endif
|
||||
frmhdr.wi_tx_ctl = htole16(WI_ENC_TX_802_11|WI_TXCNTL_TX_EX);
|
||||
@ -980,7 +980,7 @@ wi_start(struct ifnet *ifp)
|
||||
frmhdr.wi_tx_ctl |= htole16(WI_TXCNTL_NOCRYPT);
|
||||
}
|
||||
#if NBPFILTER > 0
|
||||
if (sc->sc_drvbpf) {
|
||||
if (bpf_peers_present(ic->ic_rawbpf)) {
|
||||
sc->sc_tx_th.wt_rate =
|
||||
ni->ni_rates.rs_rates[ni->ni_txrate];
|
||||
bpf_mtap2(sc->sc_drvbpf,
|
||||
@ -1534,7 +1534,7 @@ wi_rx_intr(struct wi_softc *sc)
|
||||
}
|
||||
|
||||
#if NBPFILTER > 0
|
||||
if (sc->sc_drvbpf) {
|
||||
if (bpf_peers_present(sc->sc_drvbpf)) {
|
||||
/* XXX replace divide by table */
|
||||
sc->sc_rx_th.wr_rate = frmhdr.wi_rx_rate / 5;
|
||||
sc->sc_rx_th.wr_antsignal = frmhdr.wi_rx_signal;
|
||||
|
@ -301,7 +301,6 @@ bpf_attachd(d, bp)
|
||||
LIST_INSERT_HEAD(&bp->bif_dlist, d, bd_next);
|
||||
|
||||
bpf_bpfd_cnt++;
|
||||
*bp->bif_driverp = bp;
|
||||
BPFIF_UNLOCK(bp);
|
||||
}
|
||||
|
||||
@ -327,12 +326,6 @@ bpf_detachd(d)
|
||||
LIST_REMOVE(d, bd_next);
|
||||
|
||||
bpf_bpfd_cnt--;
|
||||
/*
|
||||
* Let the driver know that there are no more listeners.
|
||||
*/
|
||||
if (LIST_EMPTY(&bp->bif_dlist))
|
||||
*bp->bif_driverp = NULL;
|
||||
|
||||
d->bd_bif = NULL;
|
||||
BPFD_UNLOCK(d);
|
||||
BPFIF_UNLOCK(bp);
|
||||
@ -1106,51 +1099,33 @@ bpf_setif(d, ifr)
|
||||
struct ifnet *theywant;
|
||||
|
||||
theywant = ifunit(ifr->ifr_name);
|
||||
if (theywant == NULL)
|
||||
return ENXIO;
|
||||
if (theywant == NULL || theywant->if_bpf == NULL)
|
||||
return (ENXIO);
|
||||
|
||||
bp = theywant->if_bpf;
|
||||
/*
|
||||
* Look through attached interfaces for the named one.
|
||||
* Allocate the packet buffers if we need to.
|
||||
* If we're already attached to requested interface,
|
||||
* just flush the buffer.
|
||||
*/
|
||||
mtx_lock(&bpf_mtx);
|
||||
LIST_FOREACH(bp, &bpf_iflist, bif_next) {
|
||||
struct ifnet *ifp = bp->bif_ifp;
|
||||
|
||||
if (ifp == NULL || ifp != theywant)
|
||||
continue;
|
||||
/* skip additional entry */
|
||||
if (bp->bif_driverp != &ifp->if_bpf)
|
||||
continue;
|
||||
|
||||
mtx_unlock(&bpf_mtx);
|
||||
/*
|
||||
* We found the requested interface.
|
||||
* Allocate the packet buffers if we need to.
|
||||
* If we're already attached to requested interface,
|
||||
* just flush the buffer.
|
||||
*/
|
||||
if (d->bd_sbuf == NULL) {
|
||||
error = bpf_allocbufs(d);
|
||||
if (error != 0)
|
||||
return (error);
|
||||
}
|
||||
if (bp != d->bd_bif) {
|
||||
if (d->bd_bif)
|
||||
/*
|
||||
* Detach if attached to something else.
|
||||
*/
|
||||
bpf_detachd(d);
|
||||
|
||||
bpf_attachd(d, bp);
|
||||
}
|
||||
BPFD_LOCK(d);
|
||||
reset_d(d);
|
||||
BPFD_UNLOCK(d);
|
||||
return (0);
|
||||
if (d->bd_sbuf == NULL) {
|
||||
error = bpf_allocbufs(d);
|
||||
if (error != 0)
|
||||
return (error);
|
||||
}
|
||||
mtx_unlock(&bpf_mtx);
|
||||
/* Not found. */
|
||||
return (ENXIO);
|
||||
if (bp != d->bd_bif) {
|
||||
if (d->bd_bif)
|
||||
/*
|
||||
* Detach if attached to something else.
|
||||
*/
|
||||
bpf_detachd(d);
|
||||
|
||||
bpf_attachd(d, bp);
|
||||
}
|
||||
BPFD_LOCK(d);
|
||||
reset_d(d);
|
||||
BPFD_UNLOCK(d);
|
||||
return (0);
|
||||
}
|
||||
|
||||
/*
|
||||
@ -1272,13 +1247,6 @@ bpf_tap(bp, pkt, pktlen)
|
||||
struct bpf_d *d;
|
||||
u_int slen;
|
||||
|
||||
/*
|
||||
* Lockless read to avoid cost of locking the interface if there are
|
||||
* no descriptors attached.
|
||||
*/
|
||||
if (LIST_EMPTY(&bp->bif_dlist))
|
||||
return;
|
||||
|
||||
BPFIF_LOCK(bp);
|
||||
LIST_FOREACH(d, &bp->bif_dlist, bd_next) {
|
||||
BPFD_LOCK(d);
|
||||
@ -1339,13 +1307,6 @@ bpf_mtap(bp, m)
|
||||
struct bpf_d *d;
|
||||
u_int pktlen, slen;
|
||||
|
||||
/*
|
||||
* Lockless read to avoid cost of locking the interface if there are
|
||||
* no descriptors attached.
|
||||
*/
|
||||
if (LIST_EMPTY(&bp->bif_dlist))
|
||||
return;
|
||||
|
||||
pktlen = m_length(m, NULL);
|
||||
|
||||
BPFIF_LOCK(bp);
|
||||
@ -1391,13 +1352,6 @@ bpf_mtap2(bp, data, dlen, m)
|
||||
struct bpf_d *d;
|
||||
u_int pktlen, slen;
|
||||
|
||||
/*
|
||||
* Lockless read to avoid cost of locking the interface if there are
|
||||
* no descriptors attached.
|
||||
*/
|
||||
if (LIST_EMPTY(&bp->bif_dlist))
|
||||
return;
|
||||
|
||||
pktlen = m_length(m, NULL);
|
||||
/*
|
||||
* Craft on-stack mbuf suitable for passing to bpf_filter.
|
||||
@ -1589,17 +1543,16 @@ bpfattach2(ifp, dlt, hdrlen, driverp)
|
||||
panic("bpfattach");
|
||||
|
||||
LIST_INIT(&bp->bif_dlist);
|
||||
bp->bif_driverp = driverp;
|
||||
bp->bif_ifp = ifp;
|
||||
bp->bif_dlt = dlt;
|
||||
mtx_init(&bp->bif_mtx, "bpf interface lock", NULL, MTX_DEF);
|
||||
KASSERT(*driverp == NULL, ("bpfattach2: driverp already initialized"));
|
||||
*driverp = bp;
|
||||
|
||||
mtx_lock(&bpf_mtx);
|
||||
LIST_INSERT_HEAD(&bpf_iflist, bp, bif_next);
|
||||
mtx_unlock(&bpf_mtx);
|
||||
|
||||
*bp->bif_driverp = NULL;
|
||||
|
||||
/*
|
||||
* Compute the length of the bpf header. This is not necessarily
|
||||
* equal to SIZEOF_BPF_HDR because we want to insert spacing such
|
||||
|
@ -603,7 +603,18 @@ struct bpf_dltlist {
|
||||
};
|
||||
|
||||
#ifdef _KERNEL
|
||||
struct bpf_if;
|
||||
/*
|
||||
* Descriptor associated with each attached hardware interface.
|
||||
*/
|
||||
struct bpf_if {
|
||||
LIST_ENTRY(bpf_if) bif_next; /* list of all interfaces */
|
||||
LIST_HEAD(, bpf_d) bif_dlist; /* descriptor list */
|
||||
u_int bif_dlt; /* link layer type */
|
||||
u_int bif_hdrlen; /* length of header (with padding) */
|
||||
struct ifnet *bif_ifp; /* corresponding interface */
|
||||
struct mtx bif_mtx; /* mutex for interface */
|
||||
};
|
||||
|
||||
int bpf_validate(const struct bpf_insn *, int);
|
||||
void bpf_tap(struct bpf_if *, u_char *, u_int);
|
||||
void bpf_mtap(struct bpf_if *, struct mbuf *);
|
||||
@ -615,18 +626,25 @@ void bpfdetach(struct ifnet *);
|
||||
void bpfilterattach(int);
|
||||
u_int bpf_filter(const struct bpf_insn *, u_char *, u_int, u_int);
|
||||
|
||||
static __inline int
|
||||
bpf_peers_present(struct bpf_if *bpf)
|
||||
{
|
||||
|
||||
return !LIST_EMPTY(&bpf->bif_dlist);
|
||||
}
|
||||
|
||||
#define BPF_TAP(_ifp,_pkt,_pktlen) do { \
|
||||
if ((_ifp)->if_bpf) \
|
||||
if (bpf_peers_present((_ifp)->if_bpf)) \
|
||||
bpf_tap((_ifp)->if_bpf, (_pkt), (_pktlen)); \
|
||||
} while (0)
|
||||
#define BPF_MTAP(_ifp,_m) do { \
|
||||
if ((_ifp)->if_bpf) { \
|
||||
if (bpf_peers_present((_ifp)->if_bpf)) { \
|
||||
M_ASSERTVALID(_m); \
|
||||
bpf_mtap((_ifp)->if_bpf, (_m)); \
|
||||
} \
|
||||
} while (0)
|
||||
#define BPF_MTAP2(_ifp,_data,_dlen,_m) do { \
|
||||
if ((_ifp)->if_bpf) { \
|
||||
if (bpf_peers_present((_ifp)->if_bpf)) { \
|
||||
M_ASSERTVALID(_m); \
|
||||
bpf_mtap2((_ifp)->if_bpf,(_data),(_dlen),(_m)); \
|
||||
} \
|
||||
|
@ -119,19 +119,6 @@ struct bpf_d {
|
||||
(((bd)->bd_immediate || (bd)->bd_state == BPF_TIMED_OUT) && \
|
||||
(bd)->bd_slen != 0))
|
||||
|
||||
/*
|
||||
* Descriptor associated with each attached hardware interface.
|
||||
*/
|
||||
struct bpf_if {
|
||||
LIST_ENTRY(bpf_if) bif_next; /* list of all interfaces */
|
||||
LIST_HEAD(, bpf_d) bif_dlist; /* descriptor list */
|
||||
struct bpf_if **bif_driverp; /* pointer into softc */
|
||||
u_int bif_dlt; /* link layer type */
|
||||
u_int bif_hdrlen; /* length of header (with padding) */
|
||||
struct ifnet *bif_ifp; /* corresponding interface */
|
||||
struct mtx bif_mtx; /* mutex for interface */
|
||||
};
|
||||
|
||||
/*
|
||||
* External representation of the bpf descriptor
|
||||
*/
|
||||
|
@ -158,7 +158,7 @@ discoutput(struct ifnet *ifp, struct mbuf *m, struct sockaddr *dst,
|
||||
dst->sa_family = af;
|
||||
}
|
||||
|
||||
if (ifp->if_bpf) {
|
||||
if (bpf_peers_present(ifp->if_bpf)) {
|
||||
u_int af = dst->sa_family;
|
||||
bpf_mtap2(ifp->if_bpf, &af, sizeof(af), m);
|
||||
}
|
||||
|
@ -425,9 +425,7 @@ gif_output(ifp, m, dst, rt)
|
||||
}
|
||||
|
||||
af = dst->sa_family;
|
||||
if (ifp->if_bpf) {
|
||||
bpf_mtap2(ifp->if_bpf, &af, sizeof(af), m);
|
||||
}
|
||||
BPF_MTAP2(ifp, &af, sizeof(af), m);
|
||||
ifp->if_opackets++;
|
||||
ifp->if_obytes += m->m_pkthdr.len;
|
||||
|
||||
@ -484,7 +482,7 @@ gif_input(m, af, ifp)
|
||||
mac_create_mbuf_from_ifnet(ifp, m);
|
||||
#endif
|
||||
|
||||
if (ifp->if_bpf) {
|
||||
if (bpf_peers_present(ifp->if_bpf)) {
|
||||
u_int32_t af1 = af;
|
||||
bpf_mtap2(ifp->if_bpf, &af1, sizeof(af1), m);
|
||||
}
|
||||
|
@ -276,7 +276,7 @@ gre_output(struct ifnet *ifp, struct mbuf *m, struct sockaddr *dst,
|
||||
dst->sa_family = af;
|
||||
}
|
||||
|
||||
if (ifp->if_bpf) {
|
||||
if (bpf_peers_present(ifp->if_bpf)) {
|
||||
af = dst->sa_family;
|
||||
bpf_mtap2(ifp->if_bpf, &af, sizeof(af), m);
|
||||
}
|
||||
|
@ -259,7 +259,7 @@ if_simloop(ifp, m, af, hlen)
|
||||
m->m_pkthdr.rcvif = ifp;
|
||||
|
||||
/* Let BPF see incoming packet */
|
||||
if (ifp->if_bpf) {
|
||||
if (bpf_peers_present(ifp->if_bpf)) {
|
||||
if (ifp->if_bpf->bif_dlt == DLT_NULL) {
|
||||
u_int32_t af1 = af; /* XXX beware sizeof(af) != 4 */
|
||||
/*
|
||||
|
@ -662,7 +662,7 @@ sltstart(struct tty *tp)
|
||||
* queueing, and the connection id compression will get
|
||||
* munged when this happens.
|
||||
*/
|
||||
if (SL2IFP(sc)->if_bpf) {
|
||||
if (bpf_peers_present(SL2IFP(sc)->if_bpf)) {
|
||||
/*
|
||||
* We need to save the TCP/IP header before it's
|
||||
* compressed. To avoid complicated code, we just
|
||||
@ -696,7 +696,7 @@ sltstart(struct tty *tp)
|
||||
*mtod(m, u_char *) |= sl_compress_tcp(m, ip,
|
||||
&sc->sc_comp, 1);
|
||||
}
|
||||
if (SL2IFP(sc)->if_bpf && sc->bpfbuf) {
|
||||
if (bpf_peers_present(SL2IFP(sc)->if_bpf) && sc->bpfbuf) {
|
||||
/*
|
||||
* Put the SLIP pseudo-"link header" in place. The
|
||||
* compressed header is now at the beginning of the
|
||||
@ -922,7 +922,7 @@ slinput(int c, struct tty *tp)
|
||||
/* less than min length packet - ignore */
|
||||
goto newpack;
|
||||
|
||||
if (SL2IFP(sc)->if_bpf) {
|
||||
if (bpf_peers_present(SL2IFP(sc)->if_bpf)) {
|
||||
/*
|
||||
* Save the compressed header, so we
|
||||
* can tack it on later. Note that we
|
||||
@ -961,7 +961,7 @@ slinput(int c, struct tty *tp)
|
||||
} else
|
||||
goto error;
|
||||
}
|
||||
if (SL2IFP(sc)->if_bpf) {
|
||||
if (bpf_peers_present(SL2IFP(sc)->if_bpf)) {
|
||||
/*
|
||||
* Put the SLIP pseudo-"link header" in place.
|
||||
* We couldn't do this any earlier since
|
||||
|
@ -476,7 +476,7 @@ stf_output(ifp, m, dst, rt)
|
||||
}
|
||||
bcopy(ptr, &in4, sizeof(in4));
|
||||
|
||||
if (ifp->if_bpf) {
|
||||
if (bpf_peers_present(ifp->if_bpf)) {
|
||||
/*
|
||||
* We need to prepend the address family as
|
||||
* a four byte field. Cons up a dummy header
|
||||
@ -723,7 +723,7 @@ in_stf_input(m, off)
|
||||
|
||||
m->m_pkthdr.rcvif = ifp;
|
||||
|
||||
if (ifp->if_bpf) {
|
||||
if (bpf_peers_present(ifp->if_bpf)) {
|
||||
/*
|
||||
* We need to prepend the address family as
|
||||
* a four byte field. Cons up a dummy header
|
||||
|
@ -509,7 +509,7 @@ tunoutput(
|
||||
dst->sa_family = af;
|
||||
}
|
||||
|
||||
if (ifp->if_bpf) {
|
||||
if (bpf_peers_present(ifp->if_bpf)) {
|
||||
af = dst->sa_family;
|
||||
bpf_mtap2(ifp->if_bpf, &af, sizeof(af), m0);
|
||||
}
|
||||
|
@ -422,7 +422,7 @@ ieee80211_input(struct ieee80211com *ic, struct mbuf *m,
|
||||
}
|
||||
|
||||
/* copy to listener after decrypt */
|
||||
if (ic->ic_rawbpf)
|
||||
if (bpf_peers_present(ic->ic_rawbpf))
|
||||
bpf_mtap(ic->ic_rawbpf, m);
|
||||
|
||||
/*
|
||||
@ -536,7 +536,7 @@ ieee80211_input(struct ieee80211com *ic, struct mbuf *m,
|
||||
wh = mtod(m, struct ieee80211_frame *);
|
||||
wh->i_fc[1] &= ~IEEE80211_FC1_WEP;
|
||||
}
|
||||
if (ic->ic_rawbpf)
|
||||
if (bpf_peers_present(ic->ic_rawbpf))
|
||||
bpf_mtap(ic->ic_rawbpf, m);
|
||||
(*ic->ic_recv_mgmt)(ic, m, ni, subtype, rssi, rstamp);
|
||||
m_freem(m);
|
||||
@ -563,7 +563,7 @@ ieee80211_input(struct ieee80211com *ic, struct mbuf *m,
|
||||
ifp->if_ierrors++;
|
||||
out:
|
||||
if (m != NULL) {
|
||||
if (ic->ic_rawbpf)
|
||||
if (bpf_peers_present(ic->ic_rawbpf))
|
||||
bpf_mtap(ic->ic_rawbpf, m);
|
||||
m_freem(m);
|
||||
}
|
||||
@ -719,7 +719,7 @@ ieee80211_deliver_data(struct ieee80211com *ic,
|
||||
return;
|
||||
out:
|
||||
if (m != NULL) {
|
||||
if (ic->ic_rawbpf)
|
||||
if (bpf_peers_present(ic->ic_rawbpf))
|
||||
bpf_mtap(ic->ic_rawbpf, m);
|
||||
m_freem(m);
|
||||
}
|
||||
|
@ -657,7 +657,7 @@ carp_input_c(struct mbuf *m, struct carp_header *ch, sa_family_t af)
|
||||
SC2IFP(sc)->if_ipackets++;
|
||||
SC2IFP(sc)->if_ibytes += m->m_pkthdr.len;
|
||||
|
||||
if (SC2IFP(sc)->if_bpf) {
|
||||
if (bpf_peers_present(SC2IFP(sc)->if_bpf)) {
|
||||
struct ip *ip = mtod(m, struct ip *);
|
||||
uint32_t af1 = af;
|
||||
|
||||
|
@ -206,7 +206,7 @@ gre_input2(struct mbuf *m ,int hlen, u_char proto)
|
||||
/* Unlike NetBSD, in FreeBSD m_adj() adjusts m->m_pkthdr.len as well */
|
||||
m_adj(m, hlen);
|
||||
|
||||
if (GRE2IFP(sc)->if_bpf) {
|
||||
if (bpf_peers_present(GRE2IFP(sc)->if_bpf)) {
|
||||
bpf_mtap2(GRE2IFP(sc)->if_bpf, &af, sizeof(af), m);
|
||||
}
|
||||
|
||||
@ -289,7 +289,7 @@ gre_mobile_input(struct mbuf *m, int hlen)
|
||||
ip->ip_sum = 0;
|
||||
ip->ip_sum = in_cksum(m, (ip->ip_hl << 2));
|
||||
|
||||
if (GRE2IFP(sc)->if_bpf) {
|
||||
if (bpf_peers_present(GRE2IFP(sc)->if_bpf)) {
|
||||
u_int32_t af = AF_INET;
|
||||
bpf_mtap2(GRE2IFP(sc)->if_bpf, &af, sizeof(af), m);
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user