On Sun, Oct 10, 2010 at 1:44 PM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: >> + Â Â snprintf(buf, 18, "%pM", ea->octet); > > That's a pretty strange use of snprintf -- just use sprintf if you don't > have a real buffer length. Exactly. However, it's out of scope of this certain patch. It should be done as a fix across entire driver. -- With Best Regards, Andy Shevchenko -- 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