Prohibit verbose strcat

Using strcat directly is more readable than passing strlen
of the copied string to strncat.

1 files changed, 6 insertions, 0 deletions

diff --git a/cfg.mk b/cfg.mk
index 71b0866d52..3f78842a5b 100644
--- a/cfg.mk
+++ b/cfg.mk
@@ -1031,6 +1031,12 @@ sc_prohibit_not_strneq:
 	halt='Use STREQ instead of !STRNEQ'	\
 	  $(_sc_search_regexp)
 
+sc_prohibit_verbose_strcat:
+	@prohibit='strncat\([^,]*,\s+([^,]*),\s+strlen\(\1\)\)'     \
+	in_vc_files='\.[ch]$$'                                      \
+	halt='Use strcat(a, b) instead of strncat(a, b, strlen(b))' \
+	  $(_sc_search_regexp)
+
 # We don't use this feature of maint.mk.
 prev_version_file = /dev/null