diff options
| author | justinstitt@google.com <justinstitt@google.com> | 2023-12-06 23:16:11 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2023-12-08 10:56:25 +0000 |
| commit | 9b5f621cea6eff2f75b851c6c62cefbdb1673c44 (patch) | |
| tree | d835f307d5e658665dfe69499b90e19e80cd04a2 /scripts/gdb/linux/vmalloc.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 <justinstitt@google.com>
Reviewed-by: Przemek Kitszel <przemyslaw.kitszel@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'scripts/gdb/linux/vmalloc.py')
0 files changed, 0 insertions, 0 deletions
