[PATCH 4/4] P2P: Fix association with an AP that is not a P2P manager

[Jouni Malinen]

On Mon, May 04, 2015 at 05:34:47AM +0000, Peer, Ilan wrote:
> In short, when an Intel device tried to associate a Soft AP started on a blackberry phone, which was actually a P2P GO, the association was denied with no specific reason. When constructing the association request, since the ssid->p2p_group was no set, the flow eventually called p2p_assoc_req_ie_wlan_ap() to add P2P capability attribute. With the information collected in the bug it seemed that the P2P GO rejected the association since it 'invalid' P2P IE was provided.

Oh.. So this is for the case of associating with a P2P GO while using a
station interface rather than P2P command and P2P Client functionality.

> So a better fix for this issue would to parse the p2p_ie and check if the P2P manageability attribute is included before adding the P2P IE?

I'm not sure this would be strictly speaking compliant with the
specification (but the same could be said about the current design
looking at P2P IE being there in the first place), but anyway, this
would sound like a reasonable interoperability workaround. In any case,
this does indeed need to be done based on the p2p_ie passed to this
function and not any previously collected P2P peer information.

-- 
Jouni Malinen                                            PGP id EFC895FA