Patchwork bug? bgscan param is not written back on "save_config"

login
register
mail settings
Submitter Ed W
Date April 1, 2012, 8:39 p.m.
Message ID <4F78BD0B.3090203@wildgooses.com>
Download mbox | patch
Permalink /patch/149984/
State Accepted
Commit af548f5320aac429043b54cc737c7895a05a1691
Headers show

Comments

Ed W - April 1, 2012, 8:39 p.m.
On 01/04/2012 21:10, Ed W wrote:
> Hi, if I use "wpa_cli save_config" then my wpa_supplicant.conf is
> written back without any bg_scan stanzas?  I guess this isn't
> intentional?  Testing hostapd-1.0-git from a few days back
>
> On a related note, it would be nice to have a global param to control
> this for the situation that I always want bg_scan enabled?
>

I believe something like the following would resolve this:



Signed-off-by: Ed Wildgoose <lists@wildgooses.com>
---
Jouni Malinen - April 6, 2012, 3:01 p.m.
On Sun, Apr 01, 2012 at 09:39:39PM +0100, Ed W wrote:
> I believe something like the following would resolve this:
> +++ wpa_supplicant/config_file.c        2012-04-01 21:29:51.000000000 +0100
> +        STR(bgscan);

Thanks, applied.

Patch

--- wpa_supplicant/config_file.c.orig   2012-04-01 21:29:28.000000000 +0100
+++ wpa_supplicant/config_file.c        2012-04-01 21:29:51.000000000 +0100
@@ -526,6 +526,7 @@ 
         write_pairwise(f, ssid);
         write_group(f, ssid);
         write_auth_alg(f, ssid);
+        STR(bgscan);
  #ifdef IEEE8021X_EAPOL
         write_eap(f, ssid);
         STR(identity);