[RFC 34/40] WPS: add requested device type
Johannes Berg
johannes
Thu Mar 17 07:42:51 PDT 2011
On Thu, 2011-03-17 at 10:41 +0100, Johannes Berg wrote:
> On Thu, 2011-03-17 at 11:29 +0200, Jouni Malinen wrote:
> > On Wed, Jan 05, 2011 at 08:53:30PM +0100, Johannes Berg wrote:
> > > When the requested device type is set, add it
> > > to the probe request WPS attributes.
> >
> > > diff --git a/src/wps/wps.h b/src/wps/wps.h
> > > @@ -90,6 +91,8 @@ struct wps_device_data {
> > > #define WPS_SEC_DEVICE_TYPES 5
> > > u8 sec_dev_type[WPS_SEC_DEVICE_TYPES][WPS_DEV_TYPE_LEN];
> > > u8 nb_sec_dev_types;
> > > + u8 req_dev_type[MAX_REQ_DEV_TYPE_COUNT][WPS_DEV_TYPE_LEN];
> > > + u8 nb_req_dev_types;
> > > u32 os_version;
> > > u8 rf_bands;
> >
> > This does not make much sense to me... Requested Device Type is a
> > parameter for scan commands. Why would it be stored as a device data?
> > Shouldn't it be a parameter for scan/find commands instead?
>
> Yes. Actually, it looks like externally (in DBus) it already is a
> parameter for p2p_find, I'll look at changing this.
http://johannes.sipsolutions.net/patches/hostap/p2p-dbus-7/0009-P2P-add-requested-device-type-to-p2p_find.patch
Actually you should probably look at
http://johannes.sipsolutions.net/patches/hostap/p2p-dbus-7/ because I
also re-ordered some patches. I can also repost if you like.
johannes
More information about the Hostap
mailing list