| Summary: | kmail often crashes when hitting "l" (lower L) for a list reply | ||
|---|---|---|---|
| Product: | [Applications] kmail2 | Reporter: | Marc Schiffbauer <mschiff> |
| Component: | general | Assignee: | kdepim bugs <kdepim-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | crash | CC: | montel, mschiff |
| Priority: | NOR | ||
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Compiled Sources | ||
| OS: | Linux | ||
| Latest Commit: | http://commits.kde.org/kdepim/b5b455005afceea6565f411d80e4d767da4babcf | Version Fixed In: | 4.9.2 |
| Sentry Crash Report: | |||
| Attachments: | New crash information added by DrKonqi | ||
|
Description
Marc Schiffbauer
2012-08-16 09:15:08 UTC
on the same mail ? Yeah seems to be mail specific... just tested again. One one of the mails it always crashes when hitting "l" could you send me in private ? or store as mbox and attach it to bug report please. I will send you a pm Ok find the bug. "Mail-Followup-To" header as more than 1 address => so code assert. Will look at rfc to show if we can add more than 1 address or not Git commit 61655e0f31ebbb8c27214774fecc1c9c3588b8b1 by Montel Laurent. Committed on 16/08/2012 at 23:38. Pushed by mlaurent into branch 'KDE/4.9'. Fix Bug 305248 - kmail often crashes when hitting "l" (lower L) for a list reply Don't assert some server can send several address. It's not good to crash FIXED-IN: 4.9.1 M +6 -2 messagecore/stringutil.cpp http://commits.kde.org/kdepim/61655e0f31ebbb8c27214774fecc1c9c3588b8b1 Today I got the same crash again. IIRC the bug was in pasring of Mail-Followup-To: Header The crash mail from today has a Header in following format: Mail-Followup-To: foo@bar.com, baz@, some@address.de, another@domain.de Please notice the wrong email-Adress in the first line. May that be the cause kmail crashes? ? backtrace ? Could you send me in private your message please ? Created attachment 73568 [details]
New crash information added by DrKonqi
kmail (4.9.1) on KDE Platform 4.9.00 using Qt 4.8.2
Here is the BT for the new crash. DrKonqui thinks its the same bug...
-- Backtrace (Reduced):
#11 0x00007fd58229e928 in MessageCore::StringUtil::mailboxFrom7BitString(QByteArray const&) () from /usr/lib64/libmessagecore.so.4
#12 0x00007fd581fcbc46 in MessageComposer::MessageFactory::createReply() () from /usr/lib64/libmessagecomposer.so.4
#13 0x00007fd587d70dfb in KMReplyCommand::execute (this=0x8e39cd0) at /var/tmp/portage/kde-base/kmail-4.9.49.9999/work/kmail-4.9.49.9999/kmail/kmcommands.cpp:833
#14 0x00007fd587d6fd21 in KMCommand::slotPostTransfer (this=0x8e39cd0, result=KMCommand::OK) at /var/tmp/portage/kde-base/kmail-4.9.49.9999/work/kmail-4.9.49.9999/kmail/kmcommands.cpp:268
[...]
#16 0x00007fd587d6fa0e in KMCommand::messagesTransfered (this=<optimized out>, _t1=KMCommand::OK) at /var/tmp/portage/kde-base/kmail-4.9.49.9999/work/kmail-4.9.49.9999_build/kmail/kmcommands.moc:117
Git commit b5b455005afceea6565f411d80e4d767da4babcf by Montel Laurent. Committed on 31/08/2012 at 08:35. Pushed by mlaurent into branch 'KDE/4.9'. Fix Bug 305248 - kmail often crashes when hitting "l" (lower L) for a list reply We can't parse address but don't crash FIXED-IN: 4.9.2 M +7 -0 messagecore/stringutil.cpp http://commits.kde.org/kdepim/b5b455005afceea6565f411d80e4d767da4babcf Ok we can't decode theses addresses so it crashed. Fixed now. Hi Laurent, ok, it does not crash anymore. Great! Is it intened that the To: Header is empty after trying list-reply to this mail? I gues this is because the Follow-Up Header could not be read. How about a warning to the user, that the Header could not be parsed correctly? Or use all addresses that could be read and just strip invalid addresses? same problem, we can't parse address. In to address there is a "foo@, ..." KMime code is very hard to understand. So I will look at if possible to fix it but not easy I think. |