Re: [PATCH] writing and reading p2p_persistent_group bit in p2p.conf file

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

 



On Wed, Dec 07, 2016 at 07:15:49PM +0530, Avichal Agarwal wrote:
> At the time of starting the wpa_supplicant it reads the persistent network
> Entries from p2p.conf file. Now p2p_persistent_group bit will register the
> the persistent network in dbus on the persistent object path.

> diff --git a/wpa_supplicant/config.c b/wpa_supplicant/config.c
> @@ -2061,6 +2061,7 @@ static const struct parse_data ssid_fields[] = {
>  	{ INT_RANGE(disabled, 0, 2) },
> +	{ INT_RANGE(p2p_persistent_group, 0, 1) },

Hmm.. This does not look correct to me. p2p_persistent_group is really
used only internally and I think it would be unwanted extra complexity
for existing configuration files to assume this is something that would
need to be set in existing persistent group entries. In addition, I
don't think it is good idea to expose option to clear
ssid->p2p_persistent_group from 1 to 0 (or the other direction, for that
matter) through the control interface.

The network entries used for persistent group credentials already have a
clear identifier: disabled=2. It would sound more convenient to set
ssid->p2p_persistent_group = 1 automatically based on the network block
having been stored with ssid->disabled == 2.

-- 
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