Re: [PATCH] Re: Problem with Kernel Oops in ipw2200

"John W. Linville" <[email protected]> Tue, 18 Nov 2008 14:23:21 -0500
Newsgroups gmane.linux.kernel.wireless.general,gmane.linux.drivers.ipw2100.devel
Message-ID <[email protected]>
On Mon, Nov 17, 2008 at 11:48:19AM +0100, Frank Seidel wrote:
> From: Zhu Yi <[email protected]>
> 
> Fixes Oops in ipw2200:ipw_tx_skb when pinging through
> a WPA enterprise connection.
> 
> Signed-off-by: Zhu Yi <[email protected]>
> Tested-by: Frank Seidel <[email protected]>
> Signed-off-by: Ffrank Seidel <[email protected]>
> 
> ---
>  drivers/net/wireless/ipw2200.c |    5 +++++
>  1 file changed, 5 insertions(+)
> 
> --- a/drivers/net/wireless/ipw2200.c
> +++ b/drivers/net/wireless/ipw2200.c
> @@ -10190,6 +10190,11 @@ static int ipw_tx_skb(struct ipw_priv *p
>  	u16 remaining_bytes;
>  	int fc;
>  
> +	if (!(priv->status & STATUS_ASSOCIATED)) {
> +		IPW_DEBUG_TX("Tx attempt while not associated.\n");
> +		goto drop;
> +	}
> +
>  	hdr_len = ieee80211_get_hdrlen(le16_to_cpu(hdr->frame_ctl));
>  	switch (priv->ieee->iw_mode) {
>  	case IW_MODE_ADHOC:
> 

Well, I'm sorry to be a PITA...but the changelog doesn't really
explain how the patch works or what it is doing.  Also, this still
seems to me more like a band-aid than a real fix...?

John
-- 
John W. Linville		Linux should be at the core
[email protected]			of your literate lifestyle.
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html