Re: [PATCH] [D-Bus][GroupStarted] Indicate whether created group is persistent or not.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, May 27, 2016 at 02:53:11PM +0530, Nishant Chaprana wrote:
> This patch sends extra parameter in GroupStarted event to indicate
> whether created group is Persistent or not. It is similar to
> [PERSISTENT] tag which comes in P2P-GROUP-STARTED over CTRL interface.

This sounds fine..

> diff --git a/wpa_supplicant/dbus/dbus_new.c b/wpa_supplicant/dbus/dbus_new.c
> @@ -1300,6 +1300,8 @@ void wpas_dbus_signal_p2p_group_started(struct wpa_supplicant *wpa_s,
> +	    !wpa_dbus_dict_append_bool(&dict_iter, "persistent",
> +					 network_id ? 1 : 0) ||

.. but this does not do that correctly. network_id is the id of the
network block and it could at least in theory be 0 for the persistent
group parameters and it can certainly be non-zero for a non-persistent
group (if there are other network blocks in the configuration).

network_id is actually unused in the current
wpas_dbus_signal_p2p_group_started(). I think it should actually be
removed since it does not seem to be a very meaningful parameter or at
least not clearly defined (which network? the current ssid block for the
group or the persistent group storage?). The same seems to apply to the
ssid parameter..

In any case, for this new "persistent" parameter, the caller would need
to provide a new function parameter to get this information reliably
into the signal.
 
-- 
Jouni Malinen                                            PGP id EFC895FA

_______________________________________________
Hostap mailing list
Hostap@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/hostap



[Index of Archives]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux