Skip to content

email.utils.parsedate_tz() crashes on certain inputs #95087

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wbolster opened this issue Jul 21, 2022 · 1 comment
Closed

email.utils.parsedate_tz() crashes on certain inputs #95087

wbolster opened this issue Jul 21, 2022 · 1 comment
Labels
topic-email type-bug An unexpected behavior, bug, or error

Comments

@wbolster
Copy link
Contributor

Date/time parsing routines in the email module can crash (instead of returning None) for certain inputs:

>>> email.utils.parsedate_tz('17 June , 2022')
IndexError: string index out of range

This is similar to bpo-45001 (GH-89164)
@wbolster wbolster added the type-bug An unexpected behavior, bug, or error label Jul 21, 2022
@wbolster wbolster mentioned this issue Jul 21, 2022
Closed
wbolster added a commit to wbolster/cpython that referenced this issue Jul 22, 2022
@wbolster
pythongh-95087: make email date parsing more robust
ee122d3 
Similar to bpo-45001 (pythonGH-89164), this makes email date parsing more
robust against malformed input. parsedate_tz() is supposed to return
None for malformed input, but could crash on certain inputs, e.g.

    >>> email.utils.parsedate_tz('17 June , 2022')
    IndexError: string index out of range

Fixes pythongh-95087.
serhiy-storchaka added a commit to serhiy-storchaka/cpython that referenced this issue Jul 24, 2022
@serhiy-storchaka @wbolster
pythongh-95087: Fix IndexError in parsing invalid date in the email m…
8fb498a 
…odule

Co-authored-by: wouter bolsterlee <[email protected]>
This was referenced Jul 24, 2022
Merged
Merged
Merged
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Jul 25, 2022
@serhiy-storchaka @miss-islington
pythongh-95087: Fix IndexError in parsing invalid date in the email m…
dd9cd77 
…odule (pythonGH-95201)

Co-authored-by: wouter bolsterlee <[email protected]>
(cherry picked from commit ea5ed0b)

Co-authored-by: Serhiy Storchaka <[email protected]>
serhiy-storchaka added a commit that referenced this issue Jul 25, 2022
@serhiy-storchaka @wbolster
gh-95087: Fix IndexError in parsing invalid date in the email module (G…
ea5ed0b 
…H-95201)

Co-authored-by: wouter bolsterlee <[email protected]>
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Jul 25, 2022
@serhiy-storchaka @miss-islington
pythongh-95087: Fix IndexError in parsing invalid date in the email m…
b8d0891 
…odule (pythonGH-95201)

Co-authored-by: wouter bolsterlee <[email protected]>
(cherry picked from commit ea5ed0b)

Co-authored-by: Serhiy Storchaka <[email protected]>
miss-islington added a commit that referenced this issue Jul 25, 2022
@miss-islington @serhiy-storchaka
gh-95087: Fix IndexError in parsing invalid date in the email module (G…
94eb1e9 
…H-95201)

Co-authored-by: wouter bolsterlee <[email protected]>
(cherry picked from commit ea5ed0b)

Co-authored-by: Serhiy Storchaka <[email protected]>
miss-islington added a commit that referenced this issue Jul 25, 2022
@miss-islington @serhiy-storchaka
gh-95087: Fix IndexError in parsing invalid date in the email module (G…
3ec139f 
…H-95201)

Co-authored-by: wouter bolsterlee <[email protected]>
(cherry picked from commit ea5ed0b)

Co-authored-by: Serhiy Storchaka <[email protected]>
@corona10
Copy link
Member

corona10 commented Jul 30, 2022 •
edited
Loading

close via #95201

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
topic-email type-bug An unexpected behavior, bug, or error
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

gh-95087: make email date parsing more robust wbolster/cpython
3 participants
@wbolster @serhiy-storchaka @corona10