Return-path: Received: from mx2.redhat.com ([66.187.237.31]:44649 "EHLO mx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751528AbYJNQx3 (ORCPT ); Tue, 14 Oct 2008 12:53:29 -0400 Subject: Re: [PATCH v2] ipw2200: change default policy for auto-associate From: Dan Williams To: Tim Gardner Cc: linux-wireless@vger.kernel.org In-Reply-To: <20081014163803.AD89B5A1D8@sepang.rtg.net> References: <20081014163803.AD89B5A1D8@sepang.rtg.net> Content-Type: text/plain Date: Tue, 14 Oct 2008 12:52:19 -0400 Message-Id: <1224003139.1799.12.camel@localhost.localdomain> (sfid-20081014_185332_751162_D1809212) Mime-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Tue, 2008-10-14 at 10:38 -0600, Tim Gardner wrote: > From 6f6a49200c74553acac806b6987d5c661346ca00 Mon Sep 17 00:00:00 2001 > From: Tim Gardner > 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 Though for completeness, maybe associate = 0; ? Acked-by: Dan Williams 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;