[Commits] [SCM] claws branch, master, updated. 3.14.1-137-g4850949

[wwp at claws-mail.org]

The branch, master has been updated
       via  4850949dad9b8d7c7628da69ad45a87fcb066067 (commit)
      from  93c6983e1c2f58c2ff0e580a1bf1942cf8f0f5d6 (commit)

Summary of changes:
 src/matcher.c |   10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)


- Log -----------------------------------------------------------------
commit 4850949dad9b8d7c7628da69ad45a87fcb066067
Author: wwp <wwp at free.fr>
Date:   Mon Jan 16 11:54:24 2017 +0100

    Fix yet another average leak in matcher.c when parsing headers (valgrind).

diff --git a/src/matcher.c b/src/matcher.c
index 0b56382..a423fab 100644
--- a/src/matcher.c
+++ b/src/matcher.c
@@ -1367,11 +1367,16 @@ static gboolean matcherprop_match_one_header(MatcherProp *matcher,
 				/* matching one address header exactly, is that the right one? */
 				header = procheader_parse_header(buf);
 				if (!header ||
-						!procheader_headername_equal(header->name, matcher->header))
+						!procheader_headername_equal(header->name, matcher->header)) {
+					procheader_free_header(header);
 					return FALSE;
+				}
 				address_list = address_list_append(address_list, header->body);
-				if (address_list == NULL)
+				if (address_list == NULL) {
+					procheader_free_header(header);
 					return FALSE;
+				}
+				procheader_free_header(header);
 
 			} else {
 				header = procheader_parse_header(buf);
@@ -1385,6 +1390,7 @@ static gboolean matcherprop_match_one_header(MatcherProp *matcher,
 					 procheader_headername_equal(header->name, "Reply-To") ||
 					 procheader_headername_equal(header->name, "Sender"))
 					address_list = address_list_append(address_list, header->body);
+				procheader_free_header(header);
 				if (address_list == NULL)
 					return FALSE;
 			}

-----------------------------------------------------------------------


hooks/post-receive
-- 
Claws Mail