Search Linux Wireless

Re: [PATCH] cfg80211: don't report SME connection errs at SIOCSIW{FREQ/AP/ESSID}

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2009-09-14 at 13:23 +0200, Holger Schurig wrote:
> Signed-off-by: Holger Schurig <hs4233@xxxxxxxxxxxxxxxxxxxx>
> 
> ---
> 
> This fixes
> 
> 1252921276.182218: Trying to associate with 00:1b:53:11:dc:40 (SSID='TEST' freq=2412 MHz)
> ioctl[SIOCSIWFREQ]: No such file or directory
> ioctl[SIOCSIWESSID]: No such file or directory
> 1252921276.268451: Association request to the driver failed
> 1252921276.355482: Associated with 00:1b:53:11:dc:40
> 
> and makes it
> 
> 1252921276.182218: Trying to associate with 00:1b:53:11:dc:40 (SSID='TEST' freq=2412 MHz)
> 1252921276.355482: Associated with 00:1b:53:11:dc:40
> 
> However, otherwise this hasn't been greatly tested and I
> don't have a great understanding of cfg80211, SME, etc ...

And doesn't look correct to me at all. probably something missing wrt.
disconnect() while connecting.

johannes

> Index: linux-wl/net/wireless/wext-sme.c
> ===================================================================
> --- linux-wl.orig/net/wireless/wext-sme.c	2009-09-14 12:05:54.000000000 +0200
> +++ linux-wl/net/wireless/wext-sme.c	2009-09-14 12:06:09.000000000 +0200
> @@ -110,7 +110,7 @@ int cfg80211_mgd_wext_siwfreq(struct net
>  		goto out;
>  	}
>  
> -	err = cfg80211_mgd_wext_connect(rdev, wdev);
> +	cfg80211_mgd_wext_connect(rdev, wdev);
>   out:
>  	wdev_unlock(wdev);
>  	mutex_unlock(&rdev->devlist_mtx);
> @@ -196,7 +196,7 @@ int cfg80211_mgd_wext_siwessid(struct ne
>  
>  	wdev->wext.connect.crypto.control_port = false;
>  
> -	err = cfg80211_mgd_wext_connect(rdev, wdev);
> +	cfg80211_mgd_wext_connect(rdev, wdev);
>   out:
>  	wdev_unlock(wdev);
>  	mutex_unlock(&rdev->devlist_mtx);
> @@ -283,7 +283,7 @@ int cfg80211_mgd_wext_siwap(struct net_d
>  	} else
>  		wdev->wext.connect.bssid = NULL;
>  
> -	err = cfg80211_mgd_wext_connect(rdev, wdev);
> +	cfg80211_mgd_wext_connect(rdev, wdev);
>   out:
>  	wdev_unlock(wdev);
>  	mutex_unlock(&rdev->devlist_mtx);
> 

Attachment: signature.asc
Description: This is a digitally signed message part


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux