aboutsummaryrefslogtreecommitdiffstats
path: root/OpenKeychain/src/main/java/org/sufficientlysecure
diff options
context:
space:
mode:
authorVincent Breitmoser <valodim@mugenguild.com>2014-04-20 20:50:24 +0200
committerVincent Breitmoser <valodim@mugenguild.com>2014-04-20 20:50:26 +0200
commit9cdb2ec4e270a572520d5bcdbee55867e3c37003 (patch)
tree0bc7f9208816c18d952ba18e67fa272a9db917c5 /OpenKeychain/src/main/java/org/sufficientlysecure
parent7017a01bdb68d2cd7351dfa05825676217e42532 (diff)
downloadopen-keychain-9cdb2ec4e270a572520d5bcdbee55867e3c37003.tar.gz
open-keychain-9cdb2ec4e270a572520d5bcdbee55867e3c37003.tar.bz2
open-keychain-9cdb2ec4e270a572520d5bcdbee55867e3c37003.zip
fix nullpointer on missing s2k
Fixes #590
Diffstat (limited to 'OpenKeychain/src/main/java/org/sufficientlysecure')
-rw-r--r--OpenKeychain/src/main/java/org/sufficientlysecure/keychain/provider/ProviderHelper.java3
1 files changed, 2 insertions, 1 deletions
diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/provider/ProviderHelper.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/provider/ProviderHelper.java
index 099ed8c1f..214a9988c 100644
--- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/provider/ProviderHelper.java
+++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/provider/ProviderHelper.java
@@ -404,8 +404,9 @@ public class ProviderHelper {
values.put(Keys.HAS_SECRET, 1);
// then, mark exactly the keys we have available
for (PGPSecretKey sub : new IterableIterator<PGPSecretKey>(keyRing.getSecretKeys())) {
+ S2K s2k = sub.getS2K();
// Set to 1, except if the encryption type is GNU_DUMMY_S2K
- if(sub.getS2K().getType() != S2K.GNU_DUMMY_S2K) {
+ if(s2k == null || s2k.getType() != S2K.GNU_DUMMY_S2K) {
mContentResolver.update(uri, values, Keys.KEY_ID + " = ?", new String[]{
Long.toString(sub.getKeyID())
});