From: Kevin McCarthy <kevin@8t8.us>
Date: Sun, 22 Sep 2019 22:59:27 +0000 (-0700)
Subject: Fix memory leak in compose write-fcc function
X-Git-Tag: 2019-10-25~23^2~1
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=9606fd2cb755f6d4e54fceb72856182bf2c93135;p=neomutt

Fix memory leak in compose write-fcc function

The outer multipart was not being freed after writing.

Co-authored-by: Richard Russon <rich@flatcap.org>
---

diff --git a/compose.c b/compose.c
index 94b9a076e..c0dc283b2 100644
--- a/compose.c
+++ b/compose.c
@@ -2133,10 +2133,10 @@ int mutt_compose_menu(struct Email *e, char *fcc, size_t fcclen, struct Email *e
           if (e->content->next)
             e->content = mutt_make_multipart(e->content);
 
-          if (mutt_write_fcc(mutt_b2s(&fname), e, NULL, false, NULL, NULL) < 0)
-            e->content = mutt_remove_multipart(e->content);
-          else
+          if (mutt_write_fcc(mutt_b2s(&fname), e, NULL, false, NULL, NULL) == 0)
             mutt_message(_("Message written"));
+
+          e->content = mutt_remove_multipart(e->content);
         }
         break;