On Thu, 2013-08-01 at 09:56 +0200, Karl Beldan wrote: > On Thu, Aug 01, 2013 at 09:35:54AM +0200, Johannes Berg wrote: > > 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; > > > I don't see what's wrong. > The whole radiotap space is already zeroed, as for the 'leaks' I don't > see how it could leak either. Oh, I didn't see that, I'll apply the patch then. 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
