[PATCH] Android: Skip explicit conf_p2p_dev loading for main interface
Arend van Spriel
arend at broadcom.com
Mon Sep 29 09:36:17 EDT 2014
On 09/29/14 13:05, Peer, Ilan wrote:
> Hi Dmitry,
> My original reply was bounced by the moderator … so here it is again:
> You are correct; the –m switch does not currently make much sense in
> case of netdev interface used for P2P_DEVICE operations (and also there
> is no need to specify it for the main interface).
> The first call to wpa_supplicant_add_interface() add the network
> interface, but in that case iface->p2p_mgmt would not be set, so
> checking would allow to skip reading the p2p device configuration file.
> After the netdev interface is created, the code creates an additional
> interface for the P2P Device management (only if one does not exist and
> the underlying device supports it). So in this case
> wpas_p2p_add_p2pdev_interface() is called, and only in this case
> iface->p2p_mgmt is set.
> So bottom line, I think that a cleaner fix would be to base the reading
> on the setting of iface->p2p_mgmt.
Related to this I created a patch that would make the -m option
redundant. This is done by skipping networks and ignore 'p2p_disabled=1'
in the configuration when iface->p2p_mgmt is set.
> HostAP mailing list
> HostAP at lists.shmoo.com
More information about the HostAP