Changeset 3665

Show
Ignore:
Timestamp:
05/22/08 14:48:52 (8 months ago)
Author:
mentor
Message:

Make sure to free the skb if it is not used. This is brute force programming; I blame gotos.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • madwifi/trunk/ath/if_ath.c

    r3659 r3665  
    66306630                 * before passing to other VAPs. Ignore MIC failures here, as  
    66316631                 * we need to recheck them. */ 
    6632                 if (rs->rs_status & ~(HAL_RXERR_MIC | HAL_RXERR_DECRYPT)) 
     6632                if (rs->rs_status & ~(HAL_RXERR_MIC | HAL_RXERR_DECRYPT)) { 
     6633                        ieee80211_dev_kfree_skb(&skb); 
    66336634                        goto rx_next; 
     6635                } 
    66346636 
    66356637                /* Remove the CRC. */ 
    …… 
    66546656drop_micfail: 
    66556657                        mic_fail = 0; 
     6658 
     6659                        ieee80211_dev_kfree_skb(&skb); 
    66566660                        goto rx_next; 
    66576661                } 
    …… 
    66646668                        sc->sc_stats.ast_rx_tooshort++; 
    66656669                        errors++; 
     6670                        ieee80211_dev_kfree_skb(&skb); 
    66666671                        goto rx_next; 
    66676672                }