commit ef9d6aa70d68cd3a765ed55f4c3781aeb8aeea23
author Bram Moolenaar <Bram@vim.org> Mon Apr 11 16:56:35 2011 +0200
committer Bram Moolenaar <Bram@vim.org> Mon Apr 11 16:56:35 2011 +0200
tree e099b52d0ebf51c535ebe3cd875d8f70c06332df
parent 0d35e91abfa9e17f7c554bfd33b119b879448c72

updated for version 7.3.160
Problem:    Unsafe string copying.
Solution:   Use vim_strncpy() instead of strcpy().  Use vim_strcat() instead
            of strcat().

src/menu.c

diff --git a/src/menu.c b/src/menu.c
index 4d2bce1..42cf67e 100644
--- a/src/menu.c
+++ b/src/menu.c
@@ -1394,7 +1394,8 @@
     int		idx;
 {
     static vimmenu_T	*menu = NULL;
-    static char_u	tbuffer[256]; /*hack*/
+#define TBUFFER_LEN 256
+    static char_u	tbuffer[TBUFFER_LEN]; /*hack*/
     char_u		*str;
 #ifdef FEAT_MULTI_LANG
     static  int		should_advance = FALSE;
@@ -1428,11 +1429,11 @@
 	{
 #ifdef FEAT_MULTI_LANG
 	    if (should_advance)
-		STRCPY(tbuffer, menu->en_dname);
+		vim_strncpy(tbuffer, menu->en_dname, TBUFFER_LEN - 2);
 	    else
 	    {
 #endif
-		STRCPY(tbuffer, menu->dname);
+		vim_strncpy(tbuffer, menu->dname,  TBUFFER_LEN - 2);
 #ifdef FEAT_MULTI_LANG
 		if (menu->en_dname == NULL)
 		    should_advance = TRUE;