If someone pulls the harware out while RX'ing a lot of traffic I would funky data may be passed, BUG_ON() seems pretty extreme so lets just drop the frame as we do when the length does not meet our criteria for processing. Cc: Christian Lamparter <chunkeey@xxxxxx> Signed-off-by: Luis R. Rodriguez <lrodriguez@xxxxxxxxxxx> --- This one depends on my previous patch. drivers/net/wireless/ath/ar9170/main.c | 7 +++++-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath/ar9170/main.c b/drivers/net/wireless/ath/ar9170/main.c index 75c317d..0bbbc36 100644 --- a/drivers/net/wireless/ath/ar9170/main.c +++ b/drivers/net/wireless/ath/ar9170/main.c @@ -1068,8 +1068,11 @@ static void ar9170_handle_mpdu(struct ar9170 *ar, u8 *buf, int len) break; default: - BUG_ON(1); - break; + if (ar9170_nag_limiter(ar)) + printk(KERN_ERR "%s: rx'd unexpected " + "type of MPDU.\n", + wiphy_name(ar->hw->wiphy)); + return; } if (unlikely(mpdu_len < FCS_LEN)) -- 1.6.3.3 -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
