talons

commit 1f3d13358f4be64e600d234db89f15d8385b2107
parent 31a342e0eeb925ed772f1a7813501866be14fe52
Author: Andrej Kacian <ticho@claws-mail.org>
Date:   Mon, 23 Oct 2017 18:06:27 +0200

Fix memory leaks introduced in recent commit 9729bf0e0.

Diffstat:
M
src/etpan/nntp-thread.c
 | 
2
++
M
src/news.c
 | 
8
++++++++

2 files changed, 10 insertions(+), 0 deletions(-)
diff --git a/src/etpan/nntp-thread.c b/src/etpan/nntp-thread.c
@@ -862,6 +862,7 @@ int nntp_threaded_xover(Folder * folder, guint32 beg, guint32 end, struct newsnn
 			debug_print("couldn't get xover for %d-%d\n", cbeg, cend);
 			if (l != NULL)
 				clist_free(l);
+			clist_free(h);
 			return result.error;
 		}
 
@@ -953,6 +954,7 @@ int nntp_threaded_xhdr(Folder * folder, const char *header, guint32 beg, guint32
 			debug_print("couldn't get xhdr %s %d-%d\n",	header, cbeg, cend);
 			if (l != NULL)
 				clist_free(l);
+			clist_free(h);
 			return result.error;
 		}
 
diff --git a/src/news.c b/src/news.c
@@ -1136,6 +1136,8 @@ static void news_get_extra_fields(NewsSession *session, FolderItem *item, GSList
 			REMOTE_FOLDER(item->folder)->session = NULL;
 		}
 		news_folder_unlock(NEWS_FOLDER(item->folder));
+		if (hdrlist != NULL)
+			clist_free(hdrlist);
 		return;
 	}
 
@@ -1161,6 +1163,8 @@ static void news_get_extra_fields(NewsSession *session, FolderItem *item, GSList
 			REMOTE_FOLDER(item->folder)->session = NULL;
 		}
 		news_folder_unlock(NEWS_FOLDER(item->folder));
+		if (hdrlist != NULL)
+			clist_free(hdrlist);
 		return;
 	}
 
@@ -1186,6 +1190,8 @@ static void news_get_extra_fields(NewsSession *session, FolderItem *item, GSList
 			REMOTE_FOLDER(item->folder)->session = NULL;
 		}
 		news_folder_unlock(NEWS_FOLDER(item->folder));
+		if (hdrlist != NULL)
+			clist_free(hdrlist);
 		return;
 	}
 
@@ -1237,6 +1243,8 @@ static GSList *news_get_msginfos_for_range(NewsSession *session, FolderItem *ite
 			REMOTE_FOLDER(item->folder)->session = NULL;
 		}
 		news_folder_unlock(NEWS_FOLDER(item->folder));
+		if (msglist != NULL)
+			clist_free(msglist);
 		return NULL;
 	}