mirror of https://git.sr.ht/~rjarry/aerc
You cannot select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
5ccd2d0d51
By preparing a maliciously crafted message an attacker could send an encrypted message without signature that would appear as signed within the aerc client. It is caused by the fact that the gpg status messages, which are used for determining the validity signature, are interspered with message contents. An example of such malicious message was added to the `reader_test.go`. This change redirects the satus-fd to stderr, while the usual stderr logs are discarded to /dev/null. In addition to fixing the vulnerability described above, this has the added benefit of stdout containing only useful output which does not need to be filtered. This simplifies the logic and avoids needless copies. Previous stderr parsing logic which detected when no valid OpenPGP data was present is replaced with detecting `NODATA 1` in status-fd messages. The stderr logs are different depending on user locale, thus, they should not be parsed. On the other hand, the status-fd are relatively stable. The previous method of detecting invalid OpenPGP data would fail on systems with non-English locale. Signed-off-by: Marcin Serwin <marcin@serwin.dev> Acked-by: Robin Jarry <robin@jarry.cc> |
1 week ago | |
---|---|---|
.. | ||
auth | 2 years ago | |
calendar | 9 months ago | |
crypto | 1 week ago | |
format | 7 months ago | |
hooks | 3 months ago | |
ipc | 7 months ago | |
iterator | 2 months ago | |
log | 3 months ago | |
marker | 2 months ago | |
notmuch | 9 months ago | |
pama | 2 months ago | |
parse | 9 months ago | |
pinentry | 4 weeks ago | |
rfc822 | 2 months ago | |
send | 2 weeks ago | |
sort | 2 months ago | |
state | 3 months ago | |
templates | 2 months ago | |
ui | 2 weeks ago | |
watchers | 9 months ago | |
xdg | 2 weeks ago | |
attachment.go | 2 weeks ago | |
dirstore.go | 1 year ago | |
dirstore_test.go | 1 year ago | |
emlview.go | 2 months ago | |
history.go | 5 years ago | |
keepalive_dummy.go | 3 years ago | |
keepalive_linux.go | 3 years ago | |
messageview.go | 2 months ago | |
msgstore.go | 2 months ago | |
notmuch_version.go | 9 months ago | |
notmuch_version_dummy.go | 9 months ago | |
oauthbearer.go | 2 years ago | |
open.go | 2 weeks ago | |
structure_helpers.go | 10 months ago | |
structure_helpers_test.go | 2 years ago | |
threadbuilder.go | 2 months ago | |
xoauth2.go | 1 year ago |