[issue28473] mailbox.MH crashes on certain Claws Mail .mh_sequences files

2016-10-29 Thread R. David Murray
R. David Murray added the comment: OK. There is definitely something that could be done here if someone wants to tackle it. At a minimum, read the lines that we can read and produce a warning of some sort for the ones we can't. If more than one other mh style programming is allowing this va

[issue28473] mailbox.MH crashes on certain Claws Mail .mh_sequences files

2016-10-29 Thread Garrett Nievin
Garrett Nievin added the comment: Interestingly, mailutils mh (GNU Mailutils 2.99.99, Ubuntu 16.10) doesn't have a problem with these files, so I'm using that instead of nmh. I assume this is relevant: http://www.nongnu.org/nmh/diff.html The limitation on the length of lines in the file contain

[issue28473] mailbox.MH crashes on certain Claws Mail .mh_sequences files

2016-10-29 Thread Garrett Nievin
Garrett Nievin added the comment: Yes, I couldn't access the mailbox at all. Here's what happens: $ cat mymail.py ; ./mymail.py #!/usr/bin/python3 import mailbox for message in mailbox.MH('~/mail/Personal'): print(message['subject']) Traceback (most recent call last): File "/usr/lib/pyth

[issue28473] mailbox.MH crashes on certain Claws Mail .mh_sequences files

2016-10-19 Thread R. David Murray
R. David Murray added the comment: I meant 'nmh'. Also, if the problem is that mailbox blows up on an .mh_sequences file with bad records and doesn't provide any way to access the valid records (like nmh mostly manages to do), then that is something that could be fixed, and we can reopen the

[issue28473] mailbox.MH crashes on certain Claws Mail .mh_sequences files

2016-10-18 Thread R. David Murray
R. David Murray added the comment: I just checked, and nhm 1.3 produces a warning for case 1 and either fails to interpret anything past a token that starts with a '-' or it hangs. You should report the invalid sequence problem to claws mail. -- resolution: -> third party stage: -> r

[issue28473] mailbox.MH crashes on certain Claws Mail .mh_sequences files

2016-10-18 Thread R. David Murray
R. David Murray added the comment: What do you mean by "crashes"? I'm sure you don't mean a segfault. Can you paste the traceback? Neither of those two cases are valid in mh, as far as I know, so I'm not sure there is anything to fix here. -- nosy: +r.david.murray type: crash -> beh

[issue28473] mailbox.MH crashes on certain Claws Mail .mh_sequences files

2016-10-18 Thread Garrett Nievin
New submission from Garrett Nievin: Using Claws Mail, and mailbox.MH crashes on two different conditions: 1) sequence spans lines, suchly: unseen: 1-222 225-300 990-1024 1048-2048 Not Pythonic, but I fixed it with this update: def get_sequences(self): """Return a name-to-key-list d