[PATCH] dbus: Making the peer's properties acessible separately

Reinette Chatre reinette.chatre
Mon Feb 13 16:34:18 PST 2012


On Sun, 2012-02-12 at 12:28 +0200, Jouni Malinen wrote:
> On Sat, Feb 11, 2012 at 05:29:52PM -0600, Dan Williams wrote:
> > > > +	{"config_method", WPAS_DBUS_NEW_IFACE_P2P_PEER, "n",
> > > > +	{"level", WPAS_DBUS_NEW_IFACE_P2P_PEER, "au",
> > > > +	{"SecondaryDeviceTypes", WPAS_DBUS_NEW_IFACE_P2P_PEER, "au",
> 
> > Hmm, using 'n' is pretty weird here since that's a 16-bit signed
> > integer.  I wonder why it wasn't just 'u' (32-bit unsigned).
> 
> I replaced that with "q" to match with DBUS_TYPE_UINT16.
> 
> > The 'au' is "array of uint32" which should be fine,
> 
> Well.. level is actually a signed integer, so "au" was quite odd for it
> (I replaced this with "i"). As far as SecondaryDeviceTypes are
> concerned, array of 32-bit unsigned integers sounds a bit odd. The
> device type is an 8-octet field (which can also be presented as a string
> like 1-0050F204-1) and this field would be an array of those. For now, I
> replaced that with "ay", but I'm not sure whether that really is the
> best approach.

When the patch came in I wrongly assumed that the types will remain as
before, just the interface changed ... Thanks to Jouni for catching
these. From what I can tell the vendor extensions property type is still
different from before. Could this not perhaps remain as bytes?

Reinette






More information about the Hostap mailing list