From f03a7d0335a675e9651d67b79094736d0527f5a2 Mon Sep 17 00:00:00 2001
From: M66B <M66B@users.noreply.github.com>
Date: Sun, 23 Feb 2020 08:58:15 +0100
Subject: [PATCH] Check for no PGP sign key selected

---
 app/src/main/java/eu/faircode/email/FragmentCompose.java | 2 ++
 app/src/main/res/values/strings.xml                      | 1 +
 2 files changed, 3 insertions(+)

diff --git a/app/src/main/java/eu/faircode/email/FragmentCompose.java b/app/src/main/java/eu/faircode/email/FragmentCompose.java
index b8f46b17e3..4f1ede186a 100644
--- a/app/src/main/java/eu/faircode/email/FragmentCompose.java
+++ b/app/src/main/java/eu/faircode/email/FragmentCompose.java
@@ -1927,6 +1927,8 @@ public class FragmentCompose extends FragmentBase {
                                 }
                             } else if (OpenPgpApi.ACTION_GET_SIGN_KEY_ID.equals(data.getAction())) {
                                 pgpSignKeyId = result.getLongExtra(OpenPgpApi.EXTRA_SIGN_KEY_ID, -1);
+                                if (pgpSignKeyId == 0)
+                                    throw new IllegalArgumentException(context.getString(R.string.title_no_sign_key));
                                 db.identity().setIdentitySignKey(identity.id, pgpSignKeyId);
 
                                 // Get public key
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index be323beb41..62f7528aeb 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -814,6 +814,7 @@
     <string name="title_user_interaction">Transferring to OpenKeychain</string>
     <string name="title_signed_data">Verify the signature to show the message text</string>
     <string name="title_not_encrypted">Message is not signed or encrypted</string>
+    <string name="title_no_sign_key">No sign key selected</string>
     <string name="title_reset_sign_key">Reset sign key</string>
 
     <string name="title_signature_none">Message not signed</string>