On Thu, 2010-09-30 at 12:25 +0200, Gerrit Renker wrote: > wireless: fix alignment problem > > With the current definition, IW_EV_POINT_PK_LEN is 12 on 64-bit and 8 on 32-bit systems, > due to the way struct iw_event is packed (24 byte on 64-bit systems instead of 20 byte > on 32-bit systems). Furthermore, the iwe_stream_add_point() in include/net/iw_handler.h > also uses IW_EV_LCP_PK_LEN as header length. > > The current definition appears to be a typo (PK_LEN instead of LEN); it causes > misalignment on 64 bit systems. So, correct me if I'm wrong, the only effect this change has is changing the second memcpy() in iwe_stream_add_point() to not copy an extra 4 bytes that will be overwritten right away by the next memcpy() (presumably, unless the data is < 4, in which case the memcpy might actually be out of bounds). Where's the misalignment issue? johannes -- 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
