[devel] wpa_supplicant is broken

[devel] wpa_supplicant is broken

[Alexander Bokovoy]

[-- Attachment #1.1: Type: text/plain, Size: 915 bytes --]

Greetings!

Обновился до вчерашнего Сизифа после некоторого "застоя". Вообщем, если с
etcnet я более-менее справился (как и с беспорядочным переименованием
интерфейсов -- посредством /etc/net/iftab, формат которого описан только в
примерах), то за wpa_supplicant, который *НЕ РАБОТАЕТ* с ipw2100 и
ipw2200, я готов кидаться своим новым хрустальным глобусом.

Алексей (Такасеев), оно вообще проверялось на функционирование?

driver_ipw.c содержит код из ipw2100.c/ipw2200.c из соответствующих
драйверов, который должен синхронизироваться с кодом драйверов. Я думаю,
что несмотря на то, что в апстрим такую синхронизацию не проводят, это не
повод не проверять работоспособность.

Патч приложен.

-- 
/ Alexander Bokovoy
Samba Team                      http://www.samba.org/
ALT Linux Team                  http://www.altlinux.org/
Midgard Project Ry              http://www.midgard-project.org/

[-- Attachment #1.2: driver_ipw.c.patch --]
[-- Type: text/plain, Size: 831 bytes --]

--- driver_ipw.c.orig	2005-05-24 09:44:25 +0400
+++ driver_ipw.c	2005-07-02 11:31:40 +0400
@@ -73,19 +73,18 @@
 		} wpa_param;
 		struct {
 			u32 len;
-			u8 reserved[32];
-			u8 data[0];
+			u8 *data;
 		} wpa_ie;
 	        struct{
-			u32 command;
-    			u32 reason_code;
+			int command;
+    			int reason_code;
 		} mlme;
 		struct {
 			u8 alg[IPW_CRYPT_ALG_NAME_LEN];
 			u8 set_tx;
 			u32 err;
 			u8 idx;
-			u8 seq[8];
+			u8 seq[8]; /* sequence counter (set: RX, get: TX) */
 			u16 key_len;
 			u8 key[0];
 		} crypt;
@@ -161,6 +160,7 @@
 	memset(param, 0, blen);
 	param->cmd = IPW_CMD_SET_WPA_IE;
 	param->u.wpa_ie.len = wpa_ie_len;
+	param->u.wpa_ie.data = (u8 *) malloc(wpa_ie_len);
 	memcpy(param->u.wpa_ie.data, wpa_ie, wpa_ie_len);
 	
 	ret = ipw_ioctl(drv, param, blen, 1);

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] wpa_supplicant is broken

[Alexei Takaseev]

On Sat, 2 Jul 2005 11:59:43 +0400
Alexander Bokovoy wrote:

> Greetings!
> 
> Обновился до вчерашнего Сизифа после некоторого "застоя". Вообщем,
> если с etcnet я более-менее справился (как и с беспорядочным
> переименованием интерфейсов -- посредством /etc/net/iftab, формат
> которого описан только в примерах), то за wpa_supplicant, который *НЕ
> РАБОТАЕТ* с ipw2100 и ipw2200, я готов кидаться своим новым
> хрустальным глобусом.
> 
> Алексей (Такасеев), оно вообще проверялось на функционирование?
> 
> driver_ipw.c содержит код из ipw2100.c/ipw2200.c из соответствующих
> драйверов, который должен синхронизироваться с кодом драйверов. Я
> думаю, что несмотря на то, что в апстрим такую синхронизацию не
> проводят, это не повод не проверять работоспособность.
> 
> Патч приложен.

У меня нету железа ipw2100 и ipw2200. С тем, что есть в наличии (карты
на PRISM2.5 и Atheros) оно работает.

Сегодня пересоберу с патчем.

Re: [devel] wpa_supplicant is broken

[Sergey Vlasov]

[-- Attachment #1: Type: text/plain, Size: 997 bytes --]

On Sat, Jul 02, 2005 at 11:59:43AM +0400, Alexander Bokovoy wrote:
> --- driver_ipw.c.orig	2005-05-24 09:44:25 +0400
> +++ driver_ipw.c	2005-07-02 11:31:40 +0400
> @@ -73,19 +73,18 @@
>  		} wpa_param;
>  		struct {
>  			u32 len;
> -			u8 reserved[32];
> -			u8 data[0];
> +			u8 *data;
>  		} wpa_ie;
>  	        struct{
> -			u32 command;
> -    			u32 reason_code;
> +			int command;
> +    			int reason_code;
>  		} mlme;
>  		struct {
>  			u8 alg[IPW_CRYPT_ALG_NAME_LEN];
>  			u8 set_tx;
>  			u32 err;
>  			u8 idx;
> -			u8 seq[8];
> +			u8 seq[8]; /* sequence counter (set: RX, get: TX) */
>  			u16 key_len;
>  			u8 key[0];
>  		} crypt;
> @@ -161,6 +160,7 @@
>  	memset(param, 0, blen);
>  	param->cmd = IPW_CMD_SET_WPA_IE;
>  	param->u.wpa_ie.len = wpa_ie_len;
> +	param->u.wpa_ie.data = (u8 *) malloc(wpa_ie_len);

А free кто будет делать?

>  	memcpy(param->u.wpa_ie.data, wpa_ie, wpa_ie_len);
>  	
>  	ret = ipw_ioctl(drv, param, blen, 1);

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] wpa_supplicant is broken

[Alexander Bokovoy]

[-- Attachment #1.1: Type: text/plain, Size: 519 bytes --]

On Sat, Jul 02, 2005 at 03:29:02PM +0400, Sergey Vlasov wrote:
> >  		} crypt;
> > @@ -161,6 +160,7 @@
> >  	memset(param, 0, blen);
> >  	param->cmd = IPW_CMD_SET_WPA_IE;
> >  	param->u.wpa_ie.len = wpa_ie_len;
> > +	param->u.wpa_ie.data = (u8 *) malloc(wpa_ie_len);
> 
> А free кто будет делать?
Ага.

-- 
/ Alexander Bokovoy
Samba Team                      http://www.samba.org/
ALT Linux Team                  http://www.altlinux.org/
Midgard Project Ry              http://www.midgard-project.org/

[-- Attachment #1.2: driver_ipw.c.patch --]
[-- Type: text/plain, Size: 902 bytes --]

--- driver_ipw.c.orig	2005-05-24 09:44:25 +0400
+++ driver_ipw.c	2005-07-02 15:55:12 +0400
@@ -73,19 +73,18 @@
 		} wpa_param;
 		struct {
 			u32 len;
-			u8 reserved[32];
-			u8 data[0];
+			u8 *data;
 		} wpa_ie;
 	        struct{
-			u32 command;
-    			u32 reason_code;
+			int command;
+    			int reason_code;
 		} mlme;
 		struct {
 			u8 alg[IPW_CRYPT_ALG_NAME_LEN];
 			u8 set_tx;
 			u32 err;
 			u8 idx;
-			u8 seq[8];
+			u8 seq[8]; /* sequence counter (set: RX, get: TX) */
 			u16 key_len;
 			u8 key[0];
 		} crypt;
@@ -161,10 +160,11 @@
 	memset(param, 0, blen);
 	param->cmd = IPW_CMD_SET_WPA_IE;
 	param->u.wpa_ie.len = wpa_ie_len;
+	param->u.wpa_ie.data = (u8 *) malloc(wpa_ie_len);
 	memcpy(param->u.wpa_ie.data, wpa_ie, wpa_ie_len);
 	
 	ret = ipw_ioctl(drv, param, blen, 1);
-
+	free(param->u.wpa_ie.data);
 	free(param);
 	return ret;
 }

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] wpa_supplicant is broken

[Denis Ovsienko]

[...]
> интерфейсов -- посредством /etc/net/iftab, формат которого описан только в
> примерах), то за wpa_supplicant, который *НЕ РАБОТАЕТ* с ipw2100 и
[...]
Существует неплохая man-страница для iftab.

-- 
    DO4-UANIC

Re: [devel] wpa_supplicant is broken

[Alexander Bokovoy]

[-- Attachment #1: Type: text/plain, Size: 609 bytes --]

On Mon, Jul 04, 2005 at 11:03:30AM +0400, Denis Ovsienko wrote:
> [...]
> > интерфейсов -- посредством /etc/net/iftab, формат которого описан только в
> > примерах), то за wpa_supplicant, который *НЕ РАБОТАЕТ* с ipw2100 и
> [...]
> Существует неплохая man-страница для iftab.
Денис, а можно ссылку на нее добавить в README? Прямо указав, что
/etc/net/iftab используется как /etc/iftab в ifrename.
-- 
/ Alexander Bokovoy
Samba Team                      http://www.samba.org/
ALT Linux Team                  http://www.altlinux.org/
Midgard Project Ry              http://www.midgard-project.org/

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] wpa_supplicant is broken

[Denis Ovsienko]

> Денис, а можно ссылку на нее добавить в README? Прямо указав, что
> /etc/net/iftab используется как /etc/iftab в ifrename.
Пожалуйста.

-- 
    DO4-UANIC

[devel] [JT] глобусоносцу :)

[Michael Shigorin]

On Sat, Jul 02, 2005 at 11:59:43AM +0400, Alexander Bokovoy wrote:
> я готов кидаться своим новым хрустальным глобусом.

Вот такие они -- сперва глобусом стукнут, потом free() зажмут...
А вообще -- поздравляю :-)

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/