[vim] - patchlevel 746

[Karsten Hopp]

commit 152f0f572ade93ee9e9d57542a9ff42ed1ba2760
Author: Karsten Hopp <karsten at redhat.com>
Date:   Mon Jan 28 12:11:10 2013 +0100

    - patchlevel 746

 7.3.746 |   96 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 files changed, 96 insertions(+), 0 deletions(-)
---
diff --git a/7.3.746 b/7.3.746
new file mode 100644
index 0000000..f5b4cfa
--- /dev/null
+++ b/7.3.746
@@ -0,0 +1,96 @@
+To: vim_dev at googlegroups.com
+Subject: Patch 7.3.746
+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.746
+Problem:    Memory leaks when using location lists.
+Solution:   Set qf_title to something. (Christian Brabandt)
+Files:	    src/eval.c, src/quickfix.c
+
+
+*** ../vim-7.3.745/src/eval.c	2012-10-21 23:55:59.000000000 +0200
+--- src/eval.c	2012-12-05 14:47:56.000000000 +0100
+***************
+*** 16292,16298 ****
+  		action = *act;
+  	}
+  
+! 	if (l != NULL && set_errorlist(wp, l, action, NULL) == OK)
+  	    rettv->vval.v_number = 0;
+      }
+  #endif
+--- 16292,16299 ----
+  		action = *act;
+  	}
+  
+! 	if (l != NULL && set_errorlist(wp, l, action,
+! 	       (char_u *)(wp == NULL ? "setqflist()" : "setloclist()")) == OK)
+  	    rettv->vval.v_number = 0;
+      }
+  #endif
+*** ../vim-7.3.745/src/quickfix.c	2012-11-28 22:12:40.000000000 +0100
+--- src/quickfix.c	2012-12-05 14:51:52.000000000 +0100
+***************
+*** 2124,2138 ****
+      int		idx;
+  {
+      qfline_T	*qfp;
+  
+      while (qi->qf_lists[idx].qf_count)
+      {
+  	qfp = qi->qf_lists[idx].qf_start->qf_next;
+! 	if (qi->qf_lists[idx].qf_title != NULL)
+  	{
+  	    vim_free(qi->qf_lists[idx].qf_start->qf_text);
+  	    vim_free(qi->qf_lists[idx].qf_start->qf_pattern);
+  	    vim_free(qi->qf_lists[idx].qf_start);
+  	}
+  	qi->qf_lists[idx].qf_start = qfp;
+  	--qi->qf_lists[idx].qf_count;
+--- 2124,2145 ----
+      int		idx;
+  {
+      qfline_T	*qfp;
++     int		stop = FALSE;
+  
+      while (qi->qf_lists[idx].qf_count)
+      {
+  	qfp = qi->qf_lists[idx].qf_start->qf_next;
+! 	if (qi->qf_lists[idx].qf_title != NULL && !stop)
+  	{
+  	    vim_free(qi->qf_lists[idx].qf_start->qf_text);
++ 	    stop = (qi->qf_lists[idx].qf_start == qfp);
+  	    vim_free(qi->qf_lists[idx].qf_start->qf_pattern);
+  	    vim_free(qi->qf_lists[idx].qf_start);
++ 	    if (stop)
++ 		/* Somehow qf_count may have an incorrect value, set it to 1
++ 		 * to avoid crashing when it's wrong.
++ 		 * TODO: Avoid qf_count being incorrect. */
++ 		qi->qf_lists[idx].qf_count = 1;
+  	}
+  	qi->qf_lists[idx].qf_start = qfp;
+  	--qi->qf_lists[idx].qf_count;
+*** ../vim-7.3.745/src/version.c	2012-12-05 14:42:56.000000000 +0100
+--- src/version.c	2012-12-05 15:15:45.000000000 +0100
+***************
+*** 727,728 ****
+--- 727,730 ----
+  {   /* Add new patch number below this line */
++ /**/
++     746,
+  /**/
+
+-- 
+hundred-and-one symptoms of being an internet addict:
+98. The Alta Vista administrators ask you what sites are missing
+    in their index files.
+
+ /// 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    ///