diff options
author | Arend van Spriel <arend@broadcom.com> | 2012-08-24 08:43:31 -0400 |
---|---|---|
committer | Johannes Berg <johannes.berg@intel.com> | 2012-09-05 10:12:44 -0400 |
commit | e5f5b2fb07353de00ffde49221cffad71e2fecfe (patch) | |
tree | 4bafa25188e94f20b24638c919009e99d1990171 | |
parent | 00ea6deb0cb1af8df4ea899330bfd238c93ef164 (diff) |
wext: include wireless event id when it has a size problem
The wext code checks is the event data is within size limits.
When this check fails a message is logged with violating size.
This patch adds the event id to put us on the right track for
resolving that violation.
Reviewed-by: Hante Meuleman <meuleman@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
-rw-r--r-- | net/wireless/wext-core.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/net/wireless/wext-core.c b/net/wireless/wext-core.c index b0eb7aa49b60..c8717c1d082e 100644 --- a/net/wireless/wext-core.c +++ b/net/wireless/wext-core.c | |||
@@ -478,13 +478,13 @@ void wireless_send_event(struct net_device * dev, | |||
478 | if (descr->header_type == IW_HEADER_TYPE_POINT) { | 478 | if (descr->header_type == IW_HEADER_TYPE_POINT) { |
479 | /* Check if number of token fits within bounds */ | 479 | /* Check if number of token fits within bounds */ |
480 | if (wrqu->data.length > descr->max_tokens) { | 480 | if (wrqu->data.length > descr->max_tokens) { |
481 | netdev_err(dev, "(WE) : Wireless Event too big (%d)\n", | 481 | netdev_err(dev, "(WE) : Wireless Event (cmd=0x%04X) too big (%d)\n", |
482 | wrqu->data.length); | 482 | cmd, wrqu->data.length); |
483 | return; | 483 | return; |
484 | } | 484 | } |
485 | if (wrqu->data.length < descr->min_tokens) { | 485 | if (wrqu->data.length < descr->min_tokens) { |
486 | netdev_err(dev, "(WE) : Wireless Event too small (%d)\n", | 486 | netdev_err(dev, "(WE) : Wireless Event (cmd=0x%04X) too small (%d)\n", |
487 | wrqu->data.length); | 487 | cmd, wrqu->data.length); |
488 | return; | 488 | return; |
489 | } | 489 | } |
490 | /* Calculate extra_len - extra is NULL for restricted events */ | 490 | /* Calculate extra_len - extra is NULL for restricted events */ |