| Summary: | a new line is inserted in a openPGP signed email during viewing, thus invalidating the signature | ||
|---|---|---|---|
| Product: | [Applications] kmail2 | Reporter: | pt8614-599 |
| Component: | crypto | Assignee: | kdepim bugs <kdepim-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | aheinecke, ecthelion, sknauss |
| Priority: | NOR | ||
| Version: | 4.12.4 | ||
| Target Milestone: | --- | ||
| Platform: | openSUSE | ||
| OS: | Linux | ||
| Latest Commit: | http://commits.kde.org/kdepimlibs/90517fa3777615a801236787e11b691480720370 | Version Fixed In: | 4.14.7 |
| Sentry Crash Report: | |||
| Attachments: |
a test email
public key |
||
|
Description
pt8614-599
2014-03-11 16:57:09 UTC
Because Bug 331991 has a text mail, i'm really sure that kmail can work with new lines in the header. And you can use "save as" to store this kind of brocken mail. The best would be if you can add a test mail with a signature that is shown broken. Created attachment 85865 [details]
a test email
Created attachment 85866 [details]
public key
to verify the signature of the test mail, you will need this public key
Now I can confirm your bug. And made sure that the signature is valid via cmdline. I'll will look into it. Seems to be the same as: https://bugs.kde.org/show_bug.cgi?id=332973 Only there it is about X.509 signatures. I also ran into this and thought it to be a Kmail1 bug as even the exported mails and if you forward such a mail as an attachment results in a corrupted signature. If you send an OpenPGP mail with attachment from KMail1 to KMail2 it will not verify because of this. https://roundup.kolab.org/issue4872 Git commit abca7ffa2be4664bfeee381a08532bf37fd98bcb by Sandro Knauß. Committed on 13/03/2015 at 20:54. Pushed by knauss into branch 'KDE/4.14'. KMIME: Do not add additional newlines, while parsing mails. If testing the signature status of mails, than nothing is allowed to change. Because the number of newlines, that seperates head and body is not specified. So make sure that at least two are written. Related: bug 332973 FIXED-IN: 15.04 M +9 -6 kmime/kmime_content.cpp M +4 -1 kmime/kmime_header_parsing.cpp M +0 -1 kmime/tests/auto/contenttest.cpp M +23 -3 kmime/tests/auto/messagetest.cpp M +1 -0 kmime/tests/auto/messagetest.h A +49 -0 kmime/tests/data/mails/dontchangemail.mbox http://commits.kde.org/kdepimlibs/abca7ffa2be4664bfeee381a08532bf37fd98bcb Git commit 90517fa3777615a801236787e11b691480720370 by Sandro Knauß. Committed on 13/03/2015 at 20:54. Pushed by knauss into branch 'KDE/4.14'. KMIME: Do not add additional newlines, while parsing mails. If testing the signature status of mails, than nothing is allowed to change. Because the number of newlines, that seperates head and body is not specified. So make sure that at least two are written. Related: bug 332973 FIXED-IN: 4.14.7 REVIEW: 122933 M +18 -12 kmime/kmime_content.cpp M +4 -1 kmime/kmime_header_parsing.cpp M +0 -1 kmime/tests/auto/contenttest.cpp M +22 -3 kmime/tests/auto/messagetest.cpp M +1 -0 kmime/tests/auto/messagetest.h A +49 -0 kmime/tests/data/mails/dontchangemail.mbox http://commits.kde.org/kdepimlibs/90517fa3777615a801236787e11b691480720370 |