On Sat, May 19, 2007 at 11:14:10AM +0800, Eugene Teo wrote: > John W. Linville wrote:
> >> done: > >> LEAVE(); > >> > >> - skb->protocol = __constant_htons(0x0019); /* ETH_P_80211_RAW > >> */ > >> - > > > > Except for this part...is this intentional? > > skb could have been freed by then. And, in libertas_upload_rx_packet(), > skb->protocol > is initialized by eth_type_trans(skb, priv->wlan_dev.netdev). OK, I see that. Looks like Florin has reposted his patch, still without this hunk. Would you like to submit a patch for this hunk? Thanks, John -- John W. Linville [EMAIL PROTECTED] - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
