Re: [PATCH] wifi: iwlwifi: mvm: fix an error code in iwl_mvm_mld_add_sta()

["Greenman, Gregory" <gregory.greenman@intel.com>]

On Thu, 2023-10-26 at 13:42 +0300, Dan Carpenter wrote:
> Ping?
> 
> regards,
> dan carpenter
> 
> On Wed, Sep 27, 2023 at 03:40:41PM +0300, Dan Carpenter wrote:
> > This error path should return -EINVAL instead of success.
> > 
> > Fixes: 57974a55d995 ("wifi: iwlwifi: mvm: refactor iwl_mvm_mac_sta_state_common()")
> > Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
> > ---
> > I copy and pasted this approach from similar code in the same patch.
> > 
> >  drivers/net/wireless/intel/iwlwifi/mvm/mld-sta.c | 4 +++-
> >  1 file changed, 3 insertions(+), 1 deletion(-)
> > 
> > diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/mld-sta.c b/drivers/net/wireless/intel/iwlwifi/mvm/mld-sta.c
> > index 1464aad039e1..f2e952481ff8 100644
> > --- a/drivers/net/wireless/intel/iwlwifi/mvm/mld-sta.c
> > +++ b/drivers/net/wireless/intel/iwlwifi/mvm/mld-sta.c
> > @@ -707,8 +707,10 @@ int iwl_mvm_mld_add_sta(struct iwl_mvm *mvm, struct ieee80211_vif *vif,
> >                         rcu_dereference_protected(mvm_sta->link[link_id],
> >                                                   lockdep_is_held(&mvm->mutex));
> >  
> > -               if (WARN_ON(!link_conf || !mvm_link_sta))
> > +               if (WARN_ON(!link_conf || !mvm_link_sta)) {
> > +                       ret = -EINVAL;
> >                         goto err;
> > +               }
> >  
> >                 ret = iwl_mvm_mld_cfg_sta(mvm, sta, vif, link_sta, link_conf,
> >                                           mvm_link_sta);
> > -- 
> > 2.39.2

Thanks for the fix.
Acked-by: Gregory Greenman <gregory.greenman@intel.com>