From e73596f7b97f0c0deee6a04ea3c39c3a8b20fb96 Mon Sep 17 00:00:00 2001 From: Vincent Breitmoser Date: Mon, 24 Apr 2017 21:04:17 +0200 Subject: [PATCH] prevent NPE in getRevocationReason() --- .../org/sufficientlysecure/keychain/pgp/WrappedSignature.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/WrappedSignature.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/WrappedSignature.java index 49b9484b3..105fdd6c9 100644 --- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/WrappedSignature.java +++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/pgp/WrappedSignature.java @@ -130,6 +130,9 @@ public class WrappedSignature { } SignatureSubpacket p = mSig.getHashedSubPackets().getSubpacket( SignatureSubpacketTags.REVOCATION_REASON); + if (p == null) { + return null; + } // For some reason, this is missing in SignatureSubpacketInputStream:146 if (!(p instanceof RevocationReason)) { p = new RevocationReason(false, false, p.getData());