diff options
author | Jouni Malinen <jkmaline@cc.hut.fi> | 2005-08-28 13:51:33 -0400 |
---|---|---|
committer | Jeff Garzik <jgarzik@pobox.com> | 2005-08-28 19:23:07 -0400 |
commit | 51e828b6a123912f27afc512377ad3e2ab40d2eb (patch) | |
tree | 878cb9f93b541af1e6ed4653a58640bb5acf7713 /net/ieee80211 | |
parent | 5f55d0850e4ae0b4bfabc9a372af5b9f52be02da (diff) |
[PATCH] ieee80211: Remove EAPOL debug
IEEE 802.11 code has no business touching payloads of EAPOL frames.
There are some EAPOL structures defined for debugging and these were
confusingly called EAP types which they are not. Let's just remove these
before someone else starts using them in the kernel.
Signed-off-by: Jouni Malinen <jkmaline@cc.hut.fi>
Signed-off-by: Jeff Garzik <jgarzik@pobox.com>
Diffstat (limited to 'net/ieee80211')
-rw-r--r-- | net/ieee80211/ieee80211_rx.c | 16 | ||||
-rw-r--r-- | net/ieee80211/ieee80211_tx.c | 9 |
2 files changed, 0 insertions, 25 deletions
diff --git a/net/ieee80211/ieee80211_rx.c b/net/ieee80211/ieee80211_rx.c index 9505f3167aeb..d582faa6447d 100644 --- a/net/ieee80211/ieee80211_rx.c +++ b/net/ieee80211/ieee80211_rx.c | |||
@@ -628,14 +628,8 @@ int ieee80211_rx(struct ieee80211_device *ieee, struct sk_buff *skb, | |||
628 | if (crypt && !(fc & IEEE80211_FCTL_PROTECTED) && !ieee->open_wep) { | 628 | if (crypt && !(fc & IEEE80211_FCTL_PROTECTED) && !ieee->open_wep) { |
629 | if (/*ieee->ieee802_1x &&*/ | 629 | if (/*ieee->ieee802_1x &&*/ |
630 | ieee80211_is_eapol_frame(ieee, skb)) { | 630 | ieee80211_is_eapol_frame(ieee, skb)) { |
631 | #ifdef CONFIG_IEEE80211_DEBUG | ||
632 | /* pass unencrypted EAPOL frames even if encryption is | 631 | /* pass unencrypted EAPOL frames even if encryption is |
633 | * configured */ | 632 | * configured */ |
634 | struct eapol *eap = (struct eapol *)(skb->data + | ||
635 | 24); | ||
636 | IEEE80211_DEBUG_EAP("RX: IEEE 802.1X EAPOL frame: %s\n", | ||
637 | eap_get_type(eap->type)); | ||
638 | #endif | ||
639 | } else { | 633 | } else { |
640 | IEEE80211_DEBUG_DROP( | 634 | IEEE80211_DEBUG_DROP( |
641 | "encryption configured, but RX " | 635 | "encryption configured, but RX " |
@@ -645,16 +639,6 @@ int ieee80211_rx(struct ieee80211_device *ieee, struct sk_buff *skb, | |||
645 | } | 639 | } |
646 | } | 640 | } |
647 | 641 | ||
648 | #ifdef CONFIG_IEEE80211_DEBUG | ||
649 | if (crypt && !(fc & IEEE80211_FCTL_PROTECTED) && | ||
650 | ieee80211_is_eapol_frame(ieee, skb)) { | ||
651 | struct eapol *eap = (struct eapol *)(skb->data + | ||
652 | 24); | ||
653 | IEEE80211_DEBUG_EAP("RX: IEEE 802.1X EAPOL frame: %s\n", | ||
654 | eap_get_type(eap->type)); | ||
655 | } | ||
656 | #endif | ||
657 | |||
658 | if (crypt && !(fc & IEEE80211_FCTL_PROTECTED) && !ieee->open_wep && | 642 | if (crypt && !(fc & IEEE80211_FCTL_PROTECTED) && !ieee->open_wep && |
659 | !ieee80211_is_eapol_frame(ieee, skb)) { | 643 | !ieee80211_is_eapol_frame(ieee, skb)) { |
660 | IEEE80211_DEBUG_DROP( | 644 | IEEE80211_DEBUG_DROP( |
diff --git a/net/ieee80211/ieee80211_tx.c b/net/ieee80211/ieee80211_tx.c index 071d093f2989..b7ea3e25e25d 100644 --- a/net/ieee80211/ieee80211_tx.c +++ b/net/ieee80211/ieee80211_tx.c | |||
@@ -292,15 +292,6 @@ int ieee80211_xmit(struct sk_buff *skb, | |||
292 | goto success; | 292 | goto success; |
293 | } | 293 | } |
294 | 294 | ||
295 | #ifdef CONFIG_IEEE80211_DEBUG | ||
296 | if (crypt && !encrypt && ether_type == ETH_P_PAE) { | ||
297 | struct eapol *eap = (struct eapol *)(skb->data + | ||
298 | sizeof(struct ethhdr) - SNAP_SIZE - sizeof(u16)); | ||
299 | IEEE80211_DEBUG_EAP("TX: IEEE 802.11 EAPOL frame: %s\n", | ||
300 | eap_get_type(eap->type)); | ||
301 | } | ||
302 | #endif | ||
303 | |||
304 | /* Save source and destination addresses */ | 295 | /* Save source and destination addresses */ |
305 | memcpy(&dest, skb->data, ETH_ALEN); | 296 | memcpy(&dest, skb->data, ETH_ALEN); |
306 | memcpy(&src, skb->data+ETH_ALEN, ETH_ALEN); | 297 | memcpy(&src, skb->data+ETH_ALEN, ETH_ALEN); |