Re: [PATCH v3] cfg80211: clean up mesh plink station change API

[Johannes Berg <johannes@sipsolutions.net>]

On Thu, 2013-02-14 at 14:52 -0800, Thomas Pedersen wrote:
> On Thu, Feb 14, 2013 at 2:37 PM, Johannes Berg
> <johannes@sipsolutions.net> wrote:
> > From: Johannes Berg <johannes.berg@intel.com>
> >
> > Make the ability to leave the plink_state unchanged not use a
> > magic -1 variable that isn't in the enum, but an explicit change
> > flag; reject invalid plink states or actions and move the needed
> > constants for plink actions to the right header file. Also
> > reject plink_state changes for non-mesh interfaces.
> >
> > Signed-off-by: Johannes Berg <johannes.berg@intel.com>
> 
> > diff --git a/net/mac80211/cfg.c b/net/mac80211/cfg.c
> > index 909053a..8e1a03e 100644
> > --- a/net/mac80211/cfg.c
> > +++ b/net/mac80211/cfg.c
> > @@ -1261,7 +1261,9 @@ static int sta_apply_parameters(struct ieee80211_local *local,
> >         if (ieee80211_vif_is_mesh(&sdata->vif)) {
> >  #ifdef CONFIG_MAC80211_MESH
> >                 u32 changed = 0;
> > -               if (sdata->u.mesh.security & IEEE80211_MESH_SEC_SECURED) {
> > +               if (sdata->u.mesh.security & IEEE80211_MESH_SEC_SECURED &&
> > +                   (params->sta_modify_mask &
> > +                                       STATION_PARAM_APPLY_PLINK_STATE)) {
> >                         switch (params->plink_state) {
> >                         case NL80211_PLINK_ESTAB:
> >                                 if (sta->plink_state != NL80211_PLINK_ESTAB)
> > @@ -1292,12 +1294,18 @@ static int sta_apply_parameters(struct ieee80211_local *local,
> >                                 /*  nothing  */
> >                                 break;
> >                         }
> > +               } else if (params->sta_modify_mask &
> > +                                       STATION_PARAM_APPLY_PLINK_STATE) {
> > +                       return -EINVAL;
> 
> It would make sense to disallow setting plink_action if MPM is in userspace.

I'm not sure what you mean?

johannes