>From 6f6a49200c74553acac806b6987d5c661346ca00 Mon Sep 17 00:00:00 2001
From: Tim Gardner <[email protected]>
Date: Tue, 14 Oct 2008 10:30:33 -0600
Subject: [PATCH v2] ipw2200: change default policy for auto-associate
Do not attempt association until directed to do so by a user space
application. In particular, this avoids race conditions with
NetworkManager association state.
Signed-off-by: Tim Gardner <[email protected]>
---
Documentation/networking/README.ipw2200 | 2 +-
drivers/net/wireless/ipw2200.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/Documentation/networking/README.ipw2200 b/Documentation/networking/README.ipw2200
index 4f2a40f..80c7285 100644
--- a/Documentation/networking/README.ipw2200
+++ b/Documentation/networking/README.ipw2200
@@ -147,7 +147,7 @@ Where the supported parameter are:
driver. If disabled, the driver will not attempt to scan
for and associate to a network until it has been configured with
one or more properties for the target network, for example configuring
- the network SSID. Default is 1 (auto-associate)
+ the network SSID. Default is 0 (do not auto-associate)
Example: % modprobe ipw2200 associate=0
diff --git a/drivers/net/wireless/ipw2200.c b/drivers/net/wireless/ipw2200.c
index dcce354..eb1230e 100644
--- a/drivers/net/wireless/ipw2200.c
+++ b/drivers/net/wireless/ipw2200.c
@@ -87,7 +87,7 @@ static int channel = 0;
static int mode = 0;
static u32 ipw_debug_level;
-static int associate = 1;
+static int associate;
static int auto_create = 1;
static int led = 0;
static int disable = 0;
--
1.5.6.3
On Tue, 2008-10-14 at 12:52 -0400, Dan Williams wrote:
> On Tue, 2008-10-14 at 10:38 -0600, Tim Gardner wrote:
> > From 6f6a49200c74553acac806b6987d5c661346ca00 Mon Sep 17 00:00:00 2001
> > From: Tim Gardner <[email protected]>
> > Date: Tue, 14 Oct 2008 10:30:33 -0600
> > Subject: [PATCH v2] ipw2200: change default policy for auto-associate
> >
> > Do not attempt association until directed to do so by a user space
> > application. In particular, this avoids race conditions with
> > NetworkManager association state.
> >
> > Signed-off-by: Tim Gardner <[email protected]>
>
> Though for completeness, maybe associate = 0; ?
Or not, just thought that since the rest two lines below do it, but
actually pointless.
Dan
>
> Acked-by: Dan Williams <[email protected]>
>
> Cards shouldn't be associating automatically with anything until told to
> do so to a specific SSID or specific BSSID/SSID combination, otherwise
> it's a security risk. The 3 people out in the middle of nowhere with
> one unencryped access point who may want this functionality should be
> turning it on manually because they are clearly more technically able
> than most.
>
> > ---
> > Documentation/networking/README.ipw2200 | 2 +-
> > drivers/net/wireless/ipw2200.c | 2 +-
> > 2 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/Documentation/networking/README.ipw2200 b/Documentation/networking/README.ipw2200
> > index 4f2a40f..80c7285 100644
> > --- a/Documentation/networking/README.ipw2200
> > +++ b/Documentation/networking/README.ipw2200
> > @@ -147,7 +147,7 @@ Where the supported parameter are:
> > driver. If disabled, the driver will not attempt to scan
> > for and associate to a network until it has been configured with
> > one or more properties for the target network, for example configuring
> > - the network SSID. Default is 1 (auto-associate)
> > + the network SSID. Default is 0 (do not auto-associate)
> >
> > Example: % modprobe ipw2200 associate=0
> >
> > diff --git a/drivers/net/wireless/ipw2200.c b/drivers/net/wireless/ipw2200.c
> > index dcce354..eb1230e 100644
> > --- a/drivers/net/wireless/ipw2200.c
> > +++ b/drivers/net/wireless/ipw2200.c
> > @@ -87,7 +87,7 @@ static int channel = 0;
> > static int mode = 0;
> >
> > static u32 ipw_debug_level;
> > -static int associate = 1;
> > +static int associate;
> > static int auto_create = 1;
> > static int led = 0;
> > static int disable = 0;
>
> --
> 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
On Tue, 2008-10-14 at 10:38 -0600, Tim Gardner wrote:
> From 6f6a49200c74553acac806b6987d5c661346ca00 Mon Sep 17 00:00:00 2001
> From: Tim Gardner <[email protected]>
> Date: Tue, 14 Oct 2008 10:30:33 -0600
> Subject: [PATCH v2] ipw2200: change default policy for auto-associate
>
> Do not attempt association until directed to do so by a user space
> application. In particular, this avoids race conditions with
> NetworkManager association state.
>
> Signed-off-by: Tim Gardner <[email protected]>
Though for completeness, maybe associate = 0; ?
Acked-by: Dan Williams <[email protected]>
Cards shouldn't be associating automatically with anything until told to
do so to a specific SSID or specific BSSID/SSID combination, otherwise
it's a security risk. The 3 people out in the middle of nowhere with
one unencryped access point who may want this functionality should be
turning it on manually because they are clearly more technically able
than most.
> ---
> Documentation/networking/README.ipw2200 | 2 +-
> drivers/net/wireless/ipw2200.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/networking/README.ipw2200 b/Documentation/networking/README.ipw2200
> index 4f2a40f..80c7285 100644
> --- a/Documentation/networking/README.ipw2200
> +++ b/Documentation/networking/README.ipw2200
> @@ -147,7 +147,7 @@ Where the supported parameter are:
> driver. If disabled, the driver will not attempt to scan
> for and associate to a network until it has been configured with
> one or more properties for the target network, for example configuring
> - the network SSID. Default is 1 (auto-associate)
> + the network SSID. Default is 0 (do not auto-associate)
>
> Example: % modprobe ipw2200 associate=0
>
> diff --git a/drivers/net/wireless/ipw2200.c b/drivers/net/wireless/ipw2200.c
> index dcce354..eb1230e 100644
> --- a/drivers/net/wireless/ipw2200.c
> +++ b/drivers/net/wireless/ipw2200.c
> @@ -87,7 +87,7 @@ static int channel = 0;
> static int mode = 0;
>
> static u32 ipw_debug_level;
> -static int associate = 1;
> +static int associate;
> static int auto_create = 1;
> static int led = 0;
> static int disable = 0;