On Thu, 2010-12-09 at 14:17 +0100, Sven Neumann wrote: [snip > Please review this patch again. I am pretty sure that there's something > broken. Perhaps my fix is not totally correct, but it fixes a > reproducable association problem for us. You're right, the if catches that. Can you please make the variable a bool, remove the blank line before it? And your line length was way above 80. A comment would also be good about why this is done, maybe a one-liner like the probe resp one, just saying "Override IEs if they were from a beacon before". Thanks! 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
