Skip to content
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.

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

K-9 Mail crash when OpenKeychain's ContentProvider throws an exception #3943

Open
kolAflash opened this issue Mar 5, 2019 · 4 comments
Open
Labels
type: bug Something is causing incorrect behavior or errors type: third-party integration Integration with external services or platforms.

Comments

@kolAflash
Copy link

Expected behavior

open a new mail and just insert the defect mail address (I guess K9Mail will inform that it can not send when the mail address is invalid and that's just fine for this case I think)

Actual behavior

crash

Steps to reproduce

  1. Open the html in the attached zip file peene-werft-107.html.zip on your Android device. (tested with Firefox and Chromium)
  2. Tap on "Icon Briefumschlag" (the mail icon). That should bring up your default mail app - e.g. K9Mail.
  3. K9Mail will crash immediately.

This is the defect mailto link:
mailto:BITTE%20DIE%20ADRESSE%20HIER%20EINTRAGEN?subject=Link-Tipp%3A%20Peene-Werft%20in%20Wolgast%3A%20Zum%20Warten%20verdammt%20%28tagesschau.de%29&body=Link-Tipp%3A%20Peene-Werft%20in%20Wolgast%3A%20Zum%20Warten%20verdammt%20%28tagesschau.de%29%0A%0Dhttp://www.tagesschau.de/wirtschaft/peene-werft-107.html%0A%0D%0A%0D

Instead of using the attached html you may also go to the following website. But the website may have fixed the defect mailto link by now!
https://www.tagesschau.de/wirtschaft/peene-werft-107.html

Environment

K-9 Mail version: 5.600 (PlayStore)

Android version: 8.1 (LineageOS-15.1)

Account type (IMAP, POP3, WebDAV/Exchange): IMAP

@cketti cketti added type: bug Something is causing incorrect behavior or errors status: needs information Needs more information to proceed labels Mar 8, 2019
@cketti
Copy link
Member

cketti commented Mar 9, 2019

I'm not able to reproduce the crash with K-9 Mail 5.600.

Please provide a debug log. See https://github.com/k9mail/k-9/wiki/LoggingErrors

@kolAflash
Copy link
Author

I couldn't reproduce on another phone either.

crashes:
Phone: FairPhone 2
OS: Android 8.1 (LineageOS-15.1)
Xposed installed but not activated
k9_mailto_crash.txt

no crash:
Phone: Samsung Galaxy S4 Mini
OS: Android 8.1 (LineageOS-15.1)
Xposed not installed

I'll try to see if it runs if I uninstall Xposed. But I got nothing K9Mail related installed in Xposed (only: AFWall+, GravityBox, I Said Off). So I think K9Mail should work fine even with Xposed active (and indeed most of K9Mail's functionality is working without problem - just not the mailto link).

@cketti
Copy link
Member

cketti commented Mar 10, 2019

The crash seems to be caused by an exception returned when communicating with OpenKeychain.

@kolAflash
Copy link
Author

Indeed, the Samsung S4 Mini doesn't have OpenKeychain installed.

But even if the fault is on OpenKeychain's site, K9Mail shouldn't crash.

@cketti cketti changed the title crash on defect mailto link K-9 Mail crash when OpenKeychain's ContentProvider throws an exception Mar 10, 2019
@cketti cketti added type: third-party integration Integration with external services or platforms. and removed status: needs information Needs more information to proceed labels Mar 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something is causing incorrect behavior or errors type: third-party integration Integration with external services or platforms.
Projects
None yet
Development

No branches or pull requests

2 participants