On Wednesday 28 March 2007 20:23, Johannes Berg wrote: > > + while ((u8*)eth < skb->data + skb->len) { > > I don't understand this cast of eth to u8*. That seems wrong given that > there's an ethhdr there where the first byte isn't a length byte. We're not comparing the data eth points to here, but the eth pointer itself. It's pretty weird code. Maybe this can be solved in a more clear way. But it might be correct as-is. -- 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