From ee4d911baf39fe242a90e12163d9853728479a35 Mon Sep 17 00:00:00 2001 From: Vincent Breitmoser Date: Mon, 13 Oct 2014 15:02:42 +0200 Subject: make PgpEncryptSign and PgpDecryptVerify subclasses of BaseOperation --- .../keychain/remote/OpenPgpService.java | 32 ++-------------------- 1 file changed, 3 insertions(+), 29 deletions(-) (limited to 'OpenKeychain/src/main/java/org/sufficientlysecure/keychain/remote/OpenPgpService.java') diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/remote/OpenPgpService.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/remote/OpenPgpService.java index 1ed1a7104..f77328d34 100644 --- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/remote/OpenPgpService.java +++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/remote/OpenPgpService.java @@ -34,11 +34,9 @@ import org.spongycastle.util.encoders.Hex; import org.sufficientlysecure.keychain.pgp.exception.PgpKeyNotFoundException; import org.sufficientlysecure.keychain.ui.NfcActivity; import org.sufficientlysecure.keychain.Constants; -import org.sufficientlysecure.keychain.pgp.PassphraseCacheInterface; import org.sufficientlysecure.keychain.pgp.PgpDecryptVerify; import org.sufficientlysecure.keychain.provider.CachedPublicKeyRing; import org.sufficientlysecure.keychain.provider.KeychainDatabase.Tables; -import org.sufficientlysecure.keychain.provider.ProviderHelper.NotFoundException; import org.sufficientlysecure.keychain.operations.results.DecryptVerifyResult; import org.sufficientlysecure.keychain.pgp.PgpHelper; import org.sufficientlysecure.keychain.pgp.PgpSignEncrypt; @@ -76,28 +74,6 @@ public class OpenPgpService extends RemoteService { static final String EMAIL_SEARCH_WHERE = Tables.KEYS + "." + KeychainContract.KeyRings.IS_REVOKED + " = 0 AND " + KeychainContract.KeyRings.IS_EXPIRED + " = 0"; - private PassphraseCacheInterface passphraseCacheInterface = new PassphraseCacheInterface() { - @Override - public String getCachedPassphrase(long subKeyId) throws NoSecretKeyException { - try { - long masterKeyId = new ProviderHelper(getContext()).getMasterKeyId(subKeyId); - return getCachedPassphrase(masterKeyId, subKeyId); - } catch (NotFoundException e) { - throw new PassphraseCacheInterface.NoSecretKeyException(); - } - } - - @Override - public String getCachedPassphrase(long masterKeyId, long subKeyId) throws NoSecretKeyException { - try { - return PassphraseCacheService.getCachedPassphrase( - getContext(), masterKeyId, subKeyId); - } catch (PassphraseCacheService.KeyNotFoundException e) { - throw new PassphraseCacheInterface.NoSecretKeyException(); - } - } - }; - /** * Search database for key ids based on emails. * @@ -304,7 +280,7 @@ public class OpenPgpService extends RemoteService { // sign-only PgpSignEncrypt.Builder builder = new PgpSignEncrypt.Builder( - new ProviderHelper(getContext()), passphraseCacheInterface, + this, new ProviderHelper(getContext()), null, inputData, os ); builder.setEnableAsciiArmorOutput(asciiArmor) @@ -396,8 +372,7 @@ public class OpenPgpService extends RemoteService { InputData inputData = new InputData(is, inputLength); PgpSignEncrypt.Builder builder = new PgpSignEncrypt.Builder( - new ProviderHelper(getContext()), passphraseCacheInterface, - inputData, os + this, new ProviderHelper(getContext()), null, inputData, os ); builder.setEnableAsciiArmorOutput(asciiArmor) .setVersionHeader(PgpHelper.getVersionForHeader(this)) @@ -513,8 +488,7 @@ public class OpenPgpService extends RemoteService { InputData inputData = new InputData(is, inputLength); PgpDecryptVerify.Builder builder = new PgpDecryptVerify.Builder( - new ProviderHelper(getContext()), passphraseCacheInterface, - inputData, os + this, new ProviderHelper(getContext()), null, inputData, os ); byte[] nfcDecryptedSessionKey = data.getByteArrayExtra(OpenPgpApi.EXTRA_NFC_DECRYPTED_SESSION_KEY); -- cgit v1.2.3