diff options
author | Justin M. Keyes <justinkz@gmail.com> | 2017-01-23 14:38:51 +0100 |
---|---|---|
committer | Justin M. Keyes <justinkz@gmail.com> | 2017-01-23 15:49:37 +0100 |
commit | b70a5cdd49ecd5f3fe749c1c66a169fee828c66e (patch) | |
tree | 974ccd5d14bb258403f3ec274ac93e8dfd651ca6 /src/clint.py | |
parent | 7e799b6e910880d37e47c86ac46f12ce1c1c8a25 (diff) | |
download | rneovim-b70a5cdd49ecd5f3fe749c1c66a169fee828c66e.tar.gz rneovim-b70a5cdd49ecd5f3fe749c1c66a169fee828c66e.tar.bz2 rneovim-b70a5cdd49ecd5f3fe749c1c66a169fee828c66e.zip |
clint.py: Suggest xstrlcat instead of strcat.
Diffstat (limited to 'src/clint.py')
-rwxr-xr-x | src/clint.py | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/src/clint.py b/src/clint.py index 0c9f55c71e..df71282362 100755 --- a/src/clint.py +++ b/src/clint.py @@ -3166,11 +3166,15 @@ def CheckLanguage(filename, clean_lines, linenum, file_extension, # Check if some verboten C functions are being used. if Search(r'\bsprintf\b', line): error(filename, linenum, 'runtime/printf', 5, - 'Never use sprintf. Use snprintf instead.') - match = Search(r'\b(strcpy|strcat)\b', line) + 'Use snprintf instead of sprintf.') + match = Search(r'\b(STRCPY|strcpy)\b', line) if match: error(filename, linenum, 'runtime/printf', 4, - 'Almost always, snprintf is better than %s' % match.group(1)) + 'Use xstrlcpy or snprintf instead of %s' % match.group(1)) + match = Search(r'\b(STRNCAT|strncat)\b', line) + if match: + error(filename, linenum, 'runtime/printf', 4, + 'Use xstrlcat instead of %s' % match.group(1)) # Check for suspicious usage of "if" like # } if (a == b) { |