[vim] - patchlevel 286

Karsten Hopp karsten at fedoraproject.org
Mon Aug 29 11:48:44 UTC 2011


commit a49bae4f01ee05b430f050c7838d38bf80c84b0a
Author: Karsten Hopp <karsten at redhat.com>
Date:   Mon Aug 29 13:45:26 2011 +0200

    - patchlevel 286

 7.3.286 |   84 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 files changed, 84 insertions(+), 0 deletions(-)
---
diff --git a/7.3.286 b/7.3.286
new file mode 100644
index 0000000..308d510
--- /dev/null
+++ b/7.3.286
@@ -0,0 +1,84 @@
+To: vim_dev at googlegroups.com
+Subject: Patch 7.3.286
+Fcc: outbox
+From: Bram Moolenaar <Bram at moolenaar.net>
+Mime-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+------------
+
+Patch 7.3.286
+Problem:    Crash when using "zd" on a large number of folds. (Sam King)
+Solution:   Recompute pointer after reallocating array.  Move fewer entries
+	    when making room.
+Files:	    src/fold.c
+
+
+*** ../vim-7.3.285/src/fold.c	2010-08-15 21:57:27.000000000 +0200
+--- src/fold.c	2011-08-26 16:03:56.000000000 +0200
+***************
+*** 1469,1479 ****
+      }
+      else
+      {
+! 	/* move nested folds one level up, to overwrite the fold that is
+  	 * deleted. */
+  	moved = fp->fd_nested.ga_len;
+  	if (ga_grow(gap, (int)(moved - 1)) == OK)
+  	{
+  	    /* adjust fd_top and fd_flags for the moved folds */
+  	    nfp = (fold_T *)fp->fd_nested.ga_data;
+  	    for (i = 0; i < moved; ++i)
+--- 1469,1482 ----
+      }
+      else
+      {
+! 	/* Move nested folds one level up, to overwrite the fold that is
+  	 * deleted. */
+  	moved = fp->fd_nested.ga_len;
+  	if (ga_grow(gap, (int)(moved - 1)) == OK)
+  	{
++ 	    /* Get "fp" again, the array may have been reallocated. */
++ 	    fp = (fold_T *)gap->ga_data + idx;
++ 
+  	    /* adjust fd_top and fd_flags for the moved folds */
+  	    nfp = (fold_T *)fp->fd_nested.ga_data;
+  	    for (i = 0; i < moved; ++i)
+***************
+*** 1486,1494 ****
+  	    }
+  
+  	    /* move the existing folds down to make room */
+! 	    if (idx < gap->ga_len)
+  		mch_memmove(fp + moved, fp + 1,
+! 					sizeof(fold_T) * (gap->ga_len - idx));
+  	    /* move the contained folds one level up */
+  	    mch_memmove(fp, nfp, (size_t)(sizeof(fold_T) * moved));
+  	    vim_free(nfp);
+--- 1489,1497 ----
+  	    }
+  
+  	    /* move the existing folds down to make room */
+! 	    if (idx + 1 < gap->ga_len)
+  		mch_memmove(fp + moved, fp + 1,
+! 				  sizeof(fold_T) * (gap->ga_len - (idx + 1)));
+  	    /* move the contained folds one level up */
+  	    mch_memmove(fp, nfp, (size_t)(sizeof(fold_T) * moved));
+  	    vim_free(nfp);
+*** ../vim-7.3.285/src/version.c	2011-08-19 22:28:58.000000000 +0200
+--- src/version.c	2011-08-26 16:07:59.000000000 +0200
+***************
+*** 711,712 ****
+--- 711,714 ----
+  {   /* Add new patch number below this line */
++ /**/
++     286,
+  /**/
+
+-- 
+To keep milk from turning sour: Keep it in the cow.
+
+ /// Bram Moolenaar -- Bram at 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    ///


More information about the scm-commits mailing list