On Mon, 9 Oct 2006 13:11:02 -0700, David Kimdon wrote:
> --- wireless-dev.orig/net/d80211/ieee80211_sta.c
> +++ wireless-dev/net/d80211/ieee80211_sta.c
> @@ -930,8 +930,8 @@ static void ieee80211_rx_mgmt_auth(struc
>               printk(KERN_DEBUG "%s: AP denied authentication (auth_alg=%d "
>                      "code=%d)\n", dev->name, ifsta->auth_alg, status_code);
>               if (status_code == WLAN_STATUS_NOT_SUPPORTED_AUTH_ALG) {
> -                     const int num_algs = 3;
> -                     u8 algs[num_algs];
> +                     u8 algs[3];
> +                     const int num_algs = ARRAY_SIZE(algs);

Wouldn't it be better just to use ARRAY_SIZE(algs) and get rid of
num_algs completely?

Thanks,

 Jiri

-- 
Jiri Benc
SUSE Labs
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to