[PATCH v2] staging: fieldbus: Use %pM format specifier for MAC addresses

Sven Van Asbroeck thesven73 at gmail.com
Wed Oct 28 23:39:58 UTC 2020


Hi Andy, thank you for the patch !

On Tue, Oct 27, 2020 at 2:34 PM Andy Shevchenko
<andriy.shevchenko at linux.intel.com> wrote:
>
> +       return snprintf(buf, max_size, "%pM\n", response.addr);

Judging from a few Outreachy patches that have hit my inbox, snprintf() is
considered unsafe in a sysfs_get callback. It should be replaced by
scnprintf() or even better, sysfs_emit(), which was recently introduced
to address sprintf-variant issues in sysfs callbacks.


More information about the devel mailing list