Show
Ignore:
Timestamp:
01/10/08 21:10:41 (4 years ago)
Author:
benoit
Message:

Fixed a kernel panic due to incorrect linux API use
hard_start_xmit() functions must either return NETDEV_TX_OK or
NETDEV_TX_BUSY (they might also return negative errno values as well,
like -ENETDOWN)
Correct a small missing static reported by sparse
This revert part of r3075

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • madwifi/trunk/net80211/ieee80211_power.c

    r3075 r3123  
    210210 * The new packet is placed on the node's saved queue, and the TIM 
    211211 * is changed, if necessary. 
     212 * It must return either NETDEV_TX_OK or NETDEV_TX_BUSY 
    212213 */ 
    213214int 
     
    232233#endif 
    233234                ieee80211_unref_node(&SKB_CB(skb)->ni); 
    234                 ieee80211_dev_kfree_skb(&skb); 
    235235                return NETDEV_TX_BUSY; 
    236236        } 
     
    340340#endif 
    341341 
    342                         (void)ieee80211_parent_queue_xmit(skb); 
     342                        ieee80211_parent_queue_xmit(skb); 
    343343                } 
    344344                vap->iv_set_tim(ni, 0); 
     
    381381                                if (skb == NULL) 
    382382                                        break; 
    383                                 (void)ieee80211_parent_queue_xmit(skb); 
     383                                ieee80211_parent_queue_xmit(skb); 
    384384                        } 
    385385                }