aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/net
diff options
context:
space:
mode:
authorAmitkumar Karwar <akarwar@marvell.com>2012-04-26 16:02:57 -0400
committerJohn W. Linville <linville@tuxdriver.com>2012-05-08 21:53:41 -0400
commit2e4c14a5582addd655e53277861b7ad853bfd6c3 (patch)
treee7d8448413d42cfadb95a90edfbf04b50eb15d1b /drivers/net
parent91a0099c190107821292d3114c396ac858ad723e (diff)
mwifiex: fix static checker warnings
"oui_type" in structure "ieee_types_vendor_header" is not used separately, so include it in "oui" array. Now complete oui will be compared fixing following warnings. drivers/net/wireless/mwifiex/sta_ioctl.c:1410 mwifiex_set_gen_ie_helper() error: memcmp() 'pvendor_ie->oui' too small (3 vs 4) drivers/net/wireless/mwifiex/sta_ioctl.c:1435 mwifiex_set_gen_ie_helper() error: memcmp() 'pvendor_ie->oui' too small (3 vs 4) drivers/net/wireless/mwifiex/scan.c:1177 mwifiex_update_bss_desc_with_ie() error: memcmp() 'vendor_ie->vend_hdr.oui' too small (3 vs 4) drivers/net/wireless/mwifiex/scan.c:1185 mwifiex_update_bss_desc_with_ie() error: memcmp() 'vendor_ie->vend_hdr.oui' too small (3 vs 4) Reported-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Amitkumar Karwar <akarwar@marvell.com> Signed-off-by: Avinash Patil <patila@marvell.com> Signed-off-by: Kiran Divekar <dkiran@marvell.com> Signed-off-by: Bing Zhao <bzhao@marvell.com> Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net')
-rw-r--r--drivers/net/wireless/mwifiex/fw.h3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/wireless/mwifiex/fw.h b/drivers/net/wireless/mwifiex/fw.h
index 2a2cabadb9b5..5f6adeb9b950 100644
--- a/drivers/net/wireless/mwifiex/fw.h
+++ b/drivers/net/wireless/mwifiex/fw.h
@@ -993,8 +993,7 @@ struct mwifiex_ie_types_wmm_queue_status {
993struct ieee_types_vendor_header { 993struct ieee_types_vendor_header {
994 u8 element_id; 994 u8 element_id;
995 u8 len; 995 u8 len;
996 u8 oui[3]; 996 u8 oui[4]; /* 0~2: oui, 3: oui_type */
997 u8 oui_type;
998 u8 oui_subtype; 997 u8 oui_subtype;
999 u8 version; 998 u8 version;
1000} __packed; 999} __packed;