* [PATCH v2] ipw2200: change default policy for auto-associate
@ 2008-10-14 16:38 Tim Gardner
2008-10-14 16:52 ` Dan Williams
0 siblings, 1 reply; 3+ messages in thread
From: Tim Gardner @ 2008-10-14 16:38 UTC (permalink / raw)
To: linux-wireless
>From 6f6a49200c74553acac806b6987d5c661346ca00 Mon Sep 17 00:00:00 2001
From: Tim Gardner <tim.gardner@canonical.com>
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 <tim.gardner@canonical.com>
---
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
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2] ipw2200: change default policy for auto-associate
2008-10-14 16:38 [PATCH v2] ipw2200: change default policy for auto-associate Tim Gardner
@ 2008-10-14 16:52 ` Dan Williams
2008-10-14 17:04 ` Dan Williams
0 siblings, 1 reply; 3+ messages in thread
From: Dan Williams @ 2008-10-14 16:52 UTC (permalink / raw)
To: Tim Gardner; +Cc: linux-wireless
On Tue, 2008-10-14 at 10:38 -0600, Tim Gardner wrote:
> From 6f6a49200c74553acac806b6987d5c661346ca00 Mon Sep 17 00:00:00 2001
> From: Tim Gardner <tim.gardner@canonical.com>
> 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 <tim.gardner@canonical.com>
Though for completeness, maybe associate = 0; ?
Acked-by: Dan Williams <dcbw@redhat.com>
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;
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] ipw2200: change default policy for auto-associate
2008-10-14 16:52 ` Dan Williams
@ 2008-10-14 17:04 ` Dan Williams
0 siblings, 0 replies; 3+ messages in thread
From: Dan Williams @ 2008-10-14 17:04 UTC (permalink / raw)
To: Tim Gardner; +Cc: linux-wireless
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 <tim.gardner@canonical.com>
> > 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 <tim.gardner@canonical.com>
>
> 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 <dcbw@redhat.com>
>
> 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 majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2008-10-14 17:05 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-10-14 16:38 [PATCH v2] ipw2200: change default policy for auto-associate Tim Gardner
2008-10-14 16:52 ` Dan Williams
2008-10-14 17:04 ` Dan Williams
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox