wifi: iwlwifi: mvm: check for iwl_mvm_mld_update_sta() errors
authorJohannes Berg <johannes.berg@intel.com>
Thu, 21 Sep 2023 08:58:09 +0000 (11:58 +0300)
committerJohannes Berg <johannes.berg@intel.com>
Mon, 25 Sep 2023 07:15:00 +0000 (09:15 +0200)
The return value of this function is assigned, but then unused.
Check for errors here.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Gregory Greenman <gregory.greenman@intel.com>
Link: https://lore.kernel.org/r/20230921110727.a9496c232d48.I74adaa8f3c6fd3252348e79f18605246936ef27d@changeid
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
drivers/net/wireless/intel/iwlwifi/mvm/mld-sta.c

index 4e9d19eb31f1364d1778606cce5a0a9549f9a2f2..1464aad039e12094de809f5ba3fe92ebf96683c3 100644 (file)
@@ -697,6 +697,8 @@ int iwl_mvm_mld_add_sta(struct iwl_mvm *mvm, struct ieee80211_vif *vif,
 
        /* at this stage sta link pointers are already allocated */
        ret = iwl_mvm_mld_update_sta(mvm, vif, sta);
+       if (ret)
+               goto err;
 
        for_each_sta_active_link(vif, sta, link_sta, link_id) {
                struct ieee80211_bss_conf *link_conf =