From d2a387e82f29d9d8ccd213302bc3809abea04ebd Mon Sep 17 00:00:00 2001 From: Daniel Gultsch Date: Sat, 2 Oct 2021 16:44:36 +0200 Subject: [PATCH] correctly calculate socks destination --- .../xmpp/jingle/JingleSocks5Transport.java | 23 ++++++------------- 1 file changed, 7 insertions(+), 16 deletions(-) diff --git a/src/main/java/eu/siacs/conversations/xmpp/jingle/JingleSocks5Transport.java b/src/main/java/eu/siacs/conversations/xmpp/jingle/JingleSocks5Transport.java index 061da64b9..44591090e 100644 --- a/src/main/java/eu/siacs/conversations/xmpp/jingle/JingleSocks5Transport.java +++ b/src/main/java/eu/siacs/conversations/xmpp/jingle/JingleSocks5Transport.java @@ -60,22 +60,12 @@ public class JingleSocks5Transport extends JingleTransport { } else { destBuilder.append(this.connection.getTransportId()); } - if (candidate.getType() == JingleCandidate.TYPE_PROXY) { - if (candidate.isOurs()) { - destBuilder.append(this.account.getJid()); - destBuilder.append(this.connection.getId().with); - } else { - destBuilder.append(this.connection.getId().with); - destBuilder.append(this.account.getJid()); - } + if (candidate.isOurs()) { + destBuilder.append(this.account.getJid()); + destBuilder.append(this.connection.getId().with); } else { - if (connection.isInitiator()) { - destBuilder.append(this.account.getJid()); - destBuilder.append(this.connection.getId().with); - } else { - destBuilder.append(this.connection.getId().with); - destBuilder.append(this.account.getJid()); - } + destBuilder.append(this.connection.getId().with); + destBuilder.append(this.account.getJid()); } messageDigest.reset(); this.destination = CryptoHelper.bytesToHex(messageDigest.digest(destBuilder.toString().getBytes())); @@ -189,7 +179,8 @@ public class JingleSocks5Transport extends JingleTransport { socket.setSoTimeout(0); isEstablished = true; callback.established(); - } catch (IOException e) { + } catch (final IOException e) { + Log.d(Config.LOGTAG, "unable to establish connection to candidate", e); callback.failed(); } }).start();