Re: [PATCH v3 3/9] cfg80211: add add_nan_func / del_nan_func

2016-10-22 Thread Johannes Berg
On Sat, 2016-10-22 at 21:17 +0200, Arend van Spriel wrote: > On 20-09-16 16:31, Luca Coelho wrote: > > > > + .flags = GENL_ADMIN_PERM, > Recently the nl80211_ops flags were changed to GENL_UNS_ADMIN_PERM so > should the NAN commands use that as well? The dangers of having such code non

Re: [PATCH v3 3/9] cfg80211: add add_nan_func / del_nan_func

2016-10-22 Thread Arend van Spriel
On 20-09-16 16:31, Luca Coelho wrote: > + .flags = GENL_ADMIN_PERM, Hi Johannes, Recently the nl80211_ops flags were changed to GENL_UNS_ADMIN_PERM so should the NAN commands use that as well? Regards, Arend

[PATCH v3 3/9] cfg80211: add add_nan_func / del_nan_func

2016-09-20 Thread Luca Coelho
From: Ayala Beker A NAN function can be either publish, subscribe or follow up. Make all the necessary verifications and just pass the request to the driver. Allow the user space application that starts NAN to forbid any other socket to add or remove functions. Signed-off-by: Andrei Otcheretians