diff mbox

libertas wext.c: IW_ENCODE_NOKEY for WEP keys [resend as plain text]

Message ID 477F20668A386D41ADCC57781B1F704306D2369E90@SC-VEXCH1.marvell.com (mailing list archive)
State Not Applicable, archived
Headers show

Commit Message

Bing Zhao Aug. 26, 2009, 7:11 p.m. UTC 
Hi all,
 
There was a commit to unset the IW_ENCODE_NOKEY flag for WEP keys.
 
"libertas: Don't set IW_ENCODE_NOKEY when returning WEP keys."
  
 
Without this change, the IW_ENCODE_NOKEY flag is set for WEP keys and then iwconfig command would display "****-****-**" as "Encryption key".
  
After this change, the IW_ENCODE_NOKEY flag is NOT set for WEP keys and then iwconfig command will display plain text of the WEP key ("1234-5678-90" in my case, below).
 
eth1      IEEE 802.11b/g  ESSID:"Cisco1-G"
          Mode:Managed  Frequency:2.462 GHz  Access Point: 00:1D:45:CE:20:D0
          Bit Rate:54 Mb/s   Tx-Power=15 dBm
          Retry short limit:8   RTS thr=2347 B   Fragment thr=2346 B
 
          Encryption key:1234-5678-90   Security mode:open
 
          Power Management:off
          Link Quality=92/100  Signal level=-66 dBm  Noise level=-94 dBm
          Rx invalid nwid:0  Rx invalid crypt:0  Rx invalid frag:0
          Tx excessive retries:0  Invalid misc:0   Missed beacon:0
 
 
 
Was there any reason to not set IW_ENCODE_NOKEY for WEP keys?
Is it feasible to set IW_ENCODE_NOKEY for WEP switch case?
 
 

Thanks for your help,
 
Bing

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Dan Williams Aug. 27, 2009, 2:06 p.m. UTC | #1 
On Wed, 2009-08-26 at 12:11 -0700, Bing Zhao wrote:
> Hi all,
>  
> There was a commit to unset the IW_ENCODE_NOKEY flag for WEP keys.
>  
> "libertas: Don't set IW_ENCODE_NOKEY when returning WEP keys."
>   
>  
> Without this change, the IW_ENCODE_NOKEY flag is set for WEP keys and then iwconfig command would display "****-****-**" as "Encryption key".
>   
> After this change, the IW_ENCODE_NOKEY flag is NOT set for WEP keys and then iwconfig command will display plain text of the WEP key ("1234-5678-90" in my case, below).

mac82011 doesn't set NOKEY in cfg80211_wext_giwencode().  I guess we
should ask if *all* drivers should set NOKEY and then be consistent.

Dan

> eth1      IEEE 802.11b/g  ESSID:"Cisco1-G"
>           Mode:Managed  Frequency:2.462 GHz  Access Point: 00:1D:45:CE:20:D0
>           Bit Rate:54 Mb/s   Tx-Power=15 dBm
>           Retry short limit:8   RTS thr=2347 B   Fragment thr=2346 B
>  
>           Encryption key:1234-5678-90   Security mode:open
>  
>           Power Management:off
>           Link Quality=92/100  Signal level=-66 dBm  Noise level=-94 dBm
>           Rx invalid nwid:0  Rx invalid crypt:0  Rx invalid frag:0
>           Tx excessive retries:0  Invalid misc:0   Missed beacon:0
>  
>  
>  
> Was there any reason to not set IW_ENCODE_NOKEY for WEP keys?
> Is it feasible to set IW_ENCODE_NOKEY for WEP switch case?
>  
> --- a/drivers/net/wireless/libertas/wext.c
> +++ b/drivers/net/wireless/libertas/wext.c
> @@ -1165,6 +1165,7 @@ static int lbs_get_encode(struct net_device *dev,
>                 dwrq->flags |= (index + 1);
>                 /* Return WEP enabled */
>                 dwrq->flags &= ~IW_ENCODE_DISABLED;
> +               dwrq->flags |= IW_ENCODE_NOKEY;
>         } else if ((priv->secinfo.WPAenabled)
>                    || (priv->secinfo.WPA2enabled)) {
>                 /* return WPA enabled */
>  
> 
> Thanks for your help,
>  
> Bing
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

--- a/drivers/net/wireless/libertas/wext.c
+++ b/drivers/net/wireless/libertas/wext.c
@@ -1165,6 +1165,7 @@  static int lbs_get_encode(struct net_device *dev,
                dwrq->flags |= (index + 1);
                /* Return WEP enabled */
                dwrq->flags &= ~IW_ENCODE_DISABLED;
+               dwrq->flags |= IW_ENCODE_NOKEY;
        } else if ((priv->secinfo.WPAenabled)
                   || (priv->secinfo.WPA2enabled)) {
                /* return WPA enabled */