Commit 342456d5 authored by Raymond Hettinger's avatar Raymond Hettinger

smptlib did not handle empty addresses.

The problem was that it expected rfc822.parseaddr() to return None
upon a parse failure.  The actual, documented return value for a
parse failure is (None, None).

Closes SF bug 602029.
parent d918884b
...@@ -168,14 +168,14 @@ def quoteaddr(addr): ...@@ -168,14 +168,14 @@ def quoteaddr(addr):
Should be able to handle anything rfc822.parseaddr can handle. Should be able to handle anything rfc822.parseaddr can handle.
""" """
m=None m = (None, None)
try: try:
m=rfc822.parseaddr(addr)[1] m=rfc822.parseaddr(addr)[1]
except AttributeError: except AttributeError:
pass pass
if not m: if m == (None, None): # Indicates parse failure or AttributeError
#something weird here.. punt -ddm #something weird here.. punt -ddm
return addr return "<%s>" % addr
else: else:
return "<%s>" % m return "<%s>" % m
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment