Re: [PATCH 3/3] staging: brcm80211: fix to use "%pM" flag to printMAC address

[Andy Shevchenko]

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-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/