Re: [PATCH 08/14] ctrl: Redirect global MGMT_TX and GET commands to P2P Device interface

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

 



On Thu, Oct 22, 2015 at 02:11:31PM +0000, Peer, Ilan wrote:
> > From: Jouni Malinen [mailto:j@xxxxx]
> > On Wed, Oct 14, 2015 at 06:43:14PM +0300, Ilan Peer wrote:
> > > diff --git a/wpa_supplicant/ctrl_iface.c b/wpa_supplicant/ctrl_iface.c
> > > wpas_global_ctrl_iface_redir_p2p(struct wpa_global *global,
> > > +#ifdef CONFIG_TESTING_OPTIONS
> > > +		"MGMT_TX ",
> > > +		"GET ",
> > > +#endif /* CONFIG_TESTING_OPTIONS */
> > 
> > Neither of these does not seem correct to redirect to a P2P interface.
> > These are in no way specific to P2P and especially MGMT_TX looks pretty
> > strange on the global interface since it is specific to an interface.
> 
> Would it be better to state the interface names explicitly? So wlanX would be used in case the P2P Device is not used, and p2p-dev otherwise?

Yes, the correct interface name would need to be specified in the test
script. I'm not completely sure what test case operations this is needed
in, so cannot comment on the proper way of handling it there, but I
don't see a wpa_supplicant change to be the correct way to do this.

-- 
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