diff options
| author | Damien Bergamini <damien@cvs.openbsd.org> | 2010-02-17 18:34:38 +0000 |
|---|---|---|
| committer | Damien Bergamini <damien@cvs.openbsd.org> | 2010-02-17 18:34:38 +0000 |
| commit | 9f2fd3e4ed1e1dc880b011635e0eed0772c0258f (patch) | |
| tree | 6c97d0ba4b450f76389470228be8598143ebd6c8 | |
| parent | eb2fee5a584dd4274f28cf1b12f5789a7565189b (diff) | |
Do not always create a new node when a beacon or a probe response
is received. The creation of a new node is already handled in
ieee80211_recv_probe_resp() when necessary.
This avoids creating empty nodes with wrong channels when beacons
are received on the wrong channel (overlapping channels).
Those empty nodes may also prevent the real node from being
discovered because of ieee80211_find_node_for_beacon() filtering.
This should prevent entries of the form:
nwid "" chan 3 bssid 00:01:02:03:04:05 0dB 54M
in "ifconfig if0 scan" output, like reported by Rivo Nurges.
| -rw-r--r-- | sys/net80211/ieee80211_node.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sys/net80211/ieee80211_node.c b/sys/net80211/ieee80211_node.c index 3c7c5ed8d20..b2e4d827b83 100644 --- a/sys/net80211/ieee80211_node.c +++ b/sys/net80211/ieee80211_node.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ieee80211_node.c,v 1.55 2009/01/29 17:19:10 damien Exp $ */ +/* $OpenBSD: ieee80211_node.c,v 1.56 2010/02/17 18:34:37 damien Exp $ */ /* $NetBSD: ieee80211_node.c,v 1.14 2004/05/09 09:18:47 dyoung Exp $ */ /*- @@ -930,7 +930,6 @@ ieee80211_needs_rxnode(struct ieee80211com *ic, switch (wh->i_fc[0] & IEEE80211_FC0_SUBTYPE_MASK) { case IEEE80211_FC0_SUBTYPE_BEACON: case IEEE80211_FC0_SUBTYPE_PROBE_RESP: - rc = 1; break; default: #ifndef IEEE80211_STA_ONLY |