On 7/31/07, Michael Buesch <mb@xxxxxxxxx> wrote: > On Tuesday 31 July 2007, Zhu Yi wrote: > > From: Tomas Winkler <tomas.winkler@xxxxxxxxx> > > > > This patch fixes endianity issues in rx configuration > > related code. > > > > Signed-off-by: Gregory Greenman <gregory.greenman@xxxxxxxxx> > > Signed-off-by: Tomas Winkler <tomas.winkler@xxxxxxxxx> > > Signed-off-by: Zhu Yi <yi.zhu@xxxxxxxxx> > > --- > > drivers/net/wireless/iwl-4965.c | 26 ++++--- > > drivers/net/wireless/iwl-base.c | 148 +++++++++++++++++++++------------------ > > drivers/net/wireless/iwl-hw.h | 10 ++-- > > 3 files changed, 101 insertions(+), 83 deletions(-) > > > > diff --git a/drivers/net/wireless/iwl-4965.c b/drivers/net/wireless/iwl-4965.c > > index 19591a6..786c5d0 100644 > > --- a/drivers/net/wireless/iwl-4965.c > > +++ b/drivers/net/wireless/iwl-4965.c > > @@ -3615,8 +3615,8 @@ static void iwl4965_handle_data_packet(struct iwl_priv *priv, int is_data, > > > > hdr = (void *)(pkt->u.raw + > > sizeof(struct iwl4965_rx_mpdu_res_start)); > > - rx_end = (u32 *) (((u8 *) hdr) + amsdu->byte_count); > > len = amsdu->byte_count; > > + rx_end = (u32 *) (((u8 *) hdr) + len); > > I doubt rx_end is a pointer to u32. Most likely you want to use > (__le32 *) here and also audit the users of rx_end, if they > need le32_to_cpu(). > You are right. Only the len parameter was fixed. I will send another patch that fixes also the pointer. Thanks Tomas > 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 > - 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