On Saturday 25 October 2008 13:28:13 Sebastian Andrzej Siewior wrote: > @@ -662,7 +663,7 @@ int zd_mac_rx(struct ieee80211_hw *hw, const u8 *buffer, unsigned int length) > && !mac->pass_ctrl) > return 0; > > - fc = *(__le16 *)buffer; > + fc = get_unaligned_le16(buffer); This still does semantically change the code. fc is supposed to be LE and is used as LE in the following code. > need_padding = ieee80211_is_data_qos(fc) ^ ieee80211_has_a4(fc); > > skb = dev_alloc_skb(length + (need_padding ? 2 : 0)); -- Greetings Michael. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html