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 | 294bb3977441b5b8727b82f4453d12892a248e44 (patch) | |
| tree | 590d095426a21bae3db169361d8c28e145ca66ce /lib/src/main/java/com/trilead/ssh2/signature | |
| parent | ca2025f49fa113286632fa47e9037a564766a3e5 (diff) | |
| download | sshlib-294bb3977441b5b8727b82f4453d12892a248e44.tar.gz sshlib-294bb3977441b5b8727b82f4453d12892a248e44.tar.bz2 sshlib-294bb3977441b5b8727b82f4453d12892a248e44.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 'lib/src/main/java/com/trilead/ssh2/signature')
| -rw-r--r-- | lib/src/main/java/com/trilead/ssh2/signature/ECDSASHA2Verify.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/src/main/java/com/trilead/ssh2/signature/ECDSASHA2Verify.java b/lib/src/main/java/com/trilead/ssh2/signature/ECDSASHA2Verify.java index f139cdf..7d8dd3e 100644 --- a/lib/src/main/java/com/trilead/ssh2/signature/ECDSASHA2Verify.java +++ b/lib/src/main/java/com/trilead/ssh2/signature/ECDSASHA2Verify.java @@ -294,8 +294,8 @@ public class ECDSASHA2Verify { System.arraycopy(sig, 4, rArray, 0, rLength); System.arraycopy(sig, 6 + rLength, sArray, 0, sLength); - BigInteger r = new BigInteger(rArray); - BigInteger s = new BigInteger(sArray); + BigInteger r = new BigInteger(1, rArray); + BigInteger s = new BigInteger(1, sArray); // Write the <r,s> to its own types writer. TypesWriter rsWriter = new TypesWriter(); |