ath9k_hw: fix smatch warning in ar9003_hw_spur_mitigate_mrc_cck
authorRajkumar Manoharan <rmanohar@qca.qualcomm.com>
Thu, 21 Jun 2012 15:04:00 +0000 (20:34 +0530)
committerJohn W. Linville <linville@tuxdriver.com>
Fri, 22 Jun 2012 18:44:06 +0000 (14:44 -0400)
drivers/net/wireless/ath/ath9k/ar9003_phy.c:211
ar9003_hw_spur_mitigate_mrc_cck() error: potential NULL dereference
'spur_fbin_ptr'.

Signed-off-by: Rajkumar Manoharan <rmanohar@qca.qualcomm.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ath/ath9k/ar9003_phy.c

index f1975b9e5393a6db7bac0ec65d77fb9947d10830..6b91ebb158fe19216c4b093d0b3a2c25ed74eb9e 100644 (file)
@@ -173,7 +173,7 @@ static void ar9003_hw_spur_mitigate_mrc_cck(struct ath_hw *ah,
        int cur_bb_spur, negative = 0, cck_spur_freq;
        int i;
        int range, max_spur_cnts, synth_freq;
-       u8 *spur_fbin_ptr = NULL;
+       u8 *spur_fbin_ptr = ar9003_get_spur_chan_ptr(ah, IS_CHAN_2GHZ(chan));
 
        /*
         * Need to verify range +/- 10 MHz in control channel, otherwise spur
@@ -181,8 +181,6 @@ static void ar9003_hw_spur_mitigate_mrc_cck(struct ath_hw *ah,
         */
 
        if (AR_SREV_9485(ah) || AR_SREV_9340(ah) || AR_SREV_9330(ah)) {
-               spur_fbin_ptr = ar9003_get_spur_chan_ptr(ah,
-                                                        IS_CHAN_2GHZ(chan));
                if (spur_fbin_ptr[0] == 0) /* No spur */
                        return;
                max_spur_cnts = 5;