iwlwifi: fix double assign in iwl_start_tx_ba_trans_ready()
authorDan Carpenter <error27@gmail.com>
Wed, 31 Aug 2011 06:37:42 +0000 (09:37 +0300)
committerJohn W. Linville <linville@tuxdriver.com>
Tue, 13 Sep 2011 19:48:48 +0000 (15:48 -0400)
"vif" is assigned twice.  We can remove the first one.

This silences a Smatch warning that "ctx" could be one step past the
end of the priv->contexts[] array.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Reviewed-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/iwlwifi/iwl-core.c

index 20dd1a5506ed82762bac0011c9c9919426fd9562..72b9203c06e2542b9c35a2eee6c4fcd379e27fde 100644 (file)
@@ -1842,7 +1842,7 @@ void iwl_start_tx_ba_trans_ready(struct iwl_priv *priv,
                                 enum iwl_rxon_context_id ctx,
                                 u8 sta_id, u8 tid)
 {
-       struct ieee80211_vif *vif = priv->contexts[ctx].vif;
+       struct ieee80211_vif *vif;
        u8 *addr = priv->stations[sta_id].sta.sta.addr;
 
        if (ctx == NUM_IWL_RXON_CTX)