kern/118369: [PATCH] ndis - a incorrect definition of struct ndis_wlan_bssid_ex

[Weongyo Jeong]

>Number:         118369
>Category:       kern
>Synopsis:       [PATCH] ndis - a incorrect definition of struct ndis_wlan_bssid_ex
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Sat Dec 01 08:10:05 UTC 2007
>Closed-Date:
>Last-Modified:
>Originator:     Weongyo Jeong
>Release:        FreeBSD-CURRENT
>Organization:
CDNetworks
>Environment:
>Description:
Following http://msdn2.microsoft.com/en-us/library/ms799391.aspx, the last element of `struct _NDIS_WLAN_BSSID_EX' should `UCHAR  IEs[1]' but in FreeBSD, it is defined as `uint32_t nwbx_ies[1]'.



>How-To-Repeat:

>Fix:
Index: ndis_var.h
===================================================================
RCS file: /data/ndis/ndis/ndis_var.h,v
retrieving revision 1.1.1.1
diff -u -r1.1.1.1 ndis_var.h
--- ndis_var.h  1 Dec 2007 06:23:11 -0000       1.1.1.1
+++ ndis_var.h  1 Dec 2007 06:45:05 -0000
@@ -466,7 +466,7 @@
        uint32_t                nwbx_netinfra;
        ndis_80211_rates_ex     nwbx_supportedrates;
        uint32_t                nwbx_ielen;
-       uint32_t                nwbx_ies[1];
+       uint8_t                 nwbx_ies[1];
 };
 
 typedef struct ndis_wlan_bssid_ex ndis_wlan_bssid_ex;


Patch attached with submission follows:

Index: ndis_var.h
===================================================================
RCS file: /data/ndis/ndis/ndis_var.h,v
retrieving revision 1.1.1.1
diff -u -r1.1.1.1 ndis_var.h
--- ndis_var.h	1 Dec 2007 06:23:11 -0000	1.1.1.1
+++ ndis_var.h	1 Dec 2007 06:45:05 -0000
@@ -466,7 +466,7 @@
 	uint32_t		nwbx_netinfra;
 	ndis_80211_rates_ex	nwbx_supportedrates;
 	uint32_t		nwbx_ielen;
-	uint32_t		nwbx_ies[1];
+	uint8_t			nwbx_ies[1];
 };
 
 typedef struct ndis_wlan_bssid_ex ndis_wlan_bssid_ex;


>Release-Note:
>Audit-Trail:
>Unformatted: