From 9ca636589cb7c54705fec58836c20b33f3585b10 Mon Sep 17 00:00:00 2001 From: Daniel Gultsch Date: Wed, 3 Oct 2018 12:50:54 +0200 Subject: [PATCH] remove more legacy otr decryption code --- .../http/HttpUploadConnection.java | 4 ++-- .../services/AbstractConnectionManager.java | 19 ++++++------------- .../xmpp/jingle/JingleConnection.java | 4 ++-- 3 files changed, 10 insertions(+), 17 deletions(-) diff --git a/src/main/java/eu/siacs/conversations/http/HttpUploadConnection.java b/src/main/java/eu/siacs/conversations/http/HttpUploadConnection.java index cd64bcda503fbecb935f5b6603ff5f75b26bcf56..0948280e633a684c48a1c8f2d46fc0c565110bf5 100644 --- a/src/main/java/eu/siacs/conversations/http/HttpUploadConnection.java +++ b/src/main/java/eu/siacs/conversations/http/HttpUploadConnection.java @@ -119,7 +119,7 @@ public class HttpUploadConnection implements Transferable { if (method == Method.P1_S3) { try { - md5 = Checksum.md5(AbstractConnectionManager.createInputStream(file, true).first); + md5 = Checksum.md5(AbstractConnectionManager.createInputStream(file).first); } catch (Exception e) { Log.d(Config.LOGTAG, account.getJid().asBareJid()+": unable to calculate md5()", e); fail(e.getMessage()); @@ -131,7 +131,7 @@ public class HttpUploadConnection implements Transferable { Pair pair; try { - pair = AbstractConnectionManager.createInputStream(file, true); + pair = AbstractConnectionManager.createInputStream(file); } catch (FileNotFoundException e) { Log.d(Config.LOGTAG, account.getJid().asBareJid()+": could not find file to upload - "+e.getMessage()); fail(e.getMessage()); diff --git a/src/main/java/eu/siacs/conversations/services/AbstractConnectionManager.java b/src/main/java/eu/siacs/conversations/services/AbstractConnectionManager.java index ae0245ecf33697ac7b28e01f539603f7d9a674e1..860e91ab729a4fe3872aaa37788aae80b7f3d4e9 100644 --- a/src/main/java/eu/siacs/conversations/services/AbstractConnectionManager.java +++ b/src/main/java/eu/siacs/conversations/services/AbstractConnectionManager.java @@ -39,7 +39,7 @@ public class AbstractConnectionManager { this.mXmppConnectionService = service; } - public static Pair createInputStream(DownloadableFile file, boolean gcm) throws FileNotFoundException { + public static Pair createInputStream(DownloadableFile file) throws FileNotFoundException { FileInputStream is; int size; is = new FileInputStream(file); @@ -48,18 +48,11 @@ public class AbstractConnectionManager { return new Pair<>(is, size); } try { - if (gcm) { - Cipher cipher = Compatibility.twentyTwo() ? Cipher.getInstance(CIPHERMODE) : Cipher.getInstance(CIPHERMODE, PROVIDER); - SecretKeySpec keySpec = new SecretKeySpec(file.getKey(), KEYTYPE); - IvParameterSpec ivSpec = new IvParameterSpec(file.getIv()); - cipher.init(Cipher.ENCRYPT_MODE, keySpec, ivSpec); - return new Pair<>(new CipherInputStream(is, cipher), cipher.getOutputSize(size)); - } else { - IvParameterSpec ips = new IvParameterSpec(file.getIv()); - Cipher cipher = Cipher.getInstance("AES/CBC/PKCS5Padding"); - cipher.init(Cipher.ENCRYPT_MODE, new SecretKeySpec(file.getKey(), KEYTYPE), ips); - return new Pair<>(new CipherInputStream(is, cipher), (size / 16 + 1) * 16); - } + Cipher cipher = Cipher.getInstance(CIPHERMODE); + SecretKeySpec keySpec = new SecretKeySpec(file.getKey(), KEYTYPE); + IvParameterSpec ivSpec = new IvParameterSpec(file.getIv()); + cipher.init(Cipher.ENCRYPT_MODE, keySpec, ivSpec); + return new Pair<>(new CipherInputStream(is, cipher), cipher.getOutputSize(size)); } catch (Exception e) { throw new AssertionError(e); } diff --git a/src/main/java/eu/siacs/conversations/xmpp/jingle/JingleConnection.java b/src/main/java/eu/siacs/conversations/xmpp/jingle/JingleConnection.java index 5be45db7d8313bd1cac878a4d40ec74f4e698866..7fb1ee4e4497383beddc4bb7d4b5eeefee94d5ed 100644 --- a/src/main/java/eu/siacs/conversations/xmpp/jingle/JingleConnection.java +++ b/src/main/java/eu/siacs/conversations/xmpp/jingle/JingleConnection.java @@ -476,11 +476,11 @@ public class JingleConnection implements Transferable { if (message.getEncryption() == Message.ENCRYPTION_AXOLOTL) { this.file.setKey(mXmppAxolotlMessage.getInnerKey()); this.file.setIv(mXmppAxolotlMessage.getIV()); - pair = AbstractConnectionManager.createInputStream(this.file, true); + pair = AbstractConnectionManager.createInputStream(this.file); this.file.setExpectedSize(pair.second); content.setFileOffer(this.file, false, this.ftVersion).addChild(mXmppAxolotlMessage.toElement()); } else { - pair = AbstractConnectionManager.createInputStream(this.file, false); + pair = AbstractConnectionManager.createInputStream(this.file); this.file.setExpectedSize(pair.second); content.setFileOffer(this.file, false, this.ftVersion); }