Tree - rpms/vim - CentOS Git server

rpms / vim

Files

Commit: 28c3372dc91880aaebefdad60ce457533a4de03a

Blob Blame History Raw

 To: vim_dev@googlegroups.com
Subject: Patch 7.3.898
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.3.898
Problem:    Memory leak reported by valgrind in test 91.
Solution:   Only use default argument when needed.
Files:	    src/eval.c, src/testdir/test91.in, src/testdir/test91.ok
 
 
*** ../vim-7.3.897/src/eval.c	2013-04-15 13:06:15.000000000 +0200
--- src/eval.c	2013-04-15 15:09:17.000000000 +0200
***************
*** 11120,11139 ****
      buf_T	*save_curbuf;
      char_u	*varname;
      dictitem_T	*v;
  
      (void)get_tv_number(&argvars[0]);	    /* issue errmsg if type error */
      varname = get_tv_string_chk(&argvars[1]);
      ++emsg_off;
      buf = get_buf_tv(&argvars[0], FALSE);
  
!     if (argvars[2].v_type != VAR_UNKNOWN)
! 	/* set the default value */
! 	copy_tv(&argvars[2], rettv);
!     else
!     {
! 	rettv->v_type = VAR_STRING;
! 	rettv->vval.v_string = NULL;
!     }
  
      if (buf != NULL && varname != NULL)
      {
--- 11120,11134 ----
      buf_T	*save_curbuf;
      char_u	*varname;
      dictitem_T	*v;
+     int		done = FALSE;
  
      (void)get_tv_number(&argvars[0]);	    /* issue errmsg if type error */
      varname = get_tv_string_chk(&argvars[1]);
      ++emsg_off;
      buf = get_buf_tv(&argvars[0], FALSE);
  
!     rettv->v_type = VAR_STRING;
!     rettv->vval.v_string = NULL;
  
      if (buf != NULL && varname != NULL)
      {
***************
*** 11142,11152 ****
  	curbuf = buf;
  
  	if (*varname == '&')	/* buffer-local-option */
! 	    get_option_tv(&varname, rettv, TRUE);
  	else if (STRCMP(varname, "changedtick") == 0)
  	{
  	    rettv->v_type = VAR_NUMBER;
  	    rettv->vval.v_number = curbuf->b_changedtick;
  	}
  	else
  	{
--- 11137,11151 ----
  	curbuf = buf;
  
  	if (*varname == '&')	/* buffer-local-option */
! 	{
! 	    if (get_option_tv(&varname, rettv, TRUE) == OK)
! 		done = TRUE;
! 	}
  	else if (STRCMP(varname, "changedtick") == 0)
  	{
  	    rettv->v_type = VAR_NUMBER;
  	    rettv->vval.v_number = curbuf->b_changedtick;
+ 	    done = TRUE;
  	}
  	else
  	{
***************
*** 11155,11167 ****
--- 11154,11173 ----
  	    v = find_var_in_ht(&curbuf->b_vars->dv_hashtab,
  							 'b', varname, FALSE);
  	    if (v != NULL)
+ 	    {
  		copy_tv(&v->di_tv, rettv);
+ 		done = TRUE;
+ 	    }
  	}
  
  	/* restore previous notion of curbuf */
  	curbuf = save_curbuf;
      }
  
+     if (!done && argvars[2].v_type != VAR_UNKNOWN)
+ 	/* use the default value */
+ 	copy_tv(&argvars[2], rettv);
+ 
      --emsg_off;
  }
  
***************
*** 11767,11772 ****
--- 11773,11779 ----
      tabpage_T	*tp;
      dictitem_T	*v;
      char_u	*varname;
+     int		done = FALSE;
  
      rettv->v_type = VAR_STRING;
      rettv->vval.v_string = NULL;
***************
*** 11778,11788 ****
  	/* look up the variable */
  	v = find_var_in_ht(&tp->tp_vars->dv_hashtab, 0, varname, FALSE);
  	if (v != NULL)
  	    copy_tv(&v->di_tv, rettv);
! 	else if (argvars[2].v_type != VAR_UNKNOWN)
! 	    copy_tv(&argvars[2], rettv);
      }
!     else if (argvars[2].v_type != VAR_UNKNOWN)
  	copy_tv(&argvars[2], rettv);
  }
  
--- 11785,11797 ----
  	/* look up the variable */
  	v = find_var_in_ht(&tp->tp_vars->dv_hashtab, 0, varname, FALSE);
  	if (v != NULL)
+ 	{
  	    copy_tv(&v->di_tv, rettv);
! 	    done = TRUE;
! 	}
      }
! 
!     if (!done && argvars[2].v_type != VAR_UNKNOWN)
  	copy_tv(&argvars[2], rettv);
  }
  
***************
*** 11894,11899 ****
--- 11903,11909 ----
      char_u	*varname;
      dictitem_T	*v;
      tabpage_T	*tp;
+     int		done = FALSE;
  
  #ifdef FEAT_WINDOWS
      if (off == 1)
***************
*** 11905,11918 ****
      varname = get_tv_string_chk(&argvars[off + 1]);
      ++emsg_off;
  
!     if (argvars[off + 2].v_type != VAR_UNKNOWN)
! 	/* set the default return value */
! 	copy_tv(&argvars[off + 2], rettv);
!     else
!     {
! 	rettv->v_type = VAR_STRING;
! 	rettv->vval.v_string = NULL;
!     }
  
      if (win != NULL && varname != NULL)
      {
--- 11915,11922 ----
      varname = get_tv_string_chk(&argvars[off + 1]);
      ++emsg_off;
  
!     rettv->v_type = VAR_STRING;
!     rettv->vval.v_string = NULL;
  
      if (win != NULL && varname != NULL)
      {
***************
*** 11923,11936 ****
  	curbuf = win->w_buffer;
  
  	if (*varname == '&')	/* window-local-option */
! 	    get_option_tv(&varname, rettv, 1);
  	else
  	{
  	    /* Look up the variable. */
  	    /* Let getwinvar({nr}, "") return the "w:" dictionary. */
  	    v = find_var_in_ht(&win->w_vars->dv_hashtab, 'w', varname, FALSE);
  	    if (v != NULL)
  		copy_tv(&v->di_tv, rettv);
  	}
  
  	/* restore previous notion of curwin */
--- 11927,11946 ----
  	curbuf = win->w_buffer;
  
  	if (*varname == '&')	/* window-local-option */
! 	{
! 	    if (get_option_tv(&varname, rettv, 1) == OK)
! 		done = TRUE;
! 	}
  	else
  	{
  	    /* Look up the variable. */
  	    /* Let getwinvar({nr}, "") return the "w:" dictionary. */
  	    v = find_var_in_ht(&win->w_vars->dv_hashtab, 'w', varname, FALSE);
  	    if (v != NULL)
+ 	    {
  		copy_tv(&v->di_tv, rettv);
+ 		done = TRUE;
+ 	    }
  	}
  
  	/* restore previous notion of curwin */
***************
*** 11938,11943 ****
--- 11948,11957 ----
  	curbuf = curwin->w_buffer;
      }
  
+     if (!done && argvars[off + 2].v_type != VAR_UNKNOWN)
+ 	/* use the default return value */
+ 	copy_tv(&argvars[off + 2], rettv);
+ 
      --emsg_off;
  }
  
*** ../vim-7.3.897/src/testdir/test91.in	2013-02-20 21:11:14.000000000 +0100
--- src/testdir/test91.in	2013-04-15 14:59:31.000000000 +0200
***************
*** 4,12 ****
  STARTTEST
  :so small.vim
  :"
! :" test for getbufvar()
! :let b:var_num = 1234
! :let def_num = 5678
  :$put =string(getbufvar(1, 'var_num'))
  :$put =string(getbufvar(1, 'var_num', def_num))
  :$put =string(getbufvar(1, ''))
--- 4,13 ----
  STARTTEST
  :so small.vim
  :"
! :" Test for getbufvar()
! :" Use strings to test for memory leaks.
! :let b:var_num = '1234'
! :let def_num = '5678'
  :$put =string(getbufvar(1, 'var_num'))
  :$put =string(getbufvar(1, 'var_num', def_num))
  :$put =string(getbufvar(1, ''))
*** ../vim-7.3.897/src/testdir/test91.ok	2013-02-20 21:11:14.000000000 +0100
--- src/testdir/test91.ok	2013-04-15 15:02:45.000000000 +0200
***************
*** 1,13 ****
  start:
! 1234
! 1234
! {'var_num': 1234}
! {'var_num': 1234}
! 5678
  {}
  {}
  ''
! 5678
  0
  0
  'Dance'
--- 1,13 ----
  start:
! '1234'
! '1234'
! {'var_num': '1234'}
! {'var_num': '1234'}
! '5678'
  {}
  {}
  ''
! '5678'
  0
  0
  'Dance'
*** ../vim-7.3.897/src/version.c	2013-04-15 14:44:53.000000000 +0200
--- src/version.c	2013-04-15 15:14:22.000000000 +0200
***************
*** 730,731 ****
--- 730,733 ----
  {   /* Add new patch number below this line */
+ /**/
+     898,
  /**/
 
-- 
If Apple would build a car...
... it would be powered by the sun, be reliable, five times
as fast and twice as easy to drive; but would only run on
five percent of the roads.
 
 /// 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    ///

	To: vim_dev@googlegroups.com
	Subject: Patch 7.3.898
	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.3.898
	Problem: Memory leak reported by valgrind in test 91.
	Solution: Only use default argument when needed.
	Files: src/eval.c, src/testdir/test91.in, src/testdir/test91.ok


	*** ../vim-7.3.897/src/eval.c 2013-04-15 13:06:15.000000000 +0200
	--- src/eval.c 2013-04-15 15:09:17.000000000 +0200
	***************
	* 11120,11139 **
	buf_T *save_curbuf;
	char_u *varname;
	dictitem_T *v;

	(void)get_tv_number(&argvars[0]); /* issue errmsg if type error */
	varname = get_tv_string_chk(&argvars[1]);
	++emsg_off;
	buf = get_buf_tv(&argvars[0], FALSE);

	! if (argvars[2].v_type != VAR_UNKNOWN)
	! /* set the default value */
	! copy_tv(&argvars[2], rettv);
	! else
	! {
	! rettv->v_type = VAR_STRING;
	! rettv->vval.v_string = NULL;
	! }

	if (buf != NULL && varname != NULL)
	{
	--- 11120,11134 ----
	buf_T *save_curbuf;
	char_u *varname;
	dictitem_T *v;
	+ int done = FALSE;

	(void)get_tv_number(&argvars[0]); /* issue errmsg if type error */
	varname = get_tv_string_chk(&argvars[1]);
	++emsg_off;
	buf = get_buf_tv(&argvars[0], FALSE);

	! rettv->v_type = VAR_STRING;
	! rettv->vval.v_string = NULL;

	if (buf != NULL && varname != NULL)
	{
	***************
	* 11142,11152 **
	curbuf = buf;

	if (varname == '&') / buffer-local-option */
	! get_option_tv(&varname, rettv, TRUE);
	else if (STRCMP(varname, "changedtick") == 0)
	{
	rettv->v_type = VAR_NUMBER;
	rettv->vval.v_number = curbuf->b_changedtick;
	}
	else
	{
	--- 11137,11151 ----
	curbuf = buf;

	if (varname == '&') / buffer-local-option */
	! {
	! if (get_option_tv(&varname, rettv, TRUE) == OK)
	! done = TRUE;
	! }
	else if (STRCMP(varname, "changedtick") == 0)
	{
	rettv->v_type = VAR_NUMBER;
	rettv->vval.v_number = curbuf->b_changedtick;
	+ done = TRUE;
	}
	else
	{
	***************
	* 11155,11167 **
	--- 11154,11173 ----
	v = find_var_in_ht(&curbuf->b_vars->dv_hashtab,
	'b', varname, FALSE);
	if (v != NULL)
	+ {
	copy_tv(&v->di_tv, rettv);
	+ done = TRUE;
	+ }
	}

	/* restore previous notion of curbuf */
	curbuf = save_curbuf;
	}

	+ if (!done && argvars[2].v_type != VAR_UNKNOWN)
	+ /* use the default value */
	+ copy_tv(&argvars[2], rettv);
	+
	--emsg_off;
	}

	***************
	* 11767,11772 **
	--- 11773,11779 ----
	tabpage_T *tp;
	dictitem_T *v;
	char_u *varname;
	+ int done = FALSE;

	rettv->v_type = VAR_STRING;
	rettv->vval.v_string = NULL;
	***************
	* 11778,11788 **
	/* look up the variable */
	v = find_var_in_ht(&tp->tp_vars->dv_hashtab, 0, varname, FALSE);
	if (v != NULL)
	copy_tv(&v->di_tv, rettv);
	! else if (argvars[2].v_type != VAR_UNKNOWN)
	! copy_tv(&argvars[2], rettv);
	}
	! else if (argvars[2].v_type != VAR_UNKNOWN)
	copy_tv(&argvars[2], rettv);
	}

	--- 11785,11797 ----
	/* look up the variable */
	v = find_var_in_ht(&tp->tp_vars->dv_hashtab, 0, varname, FALSE);
	if (v != NULL)
	+ {
	copy_tv(&v->di_tv, rettv);
	! done = TRUE;
	! }
	}
	!
	! if (!done && argvars[2].v_type != VAR_UNKNOWN)
	copy_tv(&argvars[2], rettv);
	}

	***************
	* 11894,11899 **
	--- 11903,11909 ----
	char_u *varname;
	dictitem_T *v;
	tabpage_T *tp;
	+ int done = FALSE;

	#ifdef FEAT_WINDOWS
	if (off == 1)
	***************
	* 11905,11918 **
	varname = get_tv_string_chk(&argvars[off + 1]);
	++emsg_off;

	! if (argvars[off + 2].v_type != VAR_UNKNOWN)
	! /* set the default return value */
	! copy_tv(&argvars[off + 2], rettv);
	! else
	! {
	! rettv->v_type = VAR_STRING;
	! rettv->vval.v_string = NULL;
	! }

	if (win != NULL && varname != NULL)
	{
	--- 11915,11922 ----
	varname = get_tv_string_chk(&argvars[off + 1]);
	++emsg_off;

	! rettv->v_type = VAR_STRING;
	! rettv->vval.v_string = NULL;

	if (win != NULL && varname != NULL)
	{
	***************
	* 11923,11936 **
	curbuf = win->w_buffer;

	if (varname == '&') / window-local-option */
	! get_option_tv(&varname, rettv, 1);
	else
	{
	/* Look up the variable. */
	/* Let getwinvar({nr}, "") return the "w:" dictionary. */
	v = find_var_in_ht(&win->w_vars->dv_hashtab, 'w', varname, FALSE);
	if (v != NULL)
	copy_tv(&v->di_tv, rettv);
	}

	/* restore previous notion of curwin */
	--- 11927,11946 ----
	curbuf = win->w_buffer;

	if (varname == '&') / window-local-option */
	! {
	! if (get_option_tv(&varname, rettv, 1) == OK)
	! done = TRUE;
	! }
	else
	{
	/* Look up the variable. */
	/* Let getwinvar({nr}, "") return the "w:" dictionary. */
	v = find_var_in_ht(&win->w_vars->dv_hashtab, 'w', varname, FALSE);
	if (v != NULL)
	+ {
	copy_tv(&v->di_tv, rettv);
	+ done = TRUE;
	+ }
	}

	/* restore previous notion of curwin */
	***************
	* 11938,11943 **
	--- 11948,11957 ----
	curbuf = curwin->w_buffer;
	}

	+ if (!done && argvars[off + 2].v_type != VAR_UNKNOWN)
	+ /* use the default return value */
	+ copy_tv(&argvars[off + 2], rettv);
	+
	--emsg_off;
	}

	*** ../vim-7.3.897/src/testdir/test91.in 2013-02-20 21:11:14.000000000 +0100
	--- src/testdir/test91.in 2013-04-15 14:59:31.000000000 +0200
	***************
	* 4,12 **
	STARTTEST
	:so small.vim
	:"
	! :" test for getbufvar()
	! :let b:var_num = 1234
	! :let def_num = 5678
	:$put =string(getbufvar(1, 'var_num'))
	:$put =string(getbufvar(1, 'var_num', def_num))
	:$put =string(getbufvar(1, ''))
	--- 4,13 ----
	STARTTEST
	:so small.vim
	:"
	! :" Test for getbufvar()
	! :" Use strings to test for memory leaks.
	! :let b:var_num = '1234'
	! :let def_num = '5678'
	:$put =string(getbufvar(1, 'var_num'))
	:$put =string(getbufvar(1, 'var_num', def_num))
	:$put =string(getbufvar(1, ''))
	*** ../vim-7.3.897/src/testdir/test91.ok 2013-02-20 21:11:14.000000000 +0100
	--- src/testdir/test91.ok 2013-04-15 15:02:45.000000000 +0200
	***************
	* 1,13 **
	start:
	! 1234
	! 1234
	! {'var_num': 1234}
	! {'var_num': 1234}
	! 5678
	{}
	{}
	''
	! 5678
	0
	0
	'Dance'
	--- 1,13 ----
	start:
	! '1234'
	! '1234'
	! {'var_num': '1234'}
	! {'var_num': '1234'}
	! '5678'
	{}
	{}
	''
	! '5678'
	0
	0
	'Dance'
	*** ../vim-7.3.897/src/version.c 2013-04-15 14:44:53.000000000 +0200
	--- src/version.c 2013-04-15 15:14:22.000000000 +0200
	***************
	* 730,731 **
	--- 730,733 ----
	{ /* Add new patch number below this line */
	+ /**/
	+ 898,
	/**/

	--
	If Apple would build a car...
	... it would be powered by the sun, be reliable, five times
	as fast and twice as easy to drive; but would only run on
	five percent of the roads.

	/// 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 ///

rpms / vim

Source Code

Files