Re: [PATCH v4] Add application extension data to WPS IE

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

 



On Thu, 2 Jan 2020 at 17:08, Jouni Malinen <j@xxxxx> wrote:
>
> On Thu, Jan 02, 2020 at 01:30:50PM +0300, bilalhp@xxxxxxxxx wrote:
> > Application Extension attribute is defined in WPS Spec v2.06 page 105.
> > This patch makes hostapd add this application extension data to WPS IE
> > if configured. The implementation is very similar to vendor extension.
> >
> > Added a new optional entry called "wps_application_ext[]" to hostapd config
> > file to configure this.
>
> That looks overly complex. Why would there be need to assign specific
> application extension indexes? Couldn't the next index value be assigned
> automatically so that first wps_application_ext=<hex> entry would get
> index 0 and the next such line would get index 1? Or alternatively,
> support only one Application Extension attribute if there is no existing
> use case that needs more of those.. It is not likely WSC should really
> be used for anything new and instead, DPP should be considered to be the
> current standard for doing device provisioning.

OK. I will make it single Application Extension attribute in v5.

>
> It should also be noted that the WSC technical specification does not
> say anything about including Application Extension attributes in Beacon
> or Probe Response frames; the only example use case for these attributes
> is within the encrypted section of M7 and M8. This patch seems to be
> doing something very different. Is that use defined in some public
> specification?

Actually, I have not seen any existing usage in any code base. My
understanding is that this has not been used ever in any of the WSC
implementations so far.

As you already stated, there is an example use defined in the spec,
which has never leveraged by anyone yet, and I don't think it would be
the case in the future ever as you stated.

Our understanding from WSC spec is that this is a standard TLV
definition and it can be used optionally in any of the frames,
including Beacon and Probe Responses. This is what we understand from:

<other…> O Multiple attributes are permitted.

row in tables inside sections 8.2 and 8.3.

Regards.

>
> > ---
> >  hostapd/config_file.c   | 12 ++++++++++++
> >  src/ap/ap_config.c      |  2 ++
> >  src/ap/ap_config.h      |  1 +
> >  src/ap/wps_hostapd.c    | 29 +++++++++++++++++++++++++++++
> >  src/wps/wps.h           |  5 +++++
> >  src/wps/wps_dev_attr.c  | 19 +++++++++++++++++++
> >  src/wps/wps_dev_attr.h  |  1 +
> >  src/wps/wps_registrar.c |  6 ++++--
> >  8 files changed, 73 insertions(+), 2 deletions(-)
>
> hostapd/hostapd.conf should also be updated to document the new
> configuration parameter, including the format it uses and requirements
> for the contents (UUID in the beginning).
>
> --
> 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