diff options
| author | Johannes Berg <johannes.berg@intel.com> | 2015-04-01 10:00:31 +0200 |
|---|---|---|
| committer | Emmanuel Grumbach <emmanuel.grumbach@intel.com> | 2015-04-02 09:29:12 +0300 |
| commit | 10a7c028dbf9920ba6781bf1adde1811dc50de86 (patch) | |
| tree | 3ddf98364f046a7e74ea167f21c0ceffa1757a1e /drivers/net/wireless/iwlwifi | |
| parent | d42f53503406d5dcedbad9ea18c964f189f72d84 (diff) | |
| download | lwn-10a7c028dbf9920ba6781bf1adde1811dc50de86.tar.gz lwn-10a7c028dbf9920ba6781bf1adde1811dc50de86.zip | |
iwlwifi: mvm: don't return uninitialized value in get_survey()
If ucode_loaded isn't true the function returns the 'ret' variable
without having assigned a value properly. Fix that.
Reported-by: Haim Dreyfuss <haim.dreyfuss@intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Diffstat (limited to 'drivers/net/wireless/iwlwifi')
| -rw-r--r-- | drivers/net/wireless/iwlwifi/mvm/mac80211.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/net/wireless/iwlwifi/mvm/mac80211.c b/drivers/net/wireless/iwlwifi/mvm/mac80211.c index fc73cc1bda0d..74912e7fd3b2 100644 --- a/drivers/net/wireless/iwlwifi/mvm/mac80211.c +++ b/drivers/net/wireless/iwlwifi/mvm/mac80211.c @@ -3911,6 +3911,7 @@ static int iwl_mvm_mac_get_survey(struct ieee80211_hw *hw, int idx, mvm->radio_stats.on_time_scan; do_div(survey->time_scan, USEC_PER_MSEC); + ret = 0; out: mutex_unlock(&mvm->mutex); return ret; |