From 80b2bfc2a5317bf1549707c695241857f935a779 Mon Sep 17 00:00:00 2001
From: James Houlahan <james.houlahan@protonmail.com>
Date: Wed, 3 Jun 2020 13:52:11 +0200
Subject: [PATCH] fix: crash in message.combineParts when copying a nil slice

---
 Changelog.md          | 1 +
 pkg/message/parser.go | 3 +++
 2 files changed, 4 insertions(+)

diff --git a/Changelog.md b/Changelog.md
index 135e63bd..fb3abed7 100644
--- a/Changelog.md
+++ b/Changelog.md
@@ -23,6 +23,7 @@ Changelog [format](http://keepachangelog.com/en/1.0.0/)
 * GODT-390 Don't logout user if AuthRefresh fails because internet was off
 * GODT-341 Fixed flaky unittest for Store synchronization cooldown
 * Crash when failing to match necessary html element
+* Crash in message.combineParts when copying nil slice
 
 ## [v1.2.7] Donghai-hotfix - beta (2020-05-07)
 
diff --git a/pkg/message/parser.go b/pkg/message/parser.go
index 6693d46b..ee7deea1 100644
--- a/pkg/message/parser.go
+++ b/pkg/message/parser.go
@@ -140,6 +140,9 @@ func combineParts(m *pmapi.Message, parts []io.Reader, headers []textproto.MIMEH
 			att := parseAttachment(filename, mediaType, h)
 
 			b := &bytes.Buffer{}
+			if d == nil {
+				continue
+			}
 			if _, err = io.Copy(b, d); err != nil {
 				continue
 			}