diff options
| author | Miri Korenblit <miriam.rachel.korenblit@intel.com> | 2023-08-30 11:30:54 +0300 |
|---|---|---|
| committer | Johannes Berg <johannes.berg@intel.com> | 2023-09-11 12:36:26 +0200 |
| commit | c46fcc6e43d617252945e706f04e5f82a59f2b8e (patch) | |
| tree | 2cd100de86453c8487542c59dbbbcfa580cc0a2e /drivers/net/wireless/intel/iwlwifi/mvm/mld-mac80211.c | |
| parent | 594de1229f89943f4f6140cadb8ea188749d7de8 (diff) | |
| download | linux-stable-c46fcc6e43d617252945e706f04e5f82a59f2b8e.tar.gz linux-stable-c46fcc6e43d617252945e706f04e5f82a59f2b8e.tar.bz2 linux-stable-c46fcc6e43d617252945e706f04e5f82a59f2b8e.zip | |
wifi: iwlwifi: don't use an uninitialized variable
Don't use variable err uninitialized.
The reason for removing the check instead of initializing it
in the beginning of the function is because that way
static checkers will be able to catch issues if we do something
wrong in the future.
Fixes: bf976c814c86 ("wifi: iwlwifi: mvm: implement link change ops")
Signed-off-by: Miri Korenblit <miriam.rachel.korenblit@intel.com>
Signed-off-by: Gregory Greenman <gregory.greenman@intel.com>
Link: https://lore.kernel.org/r/20230830112059.431b01bd8779.I31fc4ab35f551b85a10f974a6b18fc30191e9c35@changeid
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'drivers/net/wireless/intel/iwlwifi/mvm/mld-mac80211.c')
| -rw-r--r-- | drivers/net/wireless/intel/iwlwifi/mvm/mld-mac80211.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/mld-mac80211.c b/drivers/net/wireless/intel/iwlwifi/mvm/mld-mac80211.c index 8b6c641772ee..9615bfff7f7d 100644 --- a/drivers/net/wireless/intel/iwlwifi/mvm/mld-mac80211.c +++ b/drivers/net/wireless/intel/iwlwifi/mvm/mld-mac80211.c @@ -1084,9 +1084,6 @@ iwl_mvm_mld_change_vif_links(struct ieee80211_hw *hw, } } - if (err) - goto out_err; - err = 0; if (new_links == 0) { mvmvif->link[0] = &mvmvif->deflink; |