diff options
author | Kenny Root <kenny@the-b.org> | 2015-04-05 23:34:54 -0700 |
---|---|---|
committer | Kenny Root <kenny@the-b.org> | 2015-04-05 23:41:14 -0700 |
commit | 28b8406580e9c0565fc2090117bc06d5c28b79c2 (patch) | |
tree | 79df1d68fe485f0de14dccd980f32d1f05c73222 /src/com/trilead/ssh2/crypto | |
parent | 149080bc7dcbdbe310e39891e6a9dff0ceec8e8b (diff) | |
download | connectbot-28b8406580e9c0565fc2090117bc06d5c28b79c2.tar.gz connectbot-28b8406580e9c0565fc2090117bc06d5c28b79c2.tar.bz2 connectbot-28b8406580e9c0565fc2090117bc06d5c28b79c2.zip |
ECDH there should be no negative bigint
Since the hash is over the canonical values of the agreed parameters
when the shared secret was encoded as a negative biginteger, the two
sides didn't agree. Make sure this doesn't occur by setting the bigint
signum to 1.
Change-Id: Ib0581cd7dc280dcce8cc3309d7102f8f5a444158
Diffstat (limited to 'src/com/trilead/ssh2/crypto')
-rw-r--r-- | src/com/trilead/ssh2/crypto/dh/EcDhExchange.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/com/trilead/ssh2/crypto/dh/EcDhExchange.java b/src/com/trilead/ssh2/crypto/dh/EcDhExchange.java index 43d31ad..870a3b4 100644 --- a/src/com/trilead/ssh2/crypto/dh/EcDhExchange.java +++ b/src/com/trilead/ssh2/crypto/dh/EcDhExchange.java @@ -96,7 +96,7 @@ public class EcDhExchange extends GenericDhExchange { throw (IOException) new IOException("Invalid ECDH key").initCause(e); } - sharedSecret = new BigInteger(ka.generateSecret()); + sharedSecret = new BigInteger(1, ka.generateSecret()); } @Override |