On Mon, 2013-04-22 at 16:08 +0200, Felix Fietkau wrote: > >> + if (ratetbl->rate[0].idx < 0) > >> + return TX_DROP; > > > > This can't happen, idx is unsigned. Did you intend idx to be signed? > Right, shouldn't the compiler warn about this? Will send v6. I don't think the compiler does (probably would be too many false positives), but smatch did :-) 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