diff mbox

[wowlan] Fix save config error

Message ID 8302AFB577E4BD41809C046CD0155AE225D3A180@mtkmbs32n1
State Superseded
Headers show

Commit Message

Peiyong Feng (冯培勇) Aug. 4, 2014, 10:21 a.m. UTC 
Dear all,

From 34ff0cbe24f1ab024721c8c192b7a357a75fe6b5 Mon Sep 17 00:00:00 2001
From: "peiyong.feng" <peiyong.feng@mediatek.com>
Date: Mon, 4 Aug 2014 18:11:29 +0800
Subject: [PATCH] wowlan_triggers should contain '"'.

This will results supplicant init failure if user has saved config by save_config cmd.

Tested on Android

Signed-off-by: peiyong.feng <peiyong.feng@mediatek.com>
---
 wpa_supplicant/config_file.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Comments

Jouni Malinen Aug. 10, 2014, 8:28 a.m. UTC | #1 
On Mon, Aug 04, 2014 at 10:21:06AM +0000, Peiyong Feng (冯培勇) wrote:
> Subject: [PATCH] wowlan_triggers should contain '"'.
> 
> This will results supplicant init failure if user has saved config by save_config cmd.

Thanks. I already have an earlier patch in the queue, though, so the
same change will be included from it:
http://patchwork.ozlabs.org/patch/372863/
diff mbox

Patch

diff --git a/wpa_supplicant/config_file.c b/wpa_supplicant/config_file.c
index 98855d8..73ad57a 100644
--- a/wpa_supplicant/config_file.c
+++ b/wpa_supplicant/config_file.c
@@ -1170,7 +1170,7 @@  static void wpa_config_write_global(FILE *f, struct wpa_config *config)
 			config->tdls_external_control);
 
 	if (config->wowlan_triggers)
-		fprintf(f, "wowlan_triggers=\"%s\"\n",
+		fprintf(f, "wowlan_triggers=%s\n",
 			config->wowlan_triggers);
 
 	if (config->bgscan)