After digging deeper into supplicant dbus handling code, the variables passed via dbus are automagically taken into use so no problems here after all.

Sorry for the noise,
Jukka


On 22.05.2013 09:57, Jukka Rissanen wrote:
Hi,

I indeed noticed that code earlier. Unfortunately that commit does not
add support for passing the variable via dbus. So some additional
patches are required for dbus support.

Cheers,
Jukka


On 22.05.2013 03:42, Chengyi Zhao wrote:
git://w1.fi/srv/git/hostap.git

commit e62f4ed0df72987f874b9bc5c9fa8bec938133ac
Author: Vivek Natarajan <natar...@qca.qualcomm.com>
Date:   Mon Feb 27 16:28:45 2012 +0200

     Add support for hidden SSID in wpa_supplicant AP mode

     This also disables WPS support if hidden SSID is enabled in AP mode.

     Signed-off-by: Vivek Natarajan <natar...@qca.qualcomm.com>
     Signed-hostap: Jouni Malinen <jo...@qca.qualcomm.com>

diff --git a/wpa_supplicant/ap.c b/wpa_supplicant/ap.c
index 52f421a..48a6169 100644
--- a/wpa_supplicant/ap.c
+++ b/wpa_supplicant/ap.c
@@ -145,6 +145,8 @@ static int wpa_supplicant_conf_ap(struct
wpa_supplicant
*wpa_s,
         bss->ssid.ssid_len = ssid->ssid_len;
         bss->ssid.ssid_set = 1;

+       bss->ignore_broadcast_ssid = ssid->ignore_broadcast_ssid;
+
         if (ssid->auth_alg)
                 bss->auth_algs = ssid->auth_alg;

@@ -238,7 +240,10 @@ static int wpa_supplicant_conf_ap(struct
wpa_supplicant *wpa_s,
                               * configuration */
  #endif /* CONFIG_WPS2 */
         bss->eap_server = 1;
-       bss->wps_state = 2;
+
+       if (!ssid->ignore_broadcast_ssid)
+               bss->wps_state = 2;
+
         bss->ap_setup_locked = 2;
         if (wpa_s->conf->config_methods)
                 bss->config_methods =
os_strdup(wpa_s->conf->config_methods);
diff --git a/wpa_supplicant/config.c b/wpa_supplicant/config.c
index 9fd2a70..0aae8af 100644
--- a/wpa_supplicant/config.c
+++ b/wpa_supplicant/config.c
@@ -1589,6 +1589,7 @@ static const struct parse_data ssid_fields[] = {
         { INT_RANGE(frequency, 0, 10000) },
         { INT(wpa_ptk_rekey) },
         { STR(bgscan) },
+       { INT_RANGE(ignore_broadcast_ssid, 0, 2) },
  #ifdef CONFIG_P2P
         { FUNC(p2p_client_list) },
  #endif /* CONFIG_P2P */
diff --git a/wpa_supplicant/config_ssid.h b/wpa_supplicant/config_ssid.h
index 2605ae8..f2bb597 100644
--- a/wpa_supplicant/config_ssid.h
+++ b/wpa_supplicant/config_ssid.h
@@ -377,6 +377,20 @@ struct wpa_ssid {
         char *bgscan;


Best Regards,

Chengyi


2013/5/21 Jukka Rissanen <jukka.rissa...@linux.intel.com>

Hi,


On 21.05.2013 13:39, Chengyi Zhao wrote:

From: Chengyi Zhao <chengyix.z...@gmail.com>

Sometimes users need create the hidden Wi-Fi Access Point.

Because wpa_supplicant 2.0 has already added this hidden DBus interface
(the code "int ignore_broadcast_ssid;" of the file config_ssid.h, etc),
it is recommended to use wpa_supplicant 2.0 or later.

Signed-off-by: Chengyi Zhao <chengyix.z...@gmail.com>
---
   doc/technology-api.txt     |    5 +++++
   gsupplicant/gsupplicant.h  |    7 +++++++
   gsupplicant/supplicant.c   |    4 ++++
   include/technology.h       |    3 ++-
   plugins/bluetooth.c        |    3 ++-
   plugins/bluetooth_legacy.c |    3 ++-
   plugins/ethernet.c         |    6 ++++--
   plugins/wifi.c             |   14 +++++++++++---
   src/technology.c           |   22 +++++++++++++++++++++-
   9 files changed, 58 insertions(+), 9 deletions(-)



  --- a/gsupplicant/supplicant.c
+++ b/gsupplicant/supplicant.c
@@ -3460,6 +3460,10 @@ static void
interface_add_network_params(**DBusMessageIter
*iter, void *user_data)
                                         DBUS_TYPE_BYTE, &ssid->ssid,
                                                 ssid->ssid_len);

+       supplicant_dbus_dict_append_**basic(&dict,
"ignore_broadcast_ssid",
+                                       DBUS_TYPE_INT32,
+                                       &ssid->ignore_broadcast_ssid);
+
         supplicant_dbus_dict_close(**iter, &dict);
   }


After a quick look into wpa_supplicant sources, I could not find this
ignore_broadcast_ssid property being handled in the dbus handler code.
What version of supplicant contains support for this dbus property?


Cheers,
Jukka


______________________________**_________________
connman mailing list
connman@connman.net
https://lists.connman.net/**mailman/listinfo/connman<https://lists.connman.net/mailman/listinfo/connman>


_______________________________________________
connman mailing list
connman@connman.net
https://lists.connman.net/mailman/listinfo/connman


_______________________________________________
connman mailing list
connman@connman.net
https://lists.connman.net/mailman/listinfo/connman


_______________________________________________
connman mailing list
connman@connman.net
https://lists.connman.net/mailman/listinfo/connman

Reply via email to