diff options
| author | Jiri Benc <jbenc@suse.cz> | 2006-10-18 13:34:40 -0400 |
|---|---|---|
| committer | John W. Linville <linville@laptop.(none)> | 2006-10-31 22:15:38 -0500 |
| commit | aec41a0d02342fc9e3b6bb278eae50fa29f04d1f (patch) | |
| tree | 93449a57c4e1d92f85dd3b79da79f00715798499 | |
| parent | 16b7b2ac0148e839da86af8747b6fa4aad43a9b7 (diff) | |
[PATCH] ieee80211: don't flood log with errors
The "ieee80211: Workaround malformed 802.11 frames from AP" patch (see
http://kernel.org/git/?p=linux/kernel/git/linville/wireless-2.6.git;a=commit;h=f09fc44d8c25f22c4d985bb93857338ed02feac6 )
fixes the problem with some buggy APs but also converts debug message into
an error one. This floods the log with errors when you are near such AP (you
get a message for every beacon). This patch reverts the error message back
to the debug one.
Signed-off-by: Jiri Benc <jbenc@suse.cz>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
| -rw-r--r-- | net/ieee80211/ieee80211_rx.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/net/ieee80211/ieee80211_rx.c b/net/ieee80211/ieee80211_rx.c index 770704183a1b..2759312a4204 100644 --- a/net/ieee80211/ieee80211_rx.c +++ b/net/ieee80211/ieee80211_rx.c | |||
| @@ -1078,12 +1078,12 @@ static int ieee80211_parse_info_param(struct ieee80211_info_element | |||
| 1078 | 1078 | ||
| 1079 | while (length >= sizeof(*info_element)) { | 1079 | while (length >= sizeof(*info_element)) { |
| 1080 | if (sizeof(*info_element) + info_element->len > length) { | 1080 | if (sizeof(*info_element) + info_element->len > length) { |
| 1081 | IEEE80211_ERROR("Info elem: parse failed: " | 1081 | IEEE80211_DEBUG_MGMT("Info elem: parse failed: " |
| 1082 | "info_element->len + 2 > left : " | 1082 | "info_element->len + 2 > left : " |
| 1083 | "info_element->len+2=%zd left=%d, id=%d.\n", | 1083 | "info_element->len+2=%zd left=%d, id=%d.\n", |
| 1084 | info_element->len + | 1084 | info_element->len + |
| 1085 | sizeof(*info_element), | 1085 | sizeof(*info_element), |
| 1086 | length, info_element->id); | 1086 | length, info_element->id); |
| 1087 | /* We stop processing but don't return an error here | 1087 | /* We stop processing but don't return an error here |
| 1088 | * because some misbehaviour APs break this rule. ie. | 1088 | * because some misbehaviour APs break this rule. ie. |
| 1089 | * Orinoco AP1000. */ | 1089 | * Orinoco AP1000. */ |
