commit 57e69ff2cca9edee3546373d04e0dde6810309e5
author Bram Moolenaar <Bram@vim.org> Sat Jul 30 23:05:09 2016 +0200
committer Bram Moolenaar <Bram@vim.org> Sat Jul 30 23:05:09 2016 +0200
tree 26bdc4766043b3fef13fa870d151ff3cec4ec011
parent 623e263ffb998acacd6fd0de18c44e03af2a47d1

patch 7.4.2131
Problem:    More memory leaks when using partial, e.g. for "exit-cb".
Solution:   Don't copy the callback when using a partial.

src/channel.c

diff --git a/src/channel.c b/src/channel.c
index 722dcb8..3b8b5af 100644
--- a/src/channel.c
+++ b/src/channel.c
@@ -1272,10 +1272,14 @@
 
     if (item != NULL)
     {
-	item->cq_callback = vim_strsave(callback);
 	item->cq_partial = partial;
 	if (partial != NULL)
+	{
 	    ++partial->pt_refcount;
+	    item->cq_callback = callback;
+	}
+	else
+	    item->cq_callback = vim_strsave(callback);
 	item->cq_seq_nr = id;
 	item->cq_prev = head->cq_prev;
 	head->cq_prev = item;
@@ -4465,10 +4469,14 @@
 	}
 	else
 	{
-	    job->jv_exit_cb = vim_strsave(opt->jo_exit_cb);
 	    job->jv_exit_partial = opt->jo_exit_partial;
 	    if (job->jv_exit_partial != NULL)
+	    {
+		job->jv_exit_cb = opt->jo_exit_cb;
 		++job->jv_exit_partial->pt_refcount;
+	    }
+	    else
+		job->jv_exit_cb = vim_strsave(opt->jo_exit_cb);
 	}
     }
 }

src/version.c

diff --git a/src/version.c b/src/version.c
index 77a43c1..c28c38e 100644
--- a/src/version.c
+++ b/src/version.c
@@ -764,6 +764,8 @@
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    2131,
+/**/
     2130,
 /**/
     2129,