On Sat, 2013-07-27 at 11:47 +0200, Karl Beldan wrote: > + /* required alignment from rthdr */ > + pos = (u8 *)rthdr + ALIGN(pos - (u8 *)rthdr, 2); This is bad, it potentially leaks a byte of kernel data, please explicitly clear the padding, like if ((pos - (u8 *)rthdr) & 1) *pos++ = 0; or so. 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