mirror of
https://github.com/python/cpython.git
synced 2025-11-26 13:22:51 +00:00
bpo-33972: Fix EmailMessage.iter_attachments raising AttributeError. (GH-14119)
* bpo-33972: Fix EmailMessage.iter_attachments raising AttributeError. When certain malformed messages have content-type set to 'mulitpart/*' but still have a single part body, iter_attachments can raise AttributeError. This patch fixes it by returning a None value instead when the body is single part.
This commit is contained in:
parent
d7c87d982d
commit
02257012f6
3 changed files with 21 additions and 1 deletions
|
|
@ -1041,7 +1041,16 @@ class MIMEPart(Message):
|
|||
maintype, subtype = self.get_content_type().split('/')
|
||||
if maintype != 'multipart' or subtype == 'alternative':
|
||||
return
|
||||
parts = self.get_payload().copy()
|
||||
payload = self.get_payload()
|
||||
# Certain malformed messages can have content type set to `multipart/*`
|
||||
# but still have single part body, in which case payload.copy() can
|
||||
# fail with AttributeError.
|
||||
try:
|
||||
parts = payload.copy()
|
||||
except AttributeError:
|
||||
# payload is not a list, it is most probably a string.
|
||||
return
|
||||
|
||||
if maintype == 'multipart' and subtype == 'related':
|
||||
# For related, we treat everything but the root as an attachment.
|
||||
# The root may be indicated by 'start'; if there's no start or we
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue