diff options
author | [email protected] <[email protected]> | 2023-12-06 23:16:11 +0000 |
---|---|---|
committer | David S. Miller <[email protected]> | 2023-12-08 10:56:25 +0000 |
commit | 9b5f621cea6eff2f75b851c6c62cefbdb1673c44 (patch) | |
tree | d835f307d5e658665dfe69499b90e19e80cd04a2 /tools/perf/scripts/python/gecko.py | |
parent | 2a48c635fd9a48699805bbfeee1e4b94b8fe819d (diff) |
checkpatch: add ethtool_sprintf rules
Add some warnings for using ethtool_sprintf() where a simple
ethtool_puts() would suffice.
The two cases are:
1) Use ethtool_sprintf() with just two arguments:
| ethtool_sprintf(&data, driver[i].name);
or
2) Use ethtool_sprintf() with a standalone "%s" fmt string:
| ethtool_sprintf(&data, "%s", driver[i].name);
The former may cause -Wformat-security warnings while the latter is just
not preferred. Both are safely in the category of warnings, not errors.
Signed-off-by: Justin Stitt <[email protected]>
Reviewed-by: Przemek Kitszel <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/gecko.py')
0 files changed, 0 insertions, 0 deletions