To: vim_dev@googlegroups.com
Subject: Patch 7.4.1681
Fcc: outbox
From: Bram Moolenaar <Bram@moolenaar.net>
Mime-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
------------

Patch 7.4.1681
Problem:    Coverity warns for fixed size buffer length (false positive).
Solution:   Add a check for the name length.
Files:      src/eval.c


*** ../vim-7.4.1680/src/eval.c	2016-03-28 22:30:46.795497938 +0200
--- src/eval.c	2016-03-28 22:41:41.512841267 +0200
***************
*** 933,938 ****
--- 933,943 ----
      for (i = 0; i < VV_LEN; ++i)
      {
  	p = &vimvars[i];
+ 	if (STRLEN(p->vv_name) > 16)
+ 	{
+ 	    EMSG("INTERNAL: name too long, increase size of dictitem16_T");
+ 	    getout(1);
+ 	}
  	STRCPY(p->vv_di.di_key, p->vv_name);
  	if (p->vv_flags & VV_RO)
  	    p->vv_di.di_flags = DI_FLAGS_RO | DI_FLAGS_FIX;
*** ../vim-7.4.1680/src/version.c	2016-03-28 22:36:02.392289283 +0200
--- src/version.c	2016-03-28 22:42:27.344375259 +0200
***************
*** 750,751 ****
--- 750,753 ----
  {   /* Add new patch number below this line */
+ /**/
+     1681,
  /**/

-- 
hundred-and-one symptoms of being an internet addict:
162. You go outside and look for a brightness knob to turn down the sun.

 /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\  an exciting new programming language -- http://www.Zimbu.org        ///
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///