From: Wilfried Goesgens <dothebart@citadel.org>
Date: Mon, 15 Nov 2010 11:41:55 +0000 (+0100)
Subject: smtp_do_bounce(): don't loose msg['A'] when overwriting its content.
X-Git-Tag: v8.01~581
X-Git-Url: https://code.citadel.org/?p=citadel.git;a=commitdiff_plain;h=380a65d854a73f0d323ce64251d37c9a2843e86d

smtp_do_bounce(): don't loose msg['A'] when overwriting its content.
---

diff --git a/citadel/modules/smtp/serv_smtp.c b/citadel/modules/smtp/serv_smtp.c
index 159d36914..30f46a648 100644
--- a/citadel/modules/smtp/serv_smtp.c
+++ b/citadel/modules/smtp/serv_smtp.c
@@ -1478,6 +1478,8 @@ void smtp_do_bounce(char *instr) {
         StrBufAppendBufPlain(BounceMB, HKEY("--"), 0);
 	StrBufAppendBuf(BounceMB, boundary, 0);
 	StrBufAppendBufPlain(BounceMB, HKEY("--\r\n"), 0);
+	if (bmsg->cm_fields['A'] != NULL)
+		free(bmsg->cm_fields['A']);
 	bmsg->cm_fields['A'] = SmashStrBuf(&BounceMB);
 	/* Deliver the bounce if there's anything worth mentioning */
 	CtdlLogPrintf(CTDL_DEBUG, "num_bounces = %d\n", num_bounces);