e3acc4e
To: vim-dev@vim.org
e3acc4e
Subject: Patch 7.0.134
e3acc4e
Fcc: outbox
e3acc4e
From: Bram Moolenaar <Bram@moolenaar.net>
e3acc4e
Mime-Version: 1.0
e3acc4e
Content-Type: text/plain; charset=ISO-8859-1
e3acc4e
Content-Transfer-Encoding: 8bit
e3acc4e
------------
e3acc4e
e3acc4e
Patch 7.0.134
e3acc4e
Problem:    Crash when comparing a recursively looped List or Dictionary.
e3acc4e
Solution:   Limit recursiveness for comparing to 1000.
e3acc4e
Files:	    src/eval.c
e3acc4e
e3acc4e
e3acc4e
*** ../vim-7.0.133/src/eval.c	Tue Oct 10 12:56:09 2006
e3acc4e
--- src/eval.c	Sun Oct 15 15:08:13 2006
e3acc4e
***************
e3acc4e
*** 5520,5538 ****
e3acc4e
  {
e3acc4e
      char_u	buf1[NUMBUFLEN], buf2[NUMBUFLEN];
e3acc4e
      char_u	*s1, *s2;
e3acc4e
  
e3acc4e
!     if (tv1->v_type != tv2->v_type)
e3acc4e
  	return FALSE;
e3acc4e
  
e3acc4e
      switch (tv1->v_type)
e3acc4e
      {
e3acc4e
  	case VAR_LIST:
e3acc4e
! 	    /* recursive! */
e3acc4e
! 	    return list_equal(tv1->vval.v_list, tv2->vval.v_list, ic);
e3acc4e
  
e3acc4e
  	case VAR_DICT:
e3acc4e
! 	    /* recursive! */
e3acc4e
! 	    return dict_equal(tv1->vval.v_dict, tv2->vval.v_dict, ic);
e3acc4e
  
e3acc4e
  	case VAR_FUNC:
e3acc4e
  	    return (tv1->vval.v_string != NULL
e3acc4e
--- 5520,5546 ----
e3acc4e
  {
e3acc4e
      char_u	buf1[NUMBUFLEN], buf2[NUMBUFLEN];
e3acc4e
      char_u	*s1, *s2;
e3acc4e
+     static int  recursive = 0;	    /* cach recursive loops */
e3acc4e
+     int		r;
e3acc4e
  
e3acc4e
!     /* Catch lists and dicts that have an endless loop by limiting
e3acc4e
!      * recursiveness to 1000. */
e3acc4e
!     if (tv1->v_type != tv2->v_type || recursive >= 1000)
e3acc4e
  	return FALSE;
e3acc4e
  
e3acc4e
      switch (tv1->v_type)
e3acc4e
      {
e3acc4e
  	case VAR_LIST:
e3acc4e
! 	    ++recursive;
e3acc4e
! 	    r = list_equal(tv1->vval.v_list, tv2->vval.v_list, ic);
e3acc4e
! 	    --recursive;
e3acc4e
! 	    return r;
e3acc4e
  
e3acc4e
  	case VAR_DICT:
e3acc4e
! 	    ++recursive;
e3acc4e
! 	    r = dict_equal(tv1->vval.v_dict, tv2->vval.v_dict, ic);
e3acc4e
! 	    --recursive;
e3acc4e
! 	    return r;
e3acc4e
  
e3acc4e
  	case VAR_FUNC:
e3acc4e
  	    return (tv1->vval.v_string != NULL
e3acc4e
*** ../vim-7.0.133/src/version.c	Sat Oct 14 14:33:21 2006
e3acc4e
--- src/version.c	Sun Oct 15 15:03:30 2006
e3acc4e
***************
e3acc4e
*** 668,669 ****
e3acc4e
--- 668,671 ----
e3acc4e
  {   /* Add new patch number below this line */
e3acc4e
+ /**/
e3acc4e
+     134,
e3acc4e
  /**/
e3acc4e
e3acc4e
-- 
e3acc4e
It was recently discovered that research causes cancer in rats.
e3acc4e
e3acc4e
 /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
e3acc4e
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
e3acc4e
\\\        download, build and distribute -- http://www.A-A-P.org        ///
e3acc4e
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///